On Wed, May 8, 2013 at 11:11 AM, Sam Liddicott <[email protected]> wrote:
> However I still question if this fix should also have been applied further
> down the function; the line:
>
>                         memcpy(existing->data + OPT_DATA + old_len, buffer,
> length);
>
> possibly should also be
>
>                         memcpy(existing->data + OPT_DATA + old_len,
> allocated ? allocated : buffer, length);

Yes, looks like it should be.
Fixed in git.
Thanks!
_______________________________________________
busybox mailing list
[email protected]
http://lists.busybox.net/mailman/listinfo/busybox

Reply via email to