diff options
author | Matt Caswell <matt@openssl.org> | 2019-10-30 13:20:33 +0000 |
---|---|---|
committer | Matt Caswell <matt@openssl.org> | 2019-11-04 12:54:36 +0000 |
commit | 7bb50cbc4af78a0c8d36fdf2c141ad1330125e2f (patch) | |
tree | 7e41523f8a6f6a3f9919b9dabf9be2a204063b7a /crypto/bio | |
parent | 72f4d2f8eb0a95bbac7e247ff571bbef4158018c (diff) |
Don't leak memory in the event of a failure in i2v_GENERAL_NAMES
i2v_GENERAL_NAMES call i2v_GENERAL_NAME repeatedly as required. Each
time i2v_GENERAL_NAME gets called it allocates adds data to the passed in
stack and then returns a pointer to the stack, or NULL on failure. If
the passed in stack is itself NULL then it allocates one.
i2v_GENERAL_NAMES was not correctly handling the case where a NULL gets
returned from i2v_GENERAL_NAME. If a stack had already been allocated then
it just leaked it.
Reviewed-by: Dmitry Belyavskiy <beldmit@gmail.com>
Reviewed-by: Viktor Dukhovni <viktor@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/10300)
(cherry picked from commit 45b244620a74248b46ebe1c85e86437b9641447a)
Diffstat (limited to 'crypto/bio')
0 files changed, 0 insertions, 0 deletions