diff options
author | Endi S. Dewata <edewata@redhat.com> | 2010-07-09 20:47:32 -0500 |
---|---|---|
committer | Noriko Hosoi <nhosoi@redhat.com> | 2010-08-20 11:53:10 -0700 |
commit | 696e2f7911dae50b0ec15dcd01015f712189ed7d (patch) | |
tree | 467912d86e865934923b9610b499fb67c6b2c511 | |
parent | 404265aa364dfa5936f943e370caf23cb987e061 (diff) | |
download | ds-696e2f7911dae50b0ec15dcd01015f712189ed7d.tar.gz ds-696e2f7911dae50b0ec15dcd01015f712189ed7d.tar.xz ds-696e2f7911dae50b0ec15dcd01015f712189ed7d.zip |
Bug 613056 - fix coverify Defect Type: Null pointer dereferences issues 11892 - 11939
https://bugzilla.redhat.com/show_bug.cgi?id=613056
Resolves: bug 613056
Bug description: Fix coverify Defect Type: Null pointer dereferences issues 11892 - 11939
description: Catch possible NULL pointer in substr_dn_normalize_orig().
-rw-r--r-- | ldap/servers/slapd/dn.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/ldap/servers/slapd/dn.c b/ldap/servers/slapd/dn.c index d067d8c3..180d8094 100644 --- a/ldap/servers/slapd/dn.c +++ b/ldap/servers/slapd/dn.c @@ -333,6 +333,11 @@ substr_dn_normalize_orig( char *dn, char *end ) } } else if ( *s == '"' ) { state = B4SEPARATOR; + if (!value) { + LDAPDebug( LDAP_DEBUG_ANY, + "slapi_dn_normalize - missing value\n", 0, 0, 0 ); + break; + } if ( value_separator == dn /* 2 or more separators */ || ISSPACE( value[1] ) || ISSPACE( d[-1] ) ) { *d++ = *s; |