diff options
| author | Toomas Soome <tsoome@me.com> | 2017-07-15 12:09:17 +0300 |
|---|---|---|
| committer | Hans Rosenfeld <hans.rosenfeld@joyent.com> | 2018-04-23 18:15:40 +0200 |
| commit | 5135ad20c9f4ed5bf570f078a3bd3a837de6c2cd (patch) | |
| tree | 19dbc89527f8c3da2a140a26ff31717051865d72 | |
| parent | 7b839a17b133f5f15cfee0a4f16c5c8445d7fa25 (diff) | |
| download | illumos-joyent-5135ad20c9f4ed5bf570f078a3bd3a837de6c2cd.tar.gz | |
9243 nsswitch/ldap: comparison between pointer and zero character constant
Reviewed by: C Fraire <cfraire@me.com>
Reviewed by: Andy Fiddaman <omnios@citrus-it.co.uk>
Approved by: Robert Mustacchi <rm@joyent.com>
| -rw-r--r-- | usr/src/lib/nsswitch/ldap/common/getprinter.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/usr/src/lib/nsswitch/ldap/common/getprinter.c b/usr/src/lib/nsswitch/ldap/common/getprinter.c index aae2f326cb..079741d61e 100644 --- a/usr/src/lib/nsswitch/ldap/common/getprinter.c +++ b/usr/src/lib/nsswitch/ldap/common/getprinter.c @@ -23,8 +23,6 @@ * Use is subject to license terms. */ -#pragma ident "%Z%%M% %I% %E% SMI" - #pragma weak _nss_ldap__printers_constr = _nss_ldap_printers_constr #include "ldap_common.h" @@ -135,9 +133,8 @@ _nss_ldap_printers2str(ldap_backend_ptr be, nss_XbyY_args_t *argp) if (strcasecmp(attr->attrname, "sun-printer-kvp") == 0) { for (j = 0; j < attr->value_count; j++) { len = strlen(attr->attrvalue[j]); - if (len < 1 || - (attr->attrvalue[j] == '\0')) { - *buffer = 0; + if (len < 1 ) { + *buffer = '\0'; nss_result = (int)NSS_STR_PARSE_PARSE; goto result_printers2str; } |