You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/11/28 22:19:53 UTC

[GitHub] [arrow-rs] alamb commented on issue #3215: Off-by-one buffer size error triggers Panic when constructing RecordBatch from IPC bytes

alamb commented on issue #3215:
URL: https://github.com/apache/arrow-rs/issues/3215#issuecomment-1329830491

   Hi @aarashy  -- thank you for the report. 
   
   The validation routine you point to is basically saying that it expects that the size of the underlying buffer for the BooleanArray is at least 321 bytes, but for some reason the data has only 321. 
   
   I agree that the IPC reader should return the error rather than `panic`ing in this case.
   
   Not sure if you are up for proposing a PR to fix the issue (propagate the error) but if you are I would be most appreciative


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