This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [binutils][PATCH] Avoid zero-length VLAs.
- From: Brooks Moses <bmoses at google dot com>
- To: Nick Clifton <nickc at redhat dot com>
- Cc: binutils at sourceware dot org
- Date: Mon, 11 Jul 2016 16:52:14 -0700
- Subject: Re: [binutils][PATCH] Avoid zero-length VLAs.
- Authentication-results: sourceware.org; auth=none
- References: <CAOxa4KpheNdcBF7WU66jmmh1oGG0GaSM4wcWxiRmqEdOXqH7fQ@mail.gmail.com> <982607bf-e895-dec4-f7c5-d15dc8981e0e@redhat.com> <CAOxa4KpMCfgbEkQ6At_zvYYSxdJ+ajdF+W-e3Qx47Q4VDV_zcQ@mail.gmail.com>
On Mon, Jun 13, 2016 at 9:03 AM, Brooks Moses <bmoses@google.com> wrote:
> On Mon, Jun 13, 2016 at 3:36 AM, Nick Clifton <nickc@redhat.com> wrote:
>>> Zero-length variable-length-arrays are not allowed in standard C99,
>>> and perhaps more importantly, they cause ASAN to complain. (See,
>>> e.g., https://gcc.gnu.org/ml/gcc-patches/2013-09/msg00917.html.)
>>>
>>> With this patch, the libiberty tests, including demangler-fuzzer, are
>>> ASAN-clean.
>>
>> Thanks for reporting this problem. Unfortunately you have reported it
>> to the wrong place! The libiberty code is used by both the binutils and
>> gcc projects, but it is officially maintained by gcc. So please could
>> you repost this bug report and patch to gcc-patches@gcc.gnu.org ?
>
> Will do. Thanks for the quick reply!
And following up: This is now committed to GCC as r238233.
I'd offer to do a patch for integrating it to the binutils/gdb
repository, but I don't have commit access. Looks like there are a
number of other differences in the cp-demangle.c file between the two
versions, as well.
Thanks,
- Brooks