patch is attached
--
Ondrej Kos
Associate Software Engineer
Identity Management
Red Hat Czech

phone: +420-532-294-558
cell:  +420-736-417-909
ext:   82-62558
loc:   1013 Brno 1 office
irc:   okos @ #brno
From f1c933e42972d8cd235acfd8d778245504e02006 Mon Sep 17 00:00:00 2001
From: Ondrej Kos <[email protected]>
Date: Tue, 22 Jan 2013 13:55:09 +0100
Subject: [PATCH] TESTS: Fix coverity issues 13126, 13127

https://fedorahosted.org/sssd/ticket/1763
---
 src/tests/sysdb-tests.c | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/src/tests/sysdb-tests.c b/src/tests/sysdb-tests.c
index 0b042cbd807fc43b6a9f874174c182553ab668b0..99d6213d8fc824fc17497efd5d63a91f3abc3e54 100644
--- a/src/tests/sysdb-tests.c
+++ b/src/tests/sysdb-tests.c
@@ -4363,8 +4363,12 @@ START_TEST(test_sysdb_svc_remove_alias)
 
     ret = sysdb_svc_remove_alias(test_ctx->sysdb, dn, aliases[1]);
     fail_if (ret != EOK, "[%s]", strerror(ret));
-    sysdb_transaction_commit(test_ctx->sysdb);
-    sysdb_transaction_start(test_ctx->sysdb);
+
+    ret = sysdb_transaction_commit(test_ctx->sysdb);
+    fail_if(ret != EOK);
+
+    ret = sysdb_transaction_start(test_ctx->sysdb);
+    fail_if(ret != EOK);
 
     /* Set aliases[1] to NULL to perform validation checks */
     aliases[1] = NULL;
-- 
1.7.11.7

_______________________________________________
sssd-devel mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel

Reply via email to