summaryrefslogtreecommitdiff
path: root/source3/nsswitch
diff options
context:
space:
mode:
authorMartin Pool <mbp@samba.org>2003-02-18 01:09:19 +0000
committerMartin Pool <mbp@samba.org>2003-02-18 01:09:19 +0000
commit5511347b8d5a974d5cd0536eded92c4b77c1091d (patch)
treecaf68b84519307d7fe8d8651389ad0c01e95445d /source3/nsswitch
parent335fbbfbc957dea02c37152ec690cdcf54040ae1 (diff)
downloadsamba-5511347b8d5a974d5cd0536eded92c4b77c1091d.tar.gz
samba-5511347b8d5a974d5cd0536eded92c4b77c1091d.tar.bz2
samba-5511347b8d5a974d5cd0536eded92c4b77c1091d.zip
Check return code from string_to_sid.
(This used to be commit fe449328b226a33ad1c64e087fe63e79f90ab4ac)
Diffstat (limited to 'source3/nsswitch')
-rw-r--r--source3/nsswitch/wb_client.c9
1 files changed, 6 insertions, 3 deletions
diff --git a/source3/nsswitch/wb_client.c b/source3/nsswitch/wb_client.c
index 9ac1515d7d..62c9686960 100644
--- a/source3/nsswitch/wb_client.c
+++ b/source3/nsswitch/wb_client.c
@@ -56,7 +56,8 @@ BOOL winbind_lookup_name(const char *dom_name, const char *name, DOM_SID *sid,
if ((result = winbindd_request(WINBINDD_LOOKUPNAME, &request,
&response)) == NSS_STATUS_SUCCESS) {
- string_to_sid(sid, response.data.sid.sid);
+ if (!string_to_sid(sid, response.data.sid.sid))
+ return False;
*name_type = (enum SID_NAME_USE)response.data.sid.type;
}
@@ -158,7 +159,8 @@ BOOL winbind_uid_to_sid(DOM_SID *sid, uid_t uid)
/* Copy out result */
if (result == NSS_STATUS_SUCCESS) {
- string_to_sid(sid, response.data.sid.sid);
+ if (!string_to_sid(sid, response.data.sid.sid))
+ return False;
} else {
sid_copy(sid, &global_sid_NULL);
}
@@ -224,7 +226,8 @@ BOOL winbind_gid_to_sid(DOM_SID *sid, gid_t gid)
/* Copy out result */
if (result == NSS_STATUS_SUCCESS) {
- string_to_sid(sid, response.data.sid.sid);
+ if (!string_to_sid(sid, response.data.sid.sid))
+ return False;
} else {
sid_copy(sid, &global_sid_NULL);
}