From: Christian Ambach <ambi@samba.org>
Date: Tue, 8 May 2012 15:16:49 +0000 (+0200)
Subject: s3:winbindd:autorid check that transaction start did work
X-Git-Url: http://git.samba.org/?a=commitdiff_plain;h=088436dff3fb12ec0b82f15fa971a48d798bd9b6;p=metze%2Fsamba%2Fwip.git

s3:winbindd:autorid check that transaction start did work

this fixes Coverity #700172 CHECKED_RETURN

Autobuild-User: Christian Ambach <ambi@samba.org>
Autobuild-Date: Wed May  9 00:27:08 CEST 2012 on sn-devel-104
---

diff --git a/source3/winbindd/idmap_autorid.c b/source3/winbindd/idmap_autorid.c
index 08dcc6595f5b..df63fa992072 100644
--- a/source3/winbindd/idmap_autorid.c
+++ b/source3/winbindd/idmap_autorid.c
@@ -435,7 +435,11 @@ static NTSTATUS idmap_autorid_map_sid_to_id(struct idmap_domain *dom,
 		   sid_string_dbg(map->sid)));
 
 	/* create new mapping */
-	dbwrap_transaction_start(ctx->db);
+	res = dbwrap_transaction_start(ctx->db);
+	if (res != 0) {
+		DEBUG(2, ("transaction_start failed\n"));
+		return NT_STATUS_INTERNAL_DB_CORRUPTION;
+	}
 
 	ret = idmap_tdb_common_new_mapping(dom, map);