diff options
| author | Noriko Hosoi <nhosoi@redhat.com> | 2010-07-02 17:42:37 -0700 |
|---|---|---|
| committer | Noriko Hosoi <nhosoi@redhat.com> | 2010-07-23 13:35:26 -0700 |
| commit | 9533a4085bd1b5d462e322e366e470218f2b03b1 (patch) | |
| tree | ec8f5736df9cd108a07c038047a96e9e9793bee4 | |
| parent | 732b57f12ed407bf7c1acb661bf48504b42b28ea (diff) | |
610281 - fix coverity Defect Type: Control flow issues
https://bugzilla.redhat.com/show_bug.cgi?id=610281
11816 DEADCODE Triaged Unassigned Bug Moderate Fix Required
NS7bitAttr_Init() ds/ldap/servers/plugins/uiduniq/7bit.c
Comment:
NS7bitAttr_Init declared err twice. One at the top and another
in BEGIN - END (do - while loop). The second err in the do -
while loop is trashed when it gets out of the loop. Regardless
of the result in the do - while loop, err = 0 (success) was
returned to the caller.
We are removing the second err in the BEGIN (== do - while) scope.
692 int err;
| -rw-r--r-- | ldap/servers/plugins/uiduniq/7bit.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/ldap/servers/plugins/uiduniq/7bit.c b/ldap/servers/plugins/uiduniq/7bit.c index f575d267..f6120d65 100644 --- a/ldap/servers/plugins/uiduniq/7bit.c +++ b/ldap/servers/plugins/uiduniq/7bit.c @@ -689,7 +689,6 @@ NS7bitAttr_Init(Slapi_PBlock *pb) int err = 0; BEGIN - int err; int argc; char **argv; |