From 5376ee92896f37a193dbc77451e15180cd7e6640 Mon Sep 17 00:00:00 2001 From: Lukas Slebodnik Date: Wed, 12 Mar 2014 17:38:22 +0100 Subject: AD: Continue if sssd failes to check extra members Reported by scan-build for (mi = 0; group_only[mi]; mi++) { ^~~~~~~~~~ warning: Array access (from variable 'group_only') results in a null pointer dereference It can happend if function ad_group_extra_members fails (ret != EOK) Reviewed-by: Simo Sorce (cherry picked from commit bad65473c4c28ecbf2b6bd374a7ae2d634d57d8d) --- src/providers/ad/ad_id.c | 1 + 1 file changed, 1 insertion(+) (limited to 'src') diff --git a/src/providers/ad/ad_id.c b/src/providers/ad/ad_id.c index a35823b4b..01d18d7ae 100644 --- a/src/providers/ad/ad_id.c +++ b/src/providers/ad/ad_id.c @@ -772,6 +772,7 @@ ad_enum_cross_dom_members(struct sdap_options *opts, ret = ad_group_extra_members(tmp_ctx, msgs[i], dom, &group_only); if (ret != EOK) { DEBUG(SSSDBG_OP_FAILURE, ("Failed to check extra members\n")); + continue; } else if (group_only == NULL) { DEBUG(SSSDBG_TRACE_INTERNAL, ("No extra members\n")); continue; -- cgit