[PR] GH-49973: [C++] Fix Gandiva string length checks [arrow]

["puneetdixit200 (via GitHub)" <gi...@apache.org>]

puneetdixit200 opened a new pull request, #49984:
URL: https://github.com/apache/arrow/pull/49984

   
     ### Rationale
   
     Fixes remaining Gandiva string length safety issues from GH-49973.
   
     ### What changed
   
     - Uses checked multiplication before addition in `quote_utf8` and `to_hex_binary`.
     - Rejects negative valid input lengths in `concat_ws` before allocation/copy.
     - Adds targeted regression tests for overflow and negative length handling.
   
     ### Verification
   
     - `git diff --check`
     - `ctest --test-dir cpp\build-gandiva-vs-conda-zlib -C Release -R gandiva-precompiled-test --output-on-failure`
   


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] GH-49973: [C++] Fix Gandiva string length checks [arrow]

["puneetdixit200 (via GitHub)" <gi...@apache.org>]

puneetdixit200 commented on PR #49984:
URL: https://github.com/apache/arrow/pull/49984#issuecomment-4529139761

   Rebased this branch on current `main` (`7c9d88028`). No code changes beyond the rebase.
   
   Verified locally:
   - `git diff --check upstream/main...HEAD`


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org