Block a user
mkdir_r() has potential buffer overflow in path construction loop
chunk_size truncated to int during wire transfer — values > 2GB silently corrupted
data_compress() ignores compression_level parameter — always uses zstd default (3)
Fix 11 Gitea issues — bugs, quality, security, and refactoring
PR REVIEW SUMMARY
Branch: fix/gitea-issues → main
PR #47: "Fix 11 Gitea issues — bugs, quality, security, and refactoring"
Files reviewed: 1 C source file (`src/shared/chunk.c…
TapTap
deleted branch testing/comprehensive-test-suite from TapTap/FastSync
2026-07-20 17:05:53 +02:00
testing: add comprehensive test suite for test-driven development
testing: add comprehensive test suite for test-driven development
Verified fixes from delta (
Re-review: Approve ✅
All previous findings are fixed, CI is green across all jobs, new additions are well-structured.
Verified fixes from delta (952f693..3f73012)
Fix 11 Gitea issues — bugs, quality, security, and refactoring
quality: send_chunk has deeply nested if-else with duplicated file-sending loops
security: SSH transport uses fixed-size buffers in parse_remote_dest
quality: server_host and server_port are mutable global variables (not thread-safe)
bug: delete_extras_walk unconditionally calls rmdir on directories
bug: to_disk uses dirname() which may modify its argument (POSIX undefined)
bug: file_send_sendfile accepts compression_level parameter but ignores it
performance: entire file content loaded into RAM — no streaming for large files
quality: no unit tests for transport modules (TCP, SSH, TLS, Protocol)