<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Nov 23, 2015 at 7:54 PM, 이상진 <span dir="ltr"><<a href="mailto:lsj119@samsung.com" target="_blank">lsj119@samsung.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">






<div>
<p>>> Also (on the patch) it looks like if there is an error it destroys the<br>>> wl_socket object, could this do the wrong thing if the fd was supplied from<br>>> outside?<br><br>> So like if the socket object couldn't be initted or added to the display<br>> for some reason, it will call wl_socket_destroy() on the socket, which<br>> will close the file descriptor and the fd lock.  So I guess this<br>> question basically boils down to is it allowed for the client to close<br>> the file descriptor it was given?<br></p></div></div></div></blockquote><div>If wayland is going to call wl_socket_destroy() eventually on success, then I think it is correct that it call it on errors too, so I agree the code is correct as-is.<br><br><br></div><div> </div></div><br></div></div>