diff options
author | Lukas Slebodnik <lslebodn@redhat.com> | 2014-03-12 17:38:22 +0100 |
---|---|---|
committer | Jakub Hrozek <jhrozek@redhat.com> | 2014-03-13 18:43:41 +0100 |
commit | 2ac5fed1ea4e9f56a18d0cc3b445855cdc6757c2 (patch) | |
tree | 4dad19bdd21ae88349d22d8997bb93ef1a8bca82 | |
parent | f8a49b3bff8d3969824fc7ba4e90d229f0c4edea (diff) | |
download | sssd-2ac5fed1ea4e9f56a18d0cc3b445855cdc6757c2.tar.gz sssd-2ac5fed1ea4e9f56a18d0cc3b445855cdc6757c2.tar.xz sssd-2ac5fed1ea4e9f56a18d0cc3b445855cdc6757c2.zip |
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 <simo@redhat.com>
(cherry picked from commit bad65473c4c28ecbf2b6bd374a7ae2d634d57d8d)
-rw-r--r-- | src/providers/ad/ad_id.c | 1 |
1 files changed, 1 insertions, 0 deletions
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; |