Hi, Bill.  

Thank you for reviewing.

On Fri, 05 Sep 2014 18:08:44 -0700
Bill Spitzak <[email protected]> wrote:

> I think if the strdup's were removed it would get rid of the memory leak 
> in a much cleaner way.

Do you mean the strdup's in handle_options()?

handle_options() copies the content of argv, so if handle_option() doesn't copy 
it and someone overwrites it, we can't get correct option strings later.
argv is not immutable and this is why handle_options() copies the content of 
argv, I think.

What do you think about that?

Thanks.
-- 
Ryo Munakata <[email protected]>
_______________________________________________
wayland-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to