summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorStephen Gallagher <sgallagh@redhat.com>2009-12-08 11:16:11 -0500
committerStephen Gallagher <sgallagh@redhat.com>2009-12-08 12:31:46 -0500
commitfa9f50878bc93b4ebe5ebf012b207494ff29e6fc (patch)
tree6f11713e4a3ce92231803e2e83a7f68ea74ee8a6
parent4b6cc739779965f3dde02745b60989324ada2b62 (diff)
downloadsssd-fa9f50878bc93b4ebe5ebf012b207494ff29e6fc.tar.gz
sssd-fa9f50878bc93b4ebe5ebf012b207494ff29e6fc.tar.xz
sssd-fa9f50878bc93b4ebe5ebf012b207494ff29e6fc.zip
Fix potential uninitialized value error in responder_dp.c
If we fell into the default case of the switch statement, we would attempt to talloc_free() a random memory location. This patch guarantees that sdp_req is NULL if it has not been initialized.
-rw-r--r--server/responder/common/responder_dp.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/server/responder/common/responder_dp.c b/server/responder/common/responder_dp.c
index 03e83ec61..1fe1d95ec 100644
--- a/server/responder/common/responder_dp.c
+++ b/server/responder/common/responder_dp.c
@@ -254,7 +254,7 @@ int sss_dp_send_acct_req(struct resp_ctx *rctx, TALLOC_CTX *callback_memctx,
hash_value_t value;
TALLOC_CTX *tmp_ctx;
struct timeval tv;
- struct sss_dp_req *sdp_req;
+ struct sss_dp_req *sdp_req = NULL;
struct sss_dp_callback *cb;
/* either, or, not both */