Add missing ERROR_NO_DATA handling to NamedPipe

[bbqsrc]

Hey @yoshuawuyts, found an interesting problem.

I needed to check that the ACLs on my named pipe were set correctly, so I used accesschk. Surprisingly, everything crashed. The error I was given: Error 232, "The pipe is being closed."

I search, find this is the define ERROR_NO_DATA. The documentation for ConnectNamedPipe says:

Otherwise, ConnectNamedPipe returns zero, and GetLastError returns ERROR_NO_DATA if the previous client has closed its handle or ERROR_PIPE_CONNECTED if it has not closed its handle.

With this patch, accesschk works, and without it, it doesn't. If you change the return value to Ok(false) it also will not work correctly.

[yoshuawuyts]

LGTM, thanks heaps!

[yoshuawuyts]

Published v0.3.4 🎉