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:30:03 -0500 |
| commit | 62bbadfe0aaa9348e3a05b5ce960e8c4e5a8d44b (patch) | |
| tree | 5f542d047838e09af1188333e17a1dd042819f28 /server/responder | |
| parent | e696bf78c266fcb9653614d6e314c63b68a10963 (diff) | |
| download | sssd-62bbadfe0aaa9348e3a05b5ce960e8c4e5a8d44b.tar.gz sssd-62bbadfe0aaa9348e3a05b5ce960e8c4e5a8d44b.tar.xz sssd-62bbadfe0aaa9348e3a05b5ce960e8c4e5a8d44b.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.
Diffstat (limited to 'server/responder')
| -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 */ |