diff options
| author | Noriko Hosoi <nhosoi@redhat.com> | 2010-07-02 17:23:07 -0700 |
|---|---|---|
| committer | Noriko Hosoi <nhosoi@redhat.com> | 2010-07-23 13:35:19 -0700 |
| commit | 732b57f12ed407bf7c1acb661bf48504b42b28ea (patch) | |
| tree | ffd9374394c0f590791310003a65598010a96a4a /ldap/servers/plugins/syntaxes | |
| parent | c766b7ac3ce68eae18ace54805bdd3badcaf7a80 (diff) | |
| download | ds-732b57f12ed407bf7c1acb661bf48504b42b28ea.tar.gz ds-732b57f12ed407bf7c1acb661bf48504b42b28ea.tar.xz ds-732b57f12ed407bf7c1acb661bf48504b42b28ea.zip | |
610281 - fix coverity Defect Type: Control flow issues
https://bugzilla.redhat.com/show_bug.cgi?id=610281
11815 DEADCODE Triaged Unassigned Bug Minor Fix Required
distinguishedname_validate() ds/ldap/servers/plugins/syntaxes/validate.c
Comment:
A variable val_copy is declared and initialized, but not used.
We remove these lines:
364 char *val_copy = NULL;
403 if (val_copy) {
Execution cannot reach this statement "slapi_ch_free_string(&val_c...".
404 slapi_ch_free_string(&val_copy);
405 }
Diffstat (limited to 'ldap/servers/plugins/syntaxes')
| -rw-r--r-- | ldap/servers/plugins/syntaxes/validate.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/ldap/servers/plugins/syntaxes/validate.c b/ldap/servers/plugins/syntaxes/validate.c index 989137b5..cfdb17f0 100644 --- a/ldap/servers/plugins/syntaxes/validate.c +++ b/ldap/servers/plugins/syntaxes/validate.c @@ -361,7 +361,6 @@ int distinguishedname_validate( ) { int rc = 0; /* Assume value is valid */ - char *val_copy = NULL; const char *p = begin; const char *last = NULL; @@ -400,9 +399,6 @@ int distinguishedname_validate( } exit: - if (val_copy) { - slapi_ch_free_string(&val_copy); - } return rc; } |