[PATCH alternative v2] ifplugd: fix netlink recv
Laurent Bercot
ska-dietlibc at skarnet.org
Tue Jul 9 20:09:31 UTC 2013
> Hi,
> here v2 of the patch that mallocs a buffer the size of pagesize and not more then 8192.
Coding style question.
Is there a reason why such temporary, local buffers should be allocated
in the heap instead of in the stack ? Variable length arrays are C99, and
have been supported by gcc for ages.
I can understand not wanting to run out of stack space, but is that a
serious risk when reserving a 8k (or less) buffer ?
I've found VLAs to be a real blessing to use: less mallocs, less
heap fragmentation, less potential leaks to worry about, more streamlined
code. Are they altogether prohibited in busybox, and if so, why ?
--
Laurent
More information about the busybox
mailing list