You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jira@arrow.apache.org by "David Li (Jira)" <ji...@apache.org> on 2021/09/09 13:36:00 UTC

[jira] [Commented] (ARROW-13961) [C++] iso_calendar may be uninitialized

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

David Li commented on ARROW-13961:
----------------------------------

I just merged something here from [~aucahuasi] that may be relevant. (I could take care of this when I go back and fix ARROW-13960 as well since it's all around the same code.)

> [C++] iso_calendar may be uninitialized
> ---------------------------------------
>
>                 Key: ARROW-13961
>                 URL: https://issues.apache.org/jira/browse/ARROW-13961
>             Project: Apache Arrow
>          Issue Type: Bug
>          Components: C++
>            Reporter: Neal Richardson
>            Priority: Major
>             Fix For: 6.0.0
>
>
> {code}
> /arrow/cpp/src/arrow/scalar.h:137:64: warning: ‘*((void*)& iso_calendar +8)’ may be used uninitialized in this function [-Wmaybe-uninitialized]
>   137 |       : PrimitiveScalarBase(std::move(type), true), value(value) {}
>       |                                                                ^
> In file included from /tmp/RtmpoS4YCn/file8773f4430f/src/arrow/CMakeFiles/arrow_objlib.dir/Unity/unity_17_cxx.cxx:7:
> /arrow/cpp/src/arrow/compute/kernels/scalar_temporal.cc:697:30: note: ‘*((void*)& iso_calendar +8)’ was declared here
>   697 |       std::array<int64_t, 3> iso_calendar;
> {code}
> fyi [~rokm]



--
This message was sent by Atlassian Jira
(v8.3.4#803005)