Block a user
Fix 11 Gitea issues — bugs, quality, security, and refactoring
=== PR RE-REVIEW SUMMARY === Branch: fix/gitea-issues (PR #47) Head commit: fd7e98c (1 new commit since last review) 5 critical issues from previous review: ALL FIXED ✓
=== REMAINING FINDINGS…
ci: fix double CI runs on PR pushes
ci: fix double CI runs on PR pushes
=== PR REVIEW SUMMARY === Branch: fix/ci-double-trigger → main PR: #75 Files reviewed: 2 (own commits) Dimensions checked: code, build, CI, docs, quality
=== COMMITS REVIEWED === [1/2] 00b6f20…
Fix 11 Gitea issues — bugs, quality, security, and refactoring
=== PR REVIEW SUMMARY === Branch: fix/gitea-issues (PR #47) Head commit: 38be7c0 Files reviewed: 17 (13 .c files, 3 .h files, CMakeLists.txt, tests/runner.c, etc.)
=== FINDINGS SUMMARY === CRITICA…
ci: fix double CI runs on PR pushes
TCP connections lack timeout/keepalive configuration
No test coverage for sendfile zero-copy transfer path
receive_str() has no size limit — potential memory exhaustion DoS
send_n_data SSL path does not handle SSL_ERROR_WANT_WRITE properly
data_create_empty with size 0 calls malloc(0) — undefined behavior per C standard