git.samba.org
/
samba.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(parent:
05e5973
)
s3: Fix Coverity ID 242184 Dereference after null check
author
Volker Lendecke
<vl@samba.org>
Wed, 9 May 2012 08:30:47 +0000
(10:30 +0200)
committer
Volker Lendecke
<vl@samba.org>
Thu, 10 May 2012 07:11:57 +0000
(09:11 +0200)
rpc_query_user unconditionally dereferences user_info if successfull
source3/winbindd/winbindd_samr.c
patch
|
blob
|
history
diff --git
a/source3/winbindd/winbindd_samr.c
b/source3/winbindd/winbindd_samr.c
index c03f59381e5963f14aa9d5598f7f9a898c8adbb1..5994a354b51c748c564a4d423127565b52b6184c 100644
(file)
--- a/
source3/winbindd/winbindd_samr.c
+++ b/
source3/winbindd/winbindd_samr.c
@@
-317,11
+317,9
@@
static NTSTATUS sam_query_user(struct winbindd_domain *domain,
return NT_STATUS_NO_SUCH_USER;
}
- if (user_info) {
- user_info->homedir = NULL;
- user_info->shell = NULL;
- user_info->primary_gid = (gid_t) -1;
- }
+ user_info->homedir = NULL;
+ user_info->shell = NULL;
+ user_info->primary_gid = (gid_t) -1;
tmp_ctx = talloc_stackframe();
if (tmp_ctx == NULL) {