[RFC PATCH libiberty] Fix infinite recursion in demangler

Nathan Sidwell nathan@acm.org
Wed Mar 8 14:05:00 GMT 2017


On 03/08/2017 08:29 AM, Mark Wielaard wrote:
> On Wed, 2017-03-08 at 08:22 -0500, Nathan Sidwell wrote:
>> https://gcc.gnu.org/ml/gcc-patches/2017-03/msg00089.html?
>>
>> Patch looks good to me (not a libiberty maintainer)
>>
>> One question:
>> +  if (dc == NULL || dc->d_printing > 1)
>>
>> Can dc ever be null here, or is this just extra defensiveness?
>
> Yes, dc can be NULL. There is a similar check (and setting of
> d_print_error) in d_print_comp_inner.

thanks.  in case you'd not noticed Ian T's okayed my review.  so this is 
good to go

nathan

-- 
Nathan Sidwell



More information about the Gcc-patches mailing list