diff options
| author | Christian Ambach <ambi@samba.org> | 2012-05-08 17:16:49 +0200 |
|---|---|---|
| committer | Christian Ambach <ambi@samba.org> | 2012-05-09 00:27:08 +0200 |
| commit | 088436dff3fb12ec0b82f15fa971a48d798bd9b6 (patch) | |
| tree | a5ada5986e3eb0017958ffe11081fb29dee1a74f /source3/winbindd/idmap_autorid.c | |
| parent | 09494ed6133fd4d71161969249adf187732e2709 (diff) | |
| download | samba-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
Diffstat (limited to 'source3/winbindd/idmap_autorid.c')
| -rw-r--r-- | source3/winbindd/idmap_autorid.c | 6 |
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); |