diff options
author | Sylvain Munaut <tnt@246tNt.com> | 2012-08-02 12:12:59 -0400 |
---|---|---|
committer | Sage Weil <sage@inktank.com> | 2012-08-02 12:19:20 -0400 |
commit | f0666b1ac875ff32fe290219b150ec62eebbe10e (patch) | |
tree | 9f38d4c5ac83b77812692ccd2db6a880bf63cf9d /net/ceph/crypto.h | |
parent | 5ef50c3bec20060bc114f62d6503c5d86d70bdd7 (diff) |
libceph: fix crypto key null deref, memory leak
Avoid crashing if the crypto key payload was NULL, as when it was not correctly
allocated and initialized. Also, avoid leaking it.
Signed-off-by: Sylvain Munaut <tnt@246tNt.com>
Signed-off-by: Sage Weil <sage@inktank.com>
Reviewed-by: Alex Elder <elder@inktank.com>
Diffstat (limited to 'net/ceph/crypto.h')
-rw-r--r-- | net/ceph/crypto.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/net/ceph/crypto.h b/net/ceph/crypto.h index 1919d1550d75..3572dc518bc9 100644 --- a/net/ceph/crypto.h +++ b/net/ceph/crypto.h | |||
@@ -16,7 +16,8 @@ struct ceph_crypto_key { | |||
16 | 16 | ||
17 | static inline void ceph_crypto_key_destroy(struct ceph_crypto_key *key) | 17 | static inline void ceph_crypto_key_destroy(struct ceph_crypto_key *key) |
18 | { | 18 | { |
19 | kfree(key->key); | 19 | if (key) |
20 | kfree(key->key); | ||
20 | } | 21 | } |
21 | 22 | ||
22 | extern int ceph_crypto_key_clone(struct ceph_crypto_key *dst, | 23 | extern int ceph_crypto_key_clone(struct ceph_crypto_key *dst, |