From dd09bbb2578bd9f82afd7fc4d4fcc3a5659731d3 Mon Sep 17 00:00:00 2001
From: Lukas Slebodnik <lslebodn@redhat.com>
Date: Oct 13 2014 14:12:08 +0000
Subject: GPO: Fail if we cannot retrieve gpo from cache.


The variable value could be used uninitialized.

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>

---

diff --git a/src/providers/ad/ad_gpo.c b/src/providers/ad/ad_gpo.c
index d581db8..2d8ba82 100644
--- a/src/providers/ad/ad_gpo.c
+++ b/src/providers/ad/ad_gpo.c
@@ -1358,6 +1358,12 @@ parse_policy_setting_value(TALLOC_CTX *mem_ctx,
     char **sids_list = NULL;
 
     ret = sysdb_gpo_get_gpo_result_setting(mem_ctx, domain, key, &value);
+    if (ret != EOK) {
+        DEBUG(SSSDBG_OP_FAILURE,
+              "Cannot retrieve settings from sysdb for key: '%s' [%d][%s].\n",
+              key, ret, sss_strerror(ret));
+        goto done;
+    }
 
     if (value == NULL) {
         DEBUG(SSSDBG_TRACE_FUNC,