Clean up compiler warnings

This cleans up all of the compiler warnings produced with -Wall on RHEL/Fedora platforms.
The warnings about the %lld and %llu formats are still produced and cannot be helped.
Reviewed by: nkinder (Thanks!)

5 files changed, 18 insertions, 9 deletions

diff --git a/ldap/servers/plugins/acl/acl.c b/ldap/servers/plugins/acl/acl.c
index b708cada..d62796fc 100644
--- a/ldap/servers/plugins/acl/acl.c
+++ b/ldap/servers/plugins/acl/acl.c
@@ -3148,6 +3148,7 @@ acl_match_substring ( Slapi_Filter *f, char *str, int exact_match)
 	char		*type, *initial, *final;
 	char		**any;
 	Slapi_Regex	*re = NULL;
+	const char  *re_result = NULL;
 
 	if ( 0 != slapi_filter_get_subfilt ( f, &type, &initial, &any, &final ) ) {
 		return (ACL_FALSE);
@@ -3235,11 +3236,10 @@ acl_match_substring ( Slapi_Filter *f, char *str, int exact_match)
 	** Now we will compile the pattern and compare wth the string to
 	** see if the input string matches with the patteren or not.
 	*/
-	p = NULL;
-	re = slapi_re_comp( pat, &p );
+	re = slapi_re_comp( pat, &re_result );
 	if (NULL == re) {
 		slapi_log_error (SLAPI_LOG_ACL, plugin_name, 
-			"acl_match_substring:re_comp failed (%s)\n", p?p:"unknown");
+			"acl_match_substring:re_comp failed (%s)\n", re_result?re_result:"unknown");
 		return (ACL_ERR);
 	}
 
diff --git a/ldap/servers/plugins/acl/acllas.c b/ldap/servers/plugins/acl/acllas.c
index abde4fd7..e16fbd27 100644
--- a/ldap/servers/plugins/acl/acllas.c
+++ b/ldap/servers/plugins/acl/acllas.c
@@ -2218,7 +2218,7 @@ acllas__handle_group_entry (Slapi_Entry* e, void *callback_data)
  	Slapi_Attr		*currAttr, *nextAttr;
 	char			*n_dn, *attrType;
 	int				n;
-	int				i, j;
+	int				i;
 
 	info = (struct eval_info *) callback_data;
 	info->result = ACL_FALSE;
@@ -2252,11 +2252,18 @@ acllas__handle_group_entry (Slapi_Entry* e, void *callback_data)
 					return 0;
 				}
 				if (!(n % ACLLAS_MAX_GRP_MEMBER)) {
-					struct member_info *orig_memberInfo = info->memberInfo[0];
+					struct member_info **orig_memberInfo = info->memberInfo;
 					info->memberInfo = (struct member_info **)slapi_ch_realloc(
 							(char *)info->memberInfo,
 							(n + ACLLAS_MAX_GRP_MEMBER) *
 							sizeof(struct member_info *));
+					if (!info->memberInfo) {
+						slapi_log_error( SLAPI_LOG_FATAL, plugin_name,
+										 "acllas__handle_group_entry: out of memory - could not allocate space for %d group members\n",
+										 n + ACLLAS_MAX_GRP_MEMBER );
+						info->memberInfo = orig_memberInfo;
+						return 0;
+					}
 				}
 
 				/* allocate the space for the member and attch it to the list */
diff --git a/ldap/servers/plugins/replication/repl5_replica.c b/ldap/servers/plugins/replication/repl5_replica.c
index 5f280b1d..a563835a 100644
--- a/ldap/servers/plugins/replication/repl5_replica.c
+++ b/ldap/servers/plugins/replication/repl5_replica.c
@@ -1012,7 +1012,8 @@ replica_set_referrals(Replica *r,const Slapi_ValueSet *vs)
 		{
 			const char *ref = slapi_value_get_string(vv);
 			LDAPURLDesc *lud = NULL;
-			int myrc = slapi_ldap_url_parse(ref, &lud, 0, NULL);
+
+			(void)slapi_ldap_url_parse(ref, &lud, 0, NULL);
 			/* see if the dn is already in the referral URL */
 			if (!lud || !lud->lud_dn) {
 				/* add the dn */
diff --git a/ldap/servers/plugins/replication/replutil.c b/ldap/servers/plugins/replication/replutil.c
index 8703c7c2..aaa427bd 100644
--- a/ldap/servers/plugins/replication/replutil.c
+++ b/ldap/servers/plugins/replication/replutil.c
@@ -740,7 +740,7 @@ repl_set_mtn_state_and_referrals(
 	/* next, add the repl root dn to each referral if not present */
 	for (ii = 0; referrals_to_set && referrals_to_set[ii]; ++ii) {
 		LDAPURLDesc *lud = NULL;
-		int myrc = slapi_ldap_url_parse(referrals_to_set[ii], &lud, 0, NULL);
+		(void)slapi_ldap_url_parse(referrals_to_set[ii], &lud, 0, NULL);
 		/* see if the dn is already in the referral URL */
 		if (!lud || !lud->lud_dn) {
 			/* add the dn */
diff --git a/ldap/servers/plugins/syntaxes/string.c b/ldap/servers/plugins/syntaxes/string.c
index 218cc762..c2f0fa60 100644
--- a/ldap/servers/plugins/syntaxes/string.c
+++ b/ldap/servers/plugins/syntaxes/string.c
@@ -210,6 +210,7 @@ string_filter_sub( Slapi_PBlock *pb, char *initial, char **any, char *final,
 	int		timelimit = 0; /* search timelimit */
 	Operation	*op = NULL;
 	Slapi_Regex	*re = NULL;
+	const char  *re_result = NULL;
 
 	LDAPDebug( LDAP_DEBUG_FILTER, "=> string_filter_sub\n",
 	    0, 0, 0 );
@@ -286,10 +287,10 @@ string_filter_sub( Slapi_PBlock *pb, char *initial, char **any, char *final,
 	/* compile the regex */
 	p = (bigpat) ? bigpat : pat;
 	tmpbuf = NULL;
-	re = slapi_re_comp( p, &tmpbuf );
+	re = slapi_re_comp( p, &re_result );
 	if (NULL == re) {
 		LDAPDebug( LDAP_DEBUG_ANY, "re_comp (%s) failed (%s): %s\n",
-		    pat, p, tmpbuf?tmpbuf:"unknown" );
+		    pat, p, re_result?re_result:"unknown" );
 		rc = LDAP_OPERATIONS_ERROR;
 		goto bailout;
 	} else {