[PATCH] Unbound alloca in crypt routines

Jeff Law law@redhat.com
Thu Mar 29 17:22:00 GMT 2012


On 03/29/2012 11:10 AM, Roland McGrath wrote:
> The pontificating comments are inappropriate.  The use of
> __libc_use_alloca is well established throughout the code.
> It does not require any special comment, let alone a snarky one.
Just because it's well established throughout the code doesn't mean 
using alloca is good practice.  In fact, I would argue that the number 
of fixes folks have had to make to deal with unbound allocas is good 
evidence that the practice of using alloca is bad.

However, I'll certainly remove that part of the comment if you object to 
it, and it sounds like you do.  Aside from that part of the comment, 
what others issues need to be addressed?

jeff



More information about the Libc-alpha mailing list