PeekNamedPipe now checks both for a NULL buffer and a zero-length

buffer before trying to recv() from the pipe.

PeekNamedPipe now checks both for a NULL buffer and a zero-length
341f98ad · Alex Villacís Lasso · Alexandre Julliard · b718b0b0 · 341f98ad
Commit 341f98ad authored Mar 22, 2005 by Alex Villacís Lasso Committed by Alexandre Julliard Mar 22, 2005
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 1 deletion

sync.c dlls/kernel/sync.c +3 -1

No files found.
--- a/dlls/kernel/sync.c
+++ b/dlls/kernel/sync.c
@@ -1108,6 +1108,8 @@ BOOL WINAPI PeekNamedPipe( HANDLE hPipe, LPVOID lpvBuffer, DWORD cbBuffer,
 #ifdef FIONREAD
    int avail=0, fd, ret, flags;

+    TRACE("(%p,%p,%lu,%p,%p,%p)\n", hPipe, lpvBuffer, cbBuffer, lpcbRead, lpcbAvail, lpcbMessage);
+
    ret = wine_server_handle_to_fd( hPipe, GENERIC_READ, &fd, &flags );
    if (ret)
    {
@@ -1153,7 +1155,7 @@ BOOL WINAPI PeekNamedPipe( HANDLE hPipe, LPVOID lpvBuffer, DWORD cbBuffer,
 	*lpcbAvail = avail;
    if (lpcbRead)
        *lpcbRead = 0;
-    if (avail && lpvBuffer)
+    if (avail && lpvBuffer && cbBuffer)
    {
        int readbytes = (avail < cbBuffer) ? avail : cbBuffer;
        readbytes = recv(fd, lpvBuffer, readbytes, MSG_PEEK);