diff options
| author | Volker Lendecke <vl@samba.org> | 2010-09-10 12:49:32 +0200 |
|---|---|---|
| committer | Volker Lendecke <vl@samba.org> | 2010-09-10 10:52:18 -0700 |
| commit | 8de2c15df32d44dfffbc3d0e9c9fd105f87c6886 (patch) | |
| tree | 82d04208b5dcb414758d24efb68b9c3dca933f19 | |
| parent | f101349c73a2c52f887cf1e3789fd42307b45297 (diff) | |
| download | samba-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
| -rw-r--r-- | nsswitch/libwbclient/wbc_pam.c | 6 |
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", |