diff options
| author | Noriko Hosoi <nhosoi@redhat.com> | 2010-06-30 16:54:58 -0700 |
|---|---|---|
| committer | Noriko Hosoi <nhosoi@redhat.com> | 2010-07-07 17:15:48 -0700 |
| commit | 4b2c04ee518ab078e18707871d6c7a88f00b2d5c (patch) | |
| tree | 57912c9274d837892051b226251acc84ad2abebb /ldap/servers/slapd/entry.c | |
| parent | 7fa74e30e36274f561ef61f3cae29abf68ce05ae (diff) | |
| download | ds-4b2c04ee518ab078e18707871d6c7a88f00b2d5c.tar.gz ds-4b2c04ee518ab078e18707871d6c7a88f00b2d5c.tar.xz ds-4b2c04ee518ab078e18707871d6c7a88f00b2d5c.zip | |
609255 - fix coverity Defect Type: Memory - illegal accesses issues
https://bugzilla.redhat.com/show_bug.cgi?id=609255
12210 UNINIT Triaged Unassigned Bug Minor Fix Required
str2entry_fast() ds/ldap/servers/slapd/entry.c
Comment:
ptype is always NULL the first time through the loop, so a will
always be initialized to NULL. But we should explicitly initialize
it to NULL in the declaration.
Diffstat (limited to 'ldap/servers/slapd/entry.c')
| -rw-r--r-- | ldap/servers/slapd/entry.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ldap/servers/slapd/entry.c b/ldap/servers/slapd/entry.c index 0a8c40d5..87765d50 100644 --- a/ldap/servers/slapd/entry.c +++ b/ldap/servers/slapd/entry.c @@ -165,6 +165,7 @@ str2entry_fast( const char *rawdn, char *s, int flags, int read_stateinfo ) CSN *maxcsn = NULL; char *normdn = NULL; int strict = 0; + Slapi_Attr **a = NULL; /* Check if we should be performing strict validation. */ strict = config_get_dn_validate_strict(); @@ -206,7 +207,6 @@ str2entry_fast( const char *rawdn, char *s, int flags, int read_stateinfo ) while ( (s = ldif_getline( &next )) != NULL && attr_val_cnt < ENTRY_MAX_ATTRIBUTE_VALUE_COUNT ) { - Slapi_Attr **a; char *valuecharptr=NULL; #if defined(USE_OPENLDAP) ber_len_t valuelen; |