OmBiradar opened a new pull request, #50322: URL: https://github.com/apache/arrow/pull/50322
* The Delete function now catches out of bound index and does not throw a segmentation fault. ### Rationale for this change The `KeyValueMetadata::Delete(int64_t index)` never checked for out of bounds value of `index`, & if `index` was out of bounds, then a seg fault was thrown by the program and it aborted. ### What changes are included in this PR? The `KeyValueMetadata::Delete(int64_t index)` now returns a IndexError for out of bounds values of `index` ### Are these changes tested? Yes, CI tests pass on my fork ### Are there any user-facing changes? No API changes **This PR contains a "Critical Fix".**: a bug that causes a crash The bug (out of bounds) crashes the program by causing seg fault -- 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]
