s3:winbindd:autorid check that transaction start did work
authorChristian Ambach <ambi@samba.org>
Tue, 8 May 2012 15:16:49 +0000 (17:16 +0200)
committerChristian Ambach <ambi@samba.org>
Tue, 8 May 2012 22:27:08 +0000 (00:27 +0200)
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

source3/winbindd/idmap_autorid.c

index 08dcc6595f5b6b26127121e7409b7e0905b5fa3f..df63fa99207257bf8a9015c3212c581e123cbc42 100644 (file)
@@ -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);