There are tests all over the SMB1 code to check that srv_send_smb fails, but it never...

Even if the write to the socket/fd fails, we never return false and
will keep reading stuff off of the input buffer until it is exhausted
and then we will exit.

Signed-off-by: Richard Sharpe <rsharpe@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Autobuild-User(master): Richard Sharpe <sharpe@samba.org>
Autobuild-Date(master): Sat Aug  3 17:41:22 CEST 2013 on sn-devel-104

diff --git a/source3/smbd/process.c b/source3/smbd/process.c
index 5ef0fd335e2a491e0b781154e04775b4f063075f..3fbfc37578037d620bdf17f83b41ccb62f544bb1 100644 (file)
--- a/source3/smbd/process.c
+++ b/source3/smbd/process.c
@@ -194,7 +194,7 @@ out:
        SMB_PERFCOUNT_END(pcd);
 
        smbd_unlock_socket(sconn);
-       return true;
+       return (ret > 0);
 }
 
 /*******************************************************************