diff options
| author | Volker Lendecke <vl@samba.org> | 2014-10-10 15:37:55 +0200 |
|---|---|---|
| committer | Jeremy Allison <jra@samba.org> | 2014-10-11 22:02:04 +0200 |
| commit | 8686da231d434bb35333654277a029271e147257 (patch) | |
| tree | f524ed3b22476caed6f4514ac83fe879e24fe414 /source4/dsdb/samdb | |
| parent | 6d2c8f54e5e87485783b5173aa9f903a79d25443 (diff) | |
| download | samba-8686da231d434bb35333654277a029271e147257.tar.gz samba-8686da231d434bb35333654277a029271e147257.tar.xz samba-8686da231d434bb35333654277a029271e147257.zip | |
dsdb: Fix a crash in an error return
In an error return we have
/* Back it out, if it fails on one */
for (i--; i >= 0; i--) {
ldb_next_del_trans(data->partitions[i]->module);
}
With unsigned int i this will spin and del_trans somewhere far off :-)
Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Diffstat (limited to 'source4/dsdb/samdb')
| -rw-r--r-- | source4/dsdb/samdb/ldb_modules/partition.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/source4/dsdb/samdb/ldb_modules/partition.c b/source4/dsdb/samdb/ldb_modules/partition.c index f2624a2612..b501ff1c0c 100644 --- a/source4/dsdb/samdb/ldb_modules/partition.c +++ b/source4/dsdb/samdb/ldb_modules/partition.c @@ -805,7 +805,7 @@ static int partition_rename(struct ldb_module *module, struct ldb_request *req) /* start a transaction */ static int partition_start_trans(struct ldb_module *module) { - unsigned int i; + int i; int ret; struct partition_private_data *data = talloc_get_type(ldb_module_get_private(module), struct partition_private_data); |