diff options
| author | Stefan Metzmacher <metze@samba.org> | 2017-06-18 12:06:10 +0200 |
|---|---|---|
| committer | Stefan Metzmacher <metze@samba.org> | 2017-06-26 08:47:14 +0200 |
| commit | 5810f7468bc0c2592056d100f3174e6e2342b3b5 (patch) | |
| tree | d84b8f4b0674eff99f321a3251116a377d7772ad /source3/smbd/sesssetup.c | |
| parent | 823bc4c07add242a5a1d0cd25942ebaab3cd0e23 (diff) | |
| download | samba-5810f7468bc0c2592056d100f3174e6e2342b3b5.tar.gz | |
s3:smbd: only set user_info->auth_description on success
Otherwise we'll derefence a NULL pointer.
Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Diffstat (limited to 'source3/smbd/sesssetup.c')
| -rw-r--r-- | source3/smbd/sesssetup.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/source3/smbd/sesssetup.c b/source3/smbd/sesssetup.c index a44af7fc30b..80dc77aa468 100644 --- a/source3/smbd/sesssetup.c +++ b/source3/smbd/sesssetup.c @@ -937,9 +937,8 @@ void reply_sesssetup_and_X(struct smb_request *req) nt_status = NT_STATUS_NO_MEMORY; } - user_info->auth_description = "plaintext"; - if (NT_STATUS_IS_OK(nt_status)) { + user_info->auth_description = "plaintext"; nt_status = auth_check_password_session_info(plaintext_auth_context, req, user_info, &session_info); } |