I pushed the attached patch to master under the one-line rule.

Simo.

-- 
Simo Sorce * Red Hat, Inc * New York
>From 5f68877558b009ac810bdf3f7ff494993eb7cb88 Mon Sep 17 00:00:00 2001
From: Simo Sorce <sso...@redhat.com>
Date: Fri, 6 Jul 2012 16:11:32 -0400
Subject: [PATCH 1/4] Fix wrong check after allocation.

---
 daemons/ipa-slapi-plugins/ipa-pwd-extop/ipapwd_encoding.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/daemons/ipa-slapi-plugins/ipa-pwd-extop/ipapwd_encoding.c b/daemons/ipa-slapi-plugins/ipa-pwd-extop/ipapwd_encoding.c
index 5720d3e6e0a7e6b1520e51a5ee319bcc47354fc9..9c1623a3b28932fba48c878dc6084862a2ba7831 100644
--- a/daemons/ipa-slapi-plugins/ipa-pwd-extop/ipapwd_encoding.c
+++ b/daemons/ipa-slapi-plugins/ipa-pwd-extop/ipapwd_encoding.c
@@ -446,7 +446,7 @@ int ipapwd_gen_hashes(struct ipapwd_krbcfg *krbcfg,
 
         if (is_ipant) {
             *ntvals = (Slapi_Value **)calloc(2, sizeof(Slapi_Value *));
-            if (!svals) {
+            if (!*ntvals) {
                 LOG_OOM();
                 rc = LDAP_OPERATIONS_ERROR;
                 goto done;
-- 
1.7.10.4

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to