diff options
author | Simo Sorce <idra@samba.org> | 2009-03-09 18:04:38 -0400 |
---|---|---|
committer | Simo Sorce <idra@samba.org> | 2009-03-09 20:12:11 -0400 |
commit | f088353d37b433af7b979a17871233cccddf7aca (patch) | |
tree | 35a022bf85b756bb500fda1b195aeb938da29925 /source4/lib/ldb/ldb_tdb/ldb_index.c | |
parent | 218ce0e80532b0dbc595e72502d9596a35acdffd (diff) | |
download | samba-f088353d37b433af7b979a17871233cccddf7aca.tar.gz samba-f088353d37b433af7b979a17871233cccddf7aca.tar.bz2 samba-f088353d37b433af7b979a17871233cccddf7aca.zip |
Fix potential segfaults using freed memory.
In some code paths ltdb_context was still referenced even after we were returned
an error by one of the callbacks. Because the interface assumes that once an
error is returned the ldb_request may be freed, and because the ltdb_context was
allocated as a child of the request, this might cause access to freed memory.
Allocate the ltdb_context on ldb, and keep track of what's going on with the
request by adding a spy children on it. This way even if the request is freed
before the ltdb_callback is called, we will safely free the ctx and just quietly
return.
Diffstat (limited to 'source4/lib/ldb/ldb_tdb/ldb_index.c')
-rw-r--r-- | source4/lib/ldb/ldb_tdb/ldb_index.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/source4/lib/ldb/ldb_tdb/ldb_index.c b/source4/lib/ldb/ldb_tdb/ldb_index.c index ad27c9a9a9..c99c2936d8 100644 --- a/source4/lib/ldb/ldb_tdb/ldb_index.c +++ b/source4/lib/ldb/ldb_tdb/ldb_index.c @@ -1055,7 +1055,7 @@ static int ltdb_index_filter(const struct dn_list *dn_list, ret = ldb_module_send_entry(ac->req, msg, NULL); if (ret != LDB_SUCCESS) { - ac->callback_failed = true; + ac->request_terminated = true; return ret; } } |