Remove the horror that was the global smb_rw_error.

Each cli struct has it's own local copy of this variable,
so use that in client code. In the smbd server, add one
static to smbd/proccess.c and use that inside smbd. Fix
a bunch of places where smb_rw_error could be set by
calling read_data() in places where we weren't reading
from the SMB client socket (ie. winbindd).
Jeremy.

1 files changed, 2 insertions, 2 deletions

diff --git a/source/winbindd/winbindd_dual.c b/source/winbindd/winbindd_dual.c
index c96941531ac..c15af710f4d 100644
--- a/source/winbindd/winbindd_dual.c
+++ b/source/winbindd/winbindd_dual.c
@@ -44,7 +44,7 @@ static void child_read_request(struct winbindd_cli_state *state)
 	/* Read data */
 
 	len = read_data(state->sock, (char *)&state->request,
-			sizeof(state->request));
+			sizeof(state->request), NULL);
 
 	if (len != sizeof(state->request)) {
 		DEBUG(len > 0 ? 0 : 3, ("Got invalid request length: %d\n", (int)len));
@@ -72,7 +72,7 @@ static void child_read_request(struct winbindd_cli_state *state)
 	state->request.extra_data.data[state->request.extra_len] = '\0';
 
 	len = read_data(state->sock, state->request.extra_data.data,
-			state->request.extra_len);
+			state->request.extra_len, NULL);
 
 	if (len != state->request.extra_len) {
 		DEBUG(0, ("Could not read extra data\n"));