jmestwa-coder opened a new pull request, #646:
URL: https://github.com/apache/logging-log4cxx/pull/646

   ### Summary
   Fixes an integer overflow in `OptionConverter::toFileSize` that can break 
log rotation.
   
   ---
   
   ### Problem
   Extremely large values (e.g. `"9999999999999999999999GB"`) can overflow a 
signed `long`, become negative, and then convert to a very large `size_t`.  
   This results in log rotation never triggering.
   
   ---
   
   ### Fix
   - Use `strtoll` for parsing
   - Reject overflow, negative values, and out-of-range results
   - Safely fall back to `defaultValue`
   
   ---
   
   ### Test
   Adds a deterministic test to verify large inputs no longer produce 
unrealistic file size limits.


-- 
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]

Reply via email to