diff options
| author | Lukas Slebodnik <lslebodn@redhat.com> | 2014-06-21 17:09:31 +0200 |
|---|---|---|
| committer | Greg Hudson <ghudson@mit.edu> | 2014-07-12 13:35:47 -0400 |
| commit | b6810da129512b6d0200580d78d22d38cc214e21 (patch) | |
| tree | 26e26691707a14b770dca5ef7befcf048a4fbd86 /src/plugins | |
| parent | 315d1ef769d3e3e7cc17884623c1baaad680c26b (diff) | |
| download | krb5-b6810da129512b6d0200580d78d22d38cc214e21.zip krb5-b6810da129512b6d0200580d78d22d38cc214e21.tar.gz krb5-b6810da129512b6d0200580d78d22d38cc214e21.tar.bz2 | |
Fix error check in krb5_ldap_parse_principal_name
Test the correct variable for NULL to detect a strdup failure.
[ghudson@mit.edu: clarified commit message]
Diffstat (limited to 'src/plugins')
| -rw-r--r-- | src/plugins/kdb/ldap/libkdb_ldap/ldap_principal.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/plugins/kdb/ldap/libkdb_ldap/ldap_principal.c b/src/plugins/kdb/ldap/libkdb_ldap/ldap_principal.c index 21695a9..44bf339 100644 --- a/src/plugins/kdb/ldap/libkdb_ldap/ldap_principal.c +++ b/src/plugins/kdb/ldap/libkdb_ldap/ldap_principal.c @@ -412,7 +412,7 @@ krb5_ldap_parse_principal_name(char *i_princ_name, char **o_princ_name) at_rlm_name = strrchr(i_princ_name, '@'); if (!at_rlm_name) { *o_princ_name = strdup(i_princ_name); - if (!o_princ_name) + if (!*o_princ_name) return ENOMEM; } else { k5_buf_init_dynamic(&buf); |