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

   ### Rationale for this change
   
   MakeArrayFromScalar accepts a signed length and returns an error, but 
negative lengths currently reach buffer allocation and panic with a runtime 
slice-bounds error. This affects both valid and null scalars.
   
   ### What changes are included in this PR?
   
   * Reject negative lengths before allocating any buffers.
   * Return an error wrapping arrow.ErrInvalid instead of panicking.
   
   ### Are these changes tested?
   
   Yes. The regression test covers valid and null scalars with a negative 
length. The full arrow/scalar package, assertion build, compute packages, and 
IPC 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