diff options
| author | Noriko Hosoi <nhosoi@redhat.com> | 2010-07-08 12:11:18 -0700 |
|---|---|---|
| committer | Noriko Hosoi <nhosoi@redhat.com> | 2010-07-23 13:43:36 -0700 |
| commit | f502808c028286d69ab1205619e232e18ae56795 (patch) | |
| tree | 026dfb3f64c2b9bd22ca252eacd2b890810aacaf /lib/libaccess/register.cpp | |
| parent | 25dfa7cf81554e8795c8cb260f7493a94f59e6fb (diff) | |
| download | ds-f502808c028286d69ab1205619e232e18ae56795.tar.gz ds-f502808c028286d69ab1205619e232e18ae56795.tar.xz ds-f502808c028286d69ab1205619e232e18ae56795.zip | |
610281 - fix coverity Defect Type: Control flow issues
https://bugzilla.redhat.com/show_bug.cgi?id=610281
11841 DEADCODE Triaged Unassigned Bug Minor Fix Required
sasl_map_new_private() ds/ldap/servers/slapd/sasl_map.c
Comment:
This new_lock NULL checking is not needed.
On this path, the condition "NULL == new_lock" cannot be true.
97 if (NULL == new_lock) {
Execution cannot reach this statement "slapi_ch_free((void**)new_p...".
98 slapi_ch_free((void**)new_priv);
99 return NULL;
100 }
11842 DEADCODE Triaged Unassigned Bug Minor Fix Required
sasl_map_insert_list_entry() ds/ldap/servers/slapd/sasl_map.c
Comment:
In sasl_map_insert_list_entry, /* Check to see if it's here
already */ was not implemented. The variable ishere pointed
by coverity should have been prepared for the purpose.
Implementing a helper function sasl_map_cmp_data.
Diffstat (limited to 'lib/libaccess/register.cpp')
0 files changed, 0 insertions, 0 deletions