iemejia commented on PR #3858: URL: https://github.com/apache/avro/pull/3858#issuecomment-4951914783
This PR now also folds in the INT64_MIN block-count fix from AVRO-4275: a negative block count whose absolute value is INT64_MIN cannot be negated in int64_t (`block_count * -1` is signed-overflow UB, CWE-190), which drove an unbounded loop after the cast to `size_t`. The value reader, the datum consumer (`consume-binary.c`) and the datum skipper (`datum_skip.c`) now reject it, with `test_avro_4275` covering both arrays and maps. AVRO-4275 / #3842 is therefore superseded by this PR. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
