git.samba.org
/
metze
/
samba
/
wip.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(parent:
09494ed
)
s3:winbindd:autorid check that transaction start did work
author
Christian Ambach
<ambi@samba.org>
Tue, 8 May 2012 15:16:49 +0000
(17:16 +0200)
committer
Christian 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
patch
|
blob
|
history
diff --git
a/source3/winbindd/idmap_autorid.c
b/source3/winbindd/idmap_autorid.c
index 08dcc6595f5b6b26127121e7409b7e0905b5fa3f..df63fa99207257bf8a9015c3212c581e123cbc42 100644
(file)
--- 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);