fallintoplace opened a new pull request, #1018:
URL: https://github.com/apache/arrow-go/pull/1018

   ### Rationale for this change\n\nListViewBuilder.AppendValuesWithSizes can 
accept too few dimensions or ranges that do not fit within the child values. 
NewArray can then return an invalid ListView array that fails when read, 
sliced, or validated.\n\n### What changes are included in this PR?\n\n* Require 
enough offsets and sizes for every logical list.\n* Reject negative sizes, 
negative offsets for non-empty ranges, and ranges beyond the child array.\n* 
Apply the checks to both ListView and LargeListView builders while preserving 
valid extra offset capacity.\n\n### Are these changes tested?\n\nYes. The tests 
cover missing offsets, missing sizes, negative sizes, negative offsets, and 
ranges beyond the child values for both offset widths. The full arrow/array 
package, assertion build, compute packages, IPC package, and CSV package also 
pass.


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