summaryrefslogtreecommitdiff
path: root/source3/libsmb
diff options
context:
space:
mode:
authorVolker Lendecke <vl@samba.org>2009-05-18 07:08:34 +0200
committerVolker Lendecke <vl@samba.org>2009-05-18 07:13:04 +0200
commitf140bf2e6578e45b8603d4a6c5feef9a3b735804 (patch)
tree997ccc284229fbabc6b40f58ddd2972f2afc6679 /source3/libsmb
parent795b896ab083bb0c8336f91f0b8275ee5ef74fc1 (diff)
downloadsamba-f140bf2e6578e45b8603d4a6c5feef9a3b735804.tar.gz
samba-f140bf2e6578e45b8603d4a6c5feef9a3b735804.tar.bz2
samba-f140bf2e6578e45b8603d4a6c5feef9a3b735804.zip
Fix broken pipe handling
Metze is right: If we have *any* error at the socket level, we just can not continue. Also, apply some defensive programming: With this async stuff someone else might already have closed the socket.
Diffstat (limited to 'source3/libsmb')
-rw-r--r--source3/libsmb/async_smb.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/source3/libsmb/async_smb.c b/source3/libsmb/async_smb.c
index 96b7054b87..12bc0a6fb3 100644
--- a/source3/libsmb/async_smb.c
+++ b/source3/libsmb/async_smb.c
@@ -742,7 +742,7 @@ static void cli_smb_sent(struct tevent_req *subreq)
nwritten = writev_recv(subreq, &err);
TALLOC_FREE(subreq);
if (nwritten == -1) {
- if (err == EPIPE) {
+ if (state->cli->fd != -1) {
close(state->cli->fd);
state->cli->fd = -1;
}
@@ -791,7 +791,7 @@ static void cli_smb_received(struct tevent_req *subreq)
received = read_smb_recv(subreq, talloc_tos(), &inbuf, &err);
TALLOC_FREE(subreq);
if (received == -1) {
- if (err == EPIPE) {
+ if (cli->fd != -1) {
close(cli->fd);
cli->fd = -1;
}