[PATCH v3] unstrip: Stop using strndupa.

Mark Wielaard mark@klomp.org
Tue Oct 27 22:09:32 GMT 2020


Hi Érico,

On Tue, Oct 27, 2020 at 04:19:58PM -0300, Érico Nogueira via Elfutils-devel wrote:
> This functon has inherent safety issues, since a long enough path can
> lead to memory clobbering. Due to the recursive nature of
> make_directories(), multiple calls could also stack overflow. Instead,
> the string can be allocated in the heap.
> 
> As a bonus, this improves musl compatibility, since musl doesn't include
> the strndupa macro for now.
> 
> Also add braces around while loop.

Looks good. Pushed.
I did move and fixup the ChangeLog entry.
(each subdir has its own ChangeLog and they should tell what exactly change)

Thanks,

Mark


More information about the Elfutils-devel mailing list