[Bug 17195] NamedPipe datagrams need to be _really_ datagrams

wine-bugs at winehq.org wine-bugs at winehq.org
Wed Dec 19 11:18:35 CST 2012


http://bugs.winehq.org/show_bug.cgi?id=17195

--- Comment #90 from Adam Martinson <adam.r.martinson at gmail.com> 2012-12-19 11:18:35 CST ---
(In reply to comment #89)
> - could you merge named_pipe_peek_msg with named_pipe_recv_msg? Or at least
> move common code to a function. This way if something breaks, you will have
> only one place to fix.
I looked into that, doesn't seem reasonable as the behavior is rather
different.

> Also, this looks suspicious:
> +            while (!(end->flags & NAMED_PIPE_MESSAGE_STREAM_READ) &&
> +                   !list_empty( &end->msgs ) &&
> +                   reply->result < req->length);
It's the correct behavior, if you're reading a byte stream you might keep
going, but if you're reading a message you don't.

I'll upload my updated patches shortly, lemme know what you think.

-- 
Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email
Do not reply to this email, post in Bugzilla using the
above URL to reply.
------- You are receiving this mail because: -------
You are watching all bug changes.



More information about the wine-bugs mailing list