summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorChristian Ambach <ambi@samba.org>2012-05-08 17:16:49 +0200
committerChristian Ambach <ambi@samba.org>2012-05-09 00:27:08 +0200
commit088436dff3fb12ec0b82f15fa971a48d798bd9b6 (patch)
treea5ada5986e3eb0017958ffe11081fb29dee1a74f
parent09494ed6133fd4d71161969249adf187732e2709 (diff)
downloadsamba-088436dff3fb12ec0b82f15fa971a48d798bd9b6.tar.gz
samba-088436dff3fb12ec0b82f15fa971a48d798bd9b6.tar.bz2
samba-088436dff3fb12ec0b82f15fa971a48d798bd9b6.zip
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
-rw-r--r--source3/winbindd/idmap_autorid.c6
1 files changed, 5 insertions, 1 deletions
diff --git a/source3/winbindd/idmap_autorid.c b/source3/winbindd/idmap_autorid.c
index 08dcc6595f..df63fa9920 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);