diff options
| author | Noriko Hosoi <nhosoi@redhat.com> | 2010-06-30 10:28:14 -0700 |
|---|---|---|
| committer | Noriko Hosoi <nhosoi@redhat.com> | 2010-07-07 17:16:43 -0700 |
| commit | 866d13e4471c084ba83f82e2a83bb7fd14158548 (patch) | |
| tree | c4fa6abc04448aac1956c053eac41c3d048d42f8 /ldap/servers/plugins/uiduniq | |
| parent | 7d555304796f947e9f327041b5112bd4482f2ca9 (diff) | |
| download | ds-866d13e4471c084ba83f82e2a83bb7fd14158548.tar.gz ds-866d13e4471c084ba83f82e2a83bb7fd14158548.tar.xz ds-866d13e4471c084ba83f82e2a83bb7fd14158548.zip | |
609255 - fix coverity Defect Type: Memory - illegal accesses issues
https://bugzilla.redhat.com/show_bug.cgi?id=609255
12230 UNINIT Triaged Unassigned Bug Minor Fix Required
preop_add() ds/ldap/servers/plugins/uiduniq/7bit.c
Comment:
Some cases such as NULL attrName is passed or it does not have a value,
uninitialized "violated" is passed to slapi_ch_smprintf via issue_error.
We should init violated to NULL.
12231 UNINIT Triaged Unassigned Bug Unspecified Fix Required
preop_modify() ds/ldap/servers/plugins/uiduniq/7bit.c
Comment:
Some cases such as NULL attrName is passed or mods were empty,
uninitialized "violated" is passed to slapi_ch_smprintf via issue_error.
We should init violated to NULL.
12232 UNINIT Triaged Unassigned Bug Minor Fix Required
preop_modrdn() ds/ldap/servers/plugins/uiduniq/7bit.c
Comment:
Some cases such as NULL attrName is passed or it does not have a value,
uninitialized "violated" is passed to slapi_ch_smprintf via issue_error.
We should init violated to NULL.
Diffstat (limited to 'ldap/servers/plugins/uiduniq')
| -rw-r--r-- | ldap/servers/plugins/uiduniq/7bit.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/ldap/servers/plugins/uiduniq/7bit.c b/ldap/servers/plugins/uiduniq/7bit.c index 56eef22e..f575d267 100644 --- a/ldap/servers/plugins/uiduniq/7bit.c +++ b/ldap/servers/plugins/uiduniq/7bit.c @@ -217,7 +217,7 @@ static int preop_add(Slapi_PBlock *pb) { int result; - char *violated; + char *violated = NULL; #ifdef DEBUG slapi_log_error(SLAPI_LOG_PLUGIN, plugin_name, "ADD begin\n"); @@ -359,7 +359,7 @@ static int preop_modify(Slapi_PBlock *pb) { int result; - char *violated; + char *violated = NULL; LDAPMod **checkmods = NULL; /* holds mods to check */ int checkmodsCapacity = 0; /* max capacity of checkmods */ @@ -522,7 +522,7 @@ preop_modrdn(Slapi_PBlock *pb) { int result; Slapi_Entry *e; - char *violated; + char *violated = NULL; #ifdef DEBUG slapi_log_error(SLAPI_LOG_PLUGIN, plugin_name, |