summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSumit Bose <sbose@redhat.com>2010-10-25 11:14:04 +0200
committerStephen Gallagher <sgallagh@redhat.com>2010-10-26 07:59:50 -0400
commit09aa3d91f8fa667b09494612d5696da2b8461cb2 (patch)
treed1326781638da2cd5d7a62a7126283ea8aeb0d69
parentc475a1ce4d407f722551f3d35d2f9e50fe139d95 (diff)
downloadsssd_unused-09aa3d91f8fa667b09494612d5696da2b8461cb2.tar.gz
sssd_unused-09aa3d91f8fa667b09494612d5696da2b8461cb2.tar.xz
sssd_unused-09aa3d91f8fa667b09494612d5696da2b8461cb2.zip
Fix double free issue
-rw-r--r--src/responder/common/responder_dp.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/src/responder/common/responder_dp.c b/src/responder/common/responder_dp.c
index f8c3b6f1..b2b5d40e 100644
--- a/src/responder/common/responder_dp.c
+++ b/src/responder/common/responder_dp.c
@@ -95,12 +95,12 @@ static int sss_dp_req_destructor(void *ptr)
cb = sdp_req->cb_list;
while (cb) {
+ next = cb->next;
+ /* It is the responsibility of the callback to free cb */
cb->callback(sdp_req->err_maj,
sdp_req->err_min,
sdp_req->err_msg,
cb->callback_ctx);
- next = cb->next;
- talloc_free(cb);
cb = next;
}