diff options
| author | Volker Lendecke <vl@samba.org> | 2009-05-18 07:08:34 +0200 |
|---|---|---|
| committer | Volker Lendecke <vl@samba.org> | 2009-05-18 07:13:04 +0200 |
| commit | f140bf2e6578e45b8603d4a6c5feef9a3b735804 (patch) | |
| tree | 997ccc284229fbabc6b40f58ddd2972f2afc6679 /source3/libsmb | |
| parent | 795b896ab083bb0c8336f91f0b8275ee5ef74fc1 (diff) | |
| download | samba-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.c | 4 |
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; } |