[ 
https://issues.apache.org/jira/browse/ARROW-3362?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16632913#comment-16632913
 ] 

Wes McKinney commented on ARROW-3362:
-------------------------------------

These helper functions should be promoted to a common place and add null checks 
https://github.com/apache/arrow/blob/master/cpp/src/arrow/compute/kernels/util-internal.h#L34

> [R] Guard against null buffers
> ------------------------------
>
>                 Key: ARROW-3362
>                 URL: https://issues.apache.org/jira/browse/ARROW-3362
>             Project: Apache Arrow
>          Issue Type: Improvement
>          Components: R
>            Reporter: Wes McKinney
>            Priority: Major
>             Fix For: 0.12.0
>
>
> The R C++ bindings feature a number of cases where a data buffer is being 
> accessed when it may possibly be null in the case of a 0-length array. This 
> will cause a segfault
> https://github.com/apache/arrow/blob/master/r/src/array.cpp#L245
> I suggest defining some helper functions as we have done elsewhere to assist 
> with getting a typed offset into a buffer but passing through nullptr if the 
> buffer object is null



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to