diff options
| author | Sumit Bose <sbose@redhat.com> | 2016-11-04 17:17:13 +0100 |
|---|---|---|
| committer | Jakub Hrozek <jhrozek@redhat.com> | 2017-02-22 12:30:16 +0100 |
| commit | 454cf0c3808a9f6a0c9f79e9796e17c58907ee6c (patch) | |
| tree | ec643bc41e4e41b546a93741f10752fc4ff8664d /src/providers | |
| parent | 08bf6b4a281ef4308119dccbba4e86cf28b505d2 (diff) | |
| download | sssd-454cf0c3808a9f6a0c9f79e9796e17c58907ee6c.tar.gz sssd-454cf0c3808a9f6a0c9f79e9796e17c58907ee6c.tar.xz sssd-454cf0c3808a9f6a0c9f79e9796e17c58907ee6c.zip | |
check_duplicate: check name member before using it
Resolves https://fedorahosted.org/sssd/ticket/3231
Reviewed-by: Petr Cech <pcech@redhat.com>
Diffstat (limited to 'src/providers')
| -rw-r--r-- | src/providers/ldap/sdap.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/providers/ldap/sdap.c b/src/providers/ldap/sdap.c index 342667aae..d562a96e2 100644 --- a/src/providers/ldap/sdap.c +++ b/src/providers/ldap/sdap.c @@ -137,7 +137,7 @@ static enum duplicate_t check_duplicate(struct sdap_attr_map *map, for (i = 0; i < num_entries; i++) { if (strcmp(map[i].sys_name, sysdb_attr) == 0) { - if (strcmp(map[i].name, ldap_attr) == 0) { + if (map[i].name != NULL && strcmp(map[i].name, ldap_attr) == 0) { return ALREADY_IN_MAP; } else { return CONFLICT_WITH_MAP; |