From 31273617f97973332d8b265dcb01679025d16cbd Mon Sep 17 00:00:00 2001 From: Jakub Hrozek Date: Wed, 3 Aug 2016 18:03:59 +0200 Subject: [PATCH 2/2] IPA: Check the return value of sss_parse_internal_fqname We should fail the request if sss_parse_internal_fqname() fails. --- src/providers/ipa/ipa_subdomains_id.c | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/src/providers/ipa/ipa_subdomains_id.c b/src/providers/ipa/ipa_subdomains_id.c index 76fdaa8a1213069bd6b45e0b69b6cdb0d034d721..5369ec4c624544f7f3aec88ddaa30eac91c51735 100644 --- a/src/providers/ipa/ipa_subdomains_id.c +++ b/src/providers/ipa/ipa_subdomains_id.c @@ -509,6 +509,14 @@ static void ipa_get_subdom_acct_connected(struct tevent_req *subreq) } else { ret = sss_parse_internal_fqname(req_input, state->filter, &shortname, NULL); + if (ret != EOK) { + DEBUG(SSSDBG_CRIT_FAILURE, + "Cannot parse internal name [%s]: %d\n", + state->filter, ret); + tevent_req_error(req, ret); + return; + } + req_input->inp.name = talloc_steal(req_input, shortname); } if (req_input->inp.name == NULL) { -- 2.4.11