diff options
| author | Stephen Gallagher <sgallagh@redhat.com> | 2009-12-08 11:16:11 -0500 |
|---|---|---|
| committer | Stephen Gallagher <sgallagh@redhat.com> | 2009-12-08 12:31:46 -0500 |
| commit | fa9f50878bc93b4ebe5ebf012b207494ff29e6fc (patch) | |
| tree | 6f11713e4a3ce92231803e2e83a7f68ea74ee8a6 | |
| parent | 4b6cc739779965f3dde02745b60989324ada2b62 (diff) | |
| download | sssd-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.c | 2 |
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 */ |