ldap/servers/plugins/collation/collate.c | 24 +++----
ldap/servers/plugins/replication/windows_protocol_util.c | 9 +-
ldap/servers/slapd/back-ldbm/index.c | 17 ++---
ldap/servers/slapd/back-ldbm/ldbm_entryrdn.c | 24 +------
ldap/servers/slapd/back-ldbm/ldbm_search.c | 2
ldap/servers/slapd/back-ldbm/seq.c | 2
ldap/servers/slapd/bind.c | 2
ldap/servers/slapd/log.c | 46 +++------------
8 files changed, 40 insertions(+), 86 deletions(-)
New commits:
commit db673270e9a01e48adb22d94e676f25a223f407b
Author: Mark Reynolds <mreynolds(a)redhat.com>
Date: Tue Jun 4 13:25:22 2013 -0400
Coverity Fixes (Part 2)
11674 - logically dead code (collate.c)
11675 - logically dead code (windows_protocol_util.c)
11676 - logically dead code (index.c)
11677 - logically dead code (ldbm_entryrdn.c)
11678 - logically dead code (ldbm_entryrdn.c)
11680 - logically dead code (ldbm_search.c)
11681 - Dead default in switch (seq.c)
11683 - logically dead code (bind.c)
11684 - logically dead code (connection.c)
11686 - Dead default in switch (log.c)
11687 - Dead default in switch (log.c)
11688 - Dead default in switch (log.c)
11689 - Dead default in switch (log.c)
11690 - Dead default in switch (log.c)
11691 - Dead default in switch (log.c)
https://bugzilla.redhat.com/show_bug.cgi?id=970221
Reviewed by: nhosoi(Thanks!)
diff --git a/ldap/servers/plugins/collation/collate.c
b/ldap/servers/plugins/collation/collate.c
index 240da70..2a73ee1 100644
--- a/ldap/servers/plugins/collation/collate.c
+++ b/ldap/servers/plugins/collation/collate.c
@@ -437,11 +437,7 @@ collation_indexer_create (const char* oid)
profile->variant == NULL);
UErrorCode err = U_ZERO_ERROR;
if ( ! is_default) {
- if (locale) {
- PR_smprintf_free(locale);
- locale = NULL;
- }
- err = s_newNamedLocaleFromComponents(&locale,
+ err = s_newNamedLocaleFromComponents(&locale,
profile->language,
profile->country,
profile->variant);
@@ -460,27 +456,27 @@ collation_indexer_create (const char* oid)
ucol_setAttribute (coll, UCOL_STRENGTH, profile->strength, &err);
if (err != U_ZERO_ERROR && err != U_USING_FALLBACK_WARNING
&& (err != U_USING_DEFAULT_WARNING || !is_default)) {
- LDAPDebug (LDAP_DEBUG_ANY, "collation_indexer_create: could not "
+ LDAPDebug (LDAP_DEBUG_ANY, "collation_indexer_create: could not "
"set the collator strength for oid %s to %d: err %d\n",
oid, profile->strength, err);
}
ucol_setAttribute (coll, UCOL_DECOMPOSITION_MODE, profile->decomposition,
&err);
if (err != U_ZERO_ERROR && err != U_USING_FALLBACK_WARNING
&& (err != U_USING_DEFAULT_WARNING || !is_default)) {
- LDAPDebug (LDAP_DEBUG_ANY, "collation_indexer_create: could not "
+ LDAPDebug (LDAP_DEBUG_ANY, "collation_indexer_create: could not "
"set the collator decomposition mode for oid %s to %d: err %d\n",
oid, profile->decomposition, err);
}
etc->collator = coll;
for (id = collation_id; *id; ++id) {
- if ((*id)->profile == profile) {
- break; /* found the 'official' id */
- }
+ if ((*id)->profile == profile) {
+ break; /* found the 'official' id */
+ }
}
- if (!*id) {
- LDAPDebug (LDAP_DEBUG_ANY, "collation_indexer_create: id not found\n", 0, 0,
0);
- goto error;
- }
+ if (!*id) {
+ LDAPDebug (LDAP_DEBUG_ANY, "collation_indexer_create: id not
found\n", 0, 0, 0);
+ goto error;
+ }
ix->ix_etc = etc;
ix->ix_oid = (*id)->oid;
diff --git a/ldap/servers/plugins/replication/windows_protocol_util.c
b/ldap/servers/plugins/replication/windows_protocol_util.c
index 03d6ff3..964566a 100644
--- a/ldap/servers/plugins/replication/windows_protocol_util.c
+++ b/ldap/servers/plugins/replication/windows_protocol_util.c
@@ -4249,7 +4249,7 @@ windows_generate_update_mods(Private_Repl_Protocol *prp,Slapi_Entry
*remote_entr
windows_is_remote_entry_user_or_group(remote_entry,&is_user,&is_group);
}
- for (rc = slapi_entry_first_attr(remote_entry, &attr); rc == 0;
+ for (rc = slapi_entry_first_attr(remote_entry, &attr); rc == 0;
rc = slapi_entry_next_attr(remote_entry, attr, &attr))
{
int is_present_local = 0;
@@ -4260,7 +4260,6 @@ windows_generate_update_mods(Private_Repl_Protocol *prp,Slapi_Entry
*remote_entr
int is_guid = 0;
int mapdn = 0;
-
slapi_attr_get_type( attr, &type );
slapi_attr_get_valueset(attr,&vs);
@@ -4419,7 +4418,7 @@ windows_generate_update_mods(Private_Repl_Protocol *prp,Slapi_Entry
*remote_entr
"windows_generate_update_mods: no restricted local values found for
"
"local attribute %s in local entry %s for remote attribute "
"%s in remote entry %s\n",
- local_type ? local_type : "NULL",
+ local_type,
slapi_entry_get_dn(local_entry),
type ? type : "NULL",
slapi_entry_get_dn(remote_entry));
@@ -4431,7 +4430,7 @@ windows_generate_update_mods(Private_Repl_Protocol *prp,Slapi_Entry
*remote_entr
"windows_generate_update_mods: no local values found for "
"local attribute %s in local entry %s for remote attribute "
"%s in remote entry %s\n",
- local_type ? local_type : "NULL",
+ local_type,
slapi_entry_get_dn(local_entry),
type ? type : "NULL",
slapi_entry_get_dn(remote_entry));
@@ -4443,7 +4442,7 @@ windows_generate_update_mods(Private_Repl_Protocol *prp,Slapi_Entry
*remote_entr
"windows_generate_update_mods: could not map the values in "
"local attribute %s in local entry %s for remote attribute "
"%s in remote entry %s\n",
- local_type ? local_type : "NULL",
+ local_type,
slapi_entry_get_dn(local_entry),
type ? type : "NULL",
slapi_entry_get_dn(remote_entry));
diff --git a/ldap/servers/slapd/back-ldbm/index.c b/ldap/servers/slapd/back-ldbm/index.c
index a10c67a..e45aba8 100644
--- a/ldap/servers/slapd/back-ldbm/index.c
+++ b/ldap/servers/slapd/back-ldbm/index.c
@@ -649,18 +649,17 @@ index_add_mods(
mods[i]->mod_op |= LDAP_MOD_IGNORE;
}
}
- if (mods_valueArray) {
- rc = index_addordel_values_sv( be,
- mods[i]->mod_type,
- mods_valueArray, NULL,
- id, BE_INDEX_ADD, txn );
- if (rc) {
- ldbm_nasty(errmsg, 1042, rc);
- goto error;
- }
+ if(mods_valueArray[0]){
+ rc = index_addordel_values_sv( be, mods[i]->mod_type,
+ mods_valueArray, NULL,
+ id, BE_INDEX_ADD, txn );
} else {
rc = 0;
}
+ if (rc) {
+ ldbm_nasty(errmsg, 1042, rc);
+ goto error;
+ }
}
}
break;
diff --git a/ldap/servers/slapd/back-ldbm/ldbm_entryrdn.c
b/ldap/servers/slapd/back-ldbm/ldbm_entryrdn.c
index a3ea091..57b3a20 100644
--- a/ldap/servers/slapd/back-ldbm/ldbm_entryrdn.c
+++ b/ldap/servers/slapd/back-ldbm/ldbm_entryrdn.c
@@ -810,14 +810,8 @@ entryrdn_rename_subtree(backend *be,
renamedata.data = (void *)oldsupelem;
}
} else {
- if (mynewsupsdn) {
- renamedata.ulen = renamedata.size = newsupelemlen;
- renamedata.data = (void *)newsupelem;
- } else {
- /* never comes here */
- rc = -1;
- goto bail;
- }
+ renamedata.ulen = renamedata.size = newsupelemlen;
+ renamedata.data = (void *)newsupelem;
}
rc = _entryrdn_put_data(cursor, &key, &renamedata, RDN_INDEX_PARENT,
db_txn);
if (rc) {
@@ -902,16 +896,10 @@ entryrdn_rename_subtree(backend *be,
renamedata.data = (void *)targetelem;
}
} else {
- if (mynewsrdn) {
- memset(&renamedata, 0, sizeof(renamedata));
- renamedata.ulen = renamedata.size = newelemlen;
- renamedata.data = (void *)newelem;
- renamedata.flags = DB_DBT_USERMEM;
- } else {
- /* never comes here */
- rc = -1;
- goto bail;
- }
+ memset(&renamedata, 0, sizeof(renamedata));
+ renamedata.ulen = renamedata.size = newelemlen;
+ renamedata.data = (void *)newelem;
+ renamedata.flags = DB_DBT_USERMEM;
}
rc = _entryrdn_put_data(cursor, &key, &renamedata, RDN_INDEX_CHILD,
db_txn);
if (rc) {
diff --git a/ldap/servers/slapd/back-ldbm/ldbm_search.c
b/ldap/servers/slapd/back-ldbm/ldbm_search.c
index 00cce2c..7e5a20a 100644
--- a/ldap/servers/slapd/back-ldbm/ldbm_search.c
+++ b/ldap/servers/slapd/back-ldbm/ldbm_search.c
@@ -620,7 +620,7 @@ ldbm_back_search( Slapi_PBlock *pb )
/* This candidate list is for vlv, no need for sort only. */
switch (vlv_search_build_candidate_list(pb, basesdn, &vlv_rc,
sort_control,
- (vlv ? &vlv_request_control :
NULL),
+ &vlv_request_control,
&candidates,
&vlv_response_control)) {
case VLV_ACCESS_DENIED:
return ldbm_back_search_cleanup(pb, li, sort_control,
diff --git a/ldap/servers/slapd/back-ldbm/seq.c b/ldap/servers/slapd/back-ldbm/seq.c
index 1435ba2..ab473bd 100644
--- a/ldap/servers/slapd/back-ldbm/seq.c
+++ b/ldap/servers/slapd/back-ldbm/seq.c
@@ -221,8 +221,6 @@ ldbm_back_seq( Slapi_PBlock *pb )
}
}
break;
- default:
- PR_ASSERT(0);
}
dbc->c_close(dbc);
diff --git a/ldap/servers/slapd/bind.c b/ldap/servers/slapd/bind.c
index e92add5..ec874cc 100644
--- a/ldap/servers/slapd/bind.c
+++ b/ldap/servers/slapd/bind.c
@@ -178,7 +178,7 @@ do_bind( Slapi_PBlock *pb )
/* check that the dn is formatted correctly */
rc = slapi_dn_syntax_check(pb, rawdn, 1);
if (rc) { /* syntax check failed */
- op_shared_log_error_access(pb, "BIND", rawdn?rawdn:"",
+ op_shared_log_error_access(pb, "BIND", rawdn,
"strict: invalid bind dn");
send_ldap_result(pb, LDAP_INVALID_DN_SYNTAX,
NULL, "invalid bind dn", 0, NULL);
diff --git a/ldap/servers/slapd/log.c b/ldap/servers/slapd/log.c
index f483a57..0cf08f0 100644
--- a/ldap/servers/slapd/log.c
+++ b/ldap/servers/slapd/log.c
@@ -1002,8 +1002,6 @@ log_set_rotationsynchour(const char *attrname, char *rhour_str, int
logtype, cha
fe_cfg->auditlog_rotationsynchour = rhour;
LOG_AUDIT_UNLOCK_WRITE();
break;
- default:
- rv = 1;
}
return rv;
@@ -1056,8 +1054,6 @@ log_set_rotationsyncmin(const char *attrname, char *rmin_str, int
logtype, char
loginfo.log_audit_rotationsyncclock = log_get_rotationsyncclock(
loginfo.log_audit_rotationsynchour, rmin );
LOG_AUDIT_UNLOCK_WRITE();
break;
- default:
- rv = 1;
}
return rv;
@@ -1113,8 +1109,6 @@ log_set_rotationtime(const char *attrname, char *rtime_str, int
logtype, char *r
loginfo.log_audit_rotationtime = rtime;
runit = loginfo.log_audit_rotationunit;
break;
- default:
- rv = 1;
}
/* find out the rotation unit we have se right now */
@@ -1142,19 +1136,17 @@ log_set_rotationtime(const char *attrname, char *rtime_str, int
logtype, char *r
fe_cfg->accesslog_rotationtime = rtime;
loginfo.log_access_rotationtime_secs = value;
LOG_ACCESS_UNLOCK_WRITE();
- break;
+ break;
case SLAPD_ERROR_LOG:
fe_cfg->errorlog_rotationtime = rtime;
- loginfo.log_error_rotationtime_secs = value;
- LOG_ERROR_UNLOCK_WRITE();
- break;
+ loginfo.log_error_rotationtime_secs = value;
+ LOG_ERROR_UNLOCK_WRITE();
+ break;
case SLAPD_AUDIT_LOG:
fe_cfg->auditlog_rotationtime = rtime;
- loginfo.log_audit_rotationtime_secs = value;
- LOG_AUDIT_UNLOCK_WRITE();
- break;
- default:
- rv = 1;
+ loginfo.log_audit_rotationtime_secs = value;
+ LOG_AUDIT_UNLOCK_WRITE();
+ break;
}
return rv;
}
@@ -1210,8 +1202,6 @@ int log_set_rotationtimeunit(const char *attrname, char *runit, int
logtype, cha
LOG_AUDIT_LOCK_WRITE( );
origvalue = loginfo.log_audit_rotationtime;
break;
- default:
- rv = 1;
}
if (strcasecmp(runit, "month") == 0) {
@@ -1261,8 +1251,6 @@ int log_set_rotationtimeunit(const char *attrname, char *runit, int
logtype, cha
fe_cfg->auditlog_rotationunit = slapi_ch_strdup ( runit );
LOG_AUDIT_UNLOCK_WRITE();
break;
- default:
- rv = 1;
}
return rv;
}
@@ -1312,9 +1300,6 @@ log_set_maxdiskspace(const char *attrname, char *maxdiskspace_str,
int logtype,
LOG_AUDIT_LOCK_WRITE( );
mlogsize = loginfo.log_audit_maxlogsize;
break;
- default:
- rv = 1;
- mlogsize = -1;
}
maxdiskspace = (PRInt64)s_maxdiskspace * LOG_MB_IN_BYTES;
if (maxdiskspace < 0) {
@@ -1348,11 +1333,6 @@ log_set_maxdiskspace(const char *attrname, char *maxdiskspace_str,
int logtype,
}
LOG_AUDIT_UNLOCK_WRITE();
break;
- default:
- PR_snprintf( errorbuf, SLAPI_DSE_RETURNTEXT_SIZE,
- "%s: invalid log type (%d) for setting maximum disk space: %d MB\n",
- attrname, logtype, s_maxdiskspace);
- rv = LDAP_OPERATIONS_ERROR;
}
return rv;
@@ -1572,10 +1552,6 @@ log_set_expirationtimeunit(const char *attrname, char *expunit, int
logtype, cha
rsecs = loginfo.log_audit_rotationtime_secs;
exptimeunitp = &(loginfo.log_audit_exptimeunit);
break;
- default:
- rv = 1;
- exptime = -1;
- rsecs = -1;
}
value = -1;
@@ -1625,8 +1601,6 @@ log_set_expirationtimeunit(const char *attrname, char *expunit, int
logtype, cha
fe_cfg->auditlog_exptimeunit = slapi_ch_strdup ( expunit );
LOG_AUDIT_UNLOCK_WRITE();
break;
- default:
- rv = 1;
}
return rv;
@@ -1748,8 +1722,8 @@ slapd_log_audit_proc (
char *buffer,
int buf_len)
{
- int err;
if ( (loginfo.log_audit_state & LOGGING_ENABLED) && (loginfo.log_audit_file
!= NULL) ){
+ int err;
LOG_AUDIT_LOCK_WRITE( );
if (log__needrotation(loginfo.log_audit_fdes,
SLAPD_AUDIT_LOG) == LOG_ROTATE) {
@@ -4168,9 +4142,9 @@ static void log_append_buffer2(time_t tnl, LogBufferInfo *lbi, char
*msg1, size_
static void log_flush_buffer(LogBufferInfo *lbi, int type, int sync_now)
{
slapdFrontendConfig_t *slapdFrontendConfig = getFrontendConfig();
- int err = 0;
-
+
if (type == SLAPD_ACCESS_LOG) {
+ int err = 0;
/* It is only safe to flush once any other threads which are copying are finished */
while (lbi->refcount > 0) {