summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAron Parsons <parsonsa@bit-sys.com>2015-04-29 03:19:32 +0000
committerJakub Hrozek <jhrozek@redhat.com>2015-04-29 17:12:33 +0200
commite31d5babfd036cf64c9179dc60bbd79f541ef89b (patch)
treee2dc685f57b516a6d35fab0b572669ac8d6c6008
parent85287a6b897d818d279171a83aa3c8a0de66f13b (diff)
downloadsssd-e31d5babfd036cf64c9179dc60bbd79f541ef89b.tar.gz
sssd-e31d5babfd036cf64c9179dc60bbd79f541ef89b.tar.xz
sssd-e31d5babfd036cf64c9179dc60bbd79f541ef89b.zip
IPA: fix segfault in ipa_s2n_exop
can be triggered on demand by assigning a POSIX group with external members sudo privileges, then dropping the cache and doing a sudo -U <user> -l. Reviewed-by: Sumit Bose <sbose@redhat.com> (cherry picked from commit c520f40d1a2d77cf1d413451b5682297733521ed)
-rw-r--r--src/providers/ipa/ipa_s2n_exop.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/providers/ipa/ipa_s2n_exop.c b/src/providers/ipa/ipa_s2n_exop.c
index 688fdefe8..d07923cff 100644
--- a/src/providers/ipa/ipa_s2n_exop.c
+++ b/src/providers/ipa/ipa_s2n_exop.c
@@ -2165,7 +2165,7 @@ static errno_t ipa_s2n_save_objects(struct sss_domain_info *dom,
goto done;
}
- if (strcmp(view_name, SYSDB_DEFAULT_VIEW_NAME) != 0) {
+ if (view_name != NULL && strcmp(view_name, SYSDB_DEFAULT_VIEW_NAME) != 0) {
/* For the default view the data return by the extdom plugin already
* contains all needed data and it is not expected to have a separate
* override object. */