Properly initialise pBytesTransferred before WaitForOverlappedOperation returns.

Returning an uninitialised value was causing copying of random amounts of data
in bbackupd on command socket disconnect, and crashes.

1 files changed, 2 insertions, 0 deletions

diff --git a/lib/server/WinNamedPipeStream.cpp b/lib/server/WinNamedPipeStream.cpp
index de2822a8..448a3c9d 100644
--- a/lib/server/WinNamedPipeStream.cpp
+++ b/lib/server/WinNamedPipeStream.cpp
@@ -311,6 +311,7 @@ bool WinNamedPipeStream::WaitForOverlappedOperation(OVERLAPPED& Overlapped,
 	if (err == ERROR_HANDLE_EOF)
 	{
 		Close();
+		*pBytesTransferred = 0;
 		return true;
 	}
 
@@ -324,6 +325,7 @@ bool WinNamedPipeStream::WaitForOverlappedOperation(OVERLAPPED& Overlapped,
 		BOX_INFO(BOX_WIN_ERRNO_MESSAGE(err,
 			"Named pipe peer disconnected"));
 		Close();
+		*pBytesTransferred = 0;
 		return true;
 	}