<div dir="ltr">Hi Jackek,<div><br></div><div>I went over your patch but I believe it might have few issues that i wanted to raise before we change anything</div><div>1. For some reason when you are doing the "ok" test after CreateNamedPipe you do against != NULL should != INVALID_HANDLE_VALUE</div><div>    this is critical... since the 2nd server might have failed with OBJECT_NAME_ALREADY_EXISTS</div><div>2. When you create the 2nd server instance against the "ok" check is against NULL but not only that, you check against "server" and not "server2"</div><div><br></div><div>The reason I'm raising these issues is because from my understanding of Windows, there can be only 1 instance for a given named pipe instance,</div><div>Please see that once you change these things, my logic remains correct... and that simply server2 was never created and got INVALID_HANDLE_VALUE</div><div>with a should be GLE of 231 aka ERROR_PIPE_BUSY.</div><div><br></div><div>Please let me know how it goes so we will know how to continue.</div><div><br></div><div>Thanks and I hope you have a great weekend,</div><div>-- Jon.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Sep 27, 2017 at 6:33 PM, Jacek Caban <span dir="ltr"><<a href="mailto:jacek@codeweavers.com" target="_blank">jacek@codeweavers.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Jonathan,<br>
<br>
Your tests still don't really show intended behaviour. Your calls that<br>
set security descriptor use new SD that has the same tested values as<br>
before the call, then what's the point in setting it? Anyway, I cleaned<br>
it up and changed a bit to see how it's really supposed to work. See the<br>
attachment.<br>
<br>
It looks like client and served should indeed share security descriptor,<br>
but pipe object should not, so that part is wrong.<br>
<br>
Thanks,<br>
Jacek<br>
</blockquote></div><br></div>