diff options
| author | Volker Lendecke <vl@samba.org> | 2008-10-22 17:01:28 +0200 |
|---|---|---|
| committer | Volker Lendecke <vl@samba.org> | 2008-10-22 17:53:22 +0200 |
| commit | 6c019b875515662f5c97d89d653e2708478581ad (patch) | |
| tree | 5460e6a93b1483aba16cfae02e13a7c5e48425af | |
| parent | 84df0a68702dfd3ff687f199df4e479a892640d7 (diff) | |
| download | samba-6c019b875515662f5c97d89d653e2708478581ad.tar.gz samba-6c019b875515662f5c97d89d653e2708478581ad.tar.bz2 samba-6c019b875515662f5c97d89d653e2708478581ad.zip | |
Slightly simplify reply_sesssetup_blob(): Remove an else branch
| -rw-r--r-- | source3/smbd/sesssetup.c | 21 |
1 files changed, 11 insertions, 10 deletions
diff --git a/source3/smbd/sesssetup.c b/source3/smbd/sesssetup.c index 02a66b37c3..845bf75310 100644 --- a/source3/smbd/sesssetup.c +++ b/source3/smbd/sesssetup.c @@ -125,16 +125,17 @@ static void reply_sesssetup_blob(struct smb_request *req, if (!NT_STATUS_IS_OK(nt_status) && !NT_STATUS_EQUAL(nt_status, NT_STATUS_MORE_PROCESSING_REQUIRED)) { reply_nterror(req, nt_status_squash(nt_status)); - } else { - nt_status = nt_status_squash(nt_status); - SIVAL(req->outbuf, smb_rcls, NT_STATUS_V(nt_status)); - SSVAL(req->outbuf, smb_vwv0, 0xFF); /* no chaining possible */ - SSVAL(req->outbuf, smb_vwv3, blob.length); - - if ((message_push_blob(&req->outbuf, blob) == -1) - || (push_signature(&req->outbuf) == -1)) { - reply_nterror(req, NT_STATUS_NO_MEMORY); - } + return; + } + + nt_status = nt_status_squash(nt_status); + SIVAL(req->outbuf, smb_rcls, NT_STATUS_V(nt_status)); + SSVAL(req->outbuf, smb_vwv0, 0xFF); /* no chaining possible */ + SSVAL(req->outbuf, smb_vwv3, blob.length); + + if ((message_push_blob(&req->outbuf, blob) == -1) + || (push_signature(&req->outbuf) == -1)) { + reply_nterror(req, NT_STATUS_NO_MEMORY); } } |