[PATCH] rtld: Reject overly long LD_AUDIT path elements

Florian Weimer fweimer@redhat.com
Mon Jun 26 09:46:00 GMT 2017


On 06/26/2017 11:40 AM, Andreas Schwab wrote:
> On Jun 26 2017, Florian Weimer <fweimer@redhat.com> wrote:
> 
>> On 06/26/2017 10:47 AM, Andreas Schwab wrote:
>>
>>> Why do you need all that complexity?
>>
>> Do you mean the external iterator instead of just writing a loop?
>>
>> I did this so that the patch can be backported without changes.
>> Otherwise, we would have to reindent the audit module process loop (or
>> move it to separate function for clarity), and this will introduce a lot
>> of conflicts because there have been quite a few trivial changes to the
>> loop body.
> 
> That's not a good reason for overly complex code.

I had hoped to clean this up after the new dl-minimal malloc went in,
but I can do so now if you want that.

Thanks,
Florian



More information about the Libc-alpha mailing list