fix: fix issue #47

[pedriniedoardo]

No description provided.

[qodo-code-review]

Review Summary by Qodo

Fix integer overflow in chunk offset calculation

🐞 Bug fix

Walkthroughs

Description

• Convert chunk index and size to numeric before multiplication
• Prevents integer overflow in offset calculation for large files
• Applies fix to both server and R2 upload functions

Diagram

flowchart LR
  A["chunk_idx * chunk_size"] -->|"integer overflow risk"| B["offset calculation"]
  C["as.numeric conversion"] -->|"prevents overflow"| B
  B --> D["Upload chunk functions"]

Loading

File Changes

1. R/client.R 🐞 Bug fix +2/-2

Add numeric conversion to offset calculations

• Convert chunk_idx and chunk_size to numeric type before multiplication in
 .upload_chunk_server() function
• Convert chunk_idx and chunk_size to numeric type before multiplication in .upload_chunk_r2()
 function
• Prevents integer overflow when calculating file offset for large file uploads

R/client.R

---

[qodo-code-review]

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

ⓘ The new review experience is currently in Beta. Learn more