diff options
| author | Noriko Hosoi <nhosoi@redhat.com> | 2010-06-30 10:45:03 -0700 |
|---|---|---|
| committer | Noriko Hosoi <nhosoi@redhat.com> | 2010-07-07 17:16:58 -0700 |
| commit | 55a23aa703421013764231a3dfa547c11ef5e2dc (patch) | |
| tree | e626982c14edac2cb23c8df12d27d0343620d4dd /ldap/servers/plugins/uiduniq | |
| parent | 866d13e4471c084ba83f82e2a83bb7fd14158548 (diff) | |
| download | ds-55a23aa703421013764231a3dfa547c11ef5e2dc.tar.gz ds-55a23aa703421013764231a3dfa547c11ef5e2dc.tar.xz ds-55a23aa703421013764231a3dfa547c11ef5e2dc.zip | |
609255 - fix coverity Defect Type: Memory - illegal accesses issues
https://bugzilla.redhat.com/show_bug.cgi?id=609255
12233 UNINIT Triaged Unassigned Bug Minor Fix Required
preop_modify() ds/ldap/servers/plugins/uiduniq/uid.c
Comment:
This is not an issue since attrName is an output variable
for getArguments at the line 689. But to make coverity
happy, we init attrName to NULL.
Diffstat (limited to 'ldap/servers/plugins/uiduniq')
| -rw-r--r-- | ldap/servers/plugins/uiduniq/uid.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/ldap/servers/plugins/uiduniq/uid.c b/ldap/servers/plugins/uiduniq/uid.c index 1464b527..64b5b144 100644 --- a/ldap/servers/plugins/uiduniq/uid.c +++ b/ldap/servers/plugins/uiduniq/uid.c @@ -436,7 +436,7 @@ getArguments(Slapi_PBlock *pb, char **attrName, char **markerObjectClass, if (NULL == delimiter) { /* Old style untagged parameter */ - *attrName = *argv; + *attrName = *argv; return UNTAGGED_PARAMETER; } if (strncasecmp(param, "attribute", delimiter-param) == 0) @@ -661,7 +661,7 @@ preop_modify(Slapi_PBlock *pb) BEGIN int err; - char *attrName; + char *attrName = NULL; char *markerObjectClass=NULL; char *requiredObjectClass=NULL; LDAPMod **mods; @@ -806,7 +806,7 @@ preop_modrdn(Slapi_PBlock *pb) BEGIN int err; - char *attrName; + char *attrName = NULL; char *markerObjectClass=NULL; char *requiredObjectClass=NULL; char *dn; |