summaryrefslogtreecommitdiff
path: root/nsswitch
diff options
context:
space:
mode:
authorVolker Lendecke <vl@samba.org>2010-09-10 12:49:32 +0200
committerVolker Lendecke <vl@samba.org>2010-09-10 10:52:18 -0700
commit8de2c15df32d44dfffbc3d0e9c9fd105f87c6886 (patch)
tree82d04208b5dcb414758d24efb68b9c3dca933f19 /nsswitch
parentf101349c73a2c52f887cf1e3789fd42307b45297 (diff)
downloadsamba-8de2c15df32d44dfffbc3d0e9c9fd105f87c6886.tar.gz
samba-8de2c15df32d44dfffbc3d0e9c9fd105f87c6886.tar.bz2
samba-8de2c15df32d44dfffbc3d0e9c9fd105f87c6886.zip
s3: auth.krb5ccname and auth.unix_username are both fstrings
There's no point in checking for != NULL
Diffstat (limited to 'nsswitch')
-rw-r--r--nsswitch/libwbclient/wbc_pam.c6
1 files changed, 2 insertions, 4 deletions
diff --git a/nsswitch/libwbclient/wbc_pam.c b/nsswitch/libwbclient/wbc_pam.c
index 50524d2169..41c1142d4d 100644
--- a/nsswitch/libwbclient/wbc_pam.c
+++ b/nsswitch/libwbclient/wbc_pam.c
@@ -313,8 +313,7 @@ static wbcErr wbc_create_logon_info(struct winbindd_response *resp,
wbc_status = wbc_create_auth_info(resp, &i->info);
BAIL_ON_WBC_ERROR(wbc_status);
- if (resp->data.auth.krb5ccname &&
- strlen(resp->data.auth.krb5ccname)) {
+ if (resp->data.auth.krb5ccname[0] != '\0') {
wbc_status = wbcAddNamedBlob(&i->num_blobs,
&i->blobs,
"krb5ccname",
@@ -324,8 +323,7 @@ static wbcErr wbc_create_logon_info(struct winbindd_response *resp,
BAIL_ON_WBC_ERROR(wbc_status);
}
- if (resp->data.auth.unix_username &&
- strlen(resp->data.auth.unix_username)) {
+ if (resp->data.auth.unix_username[0] != '\0') {
wbc_status = wbcAddNamedBlob(&i->num_blobs,
&i->blobs,
"unix_username",