diff options
| author | tb <> | 2020-09-20 03:19:52 +0000 |
|---|---|---|
| committer | tb <> | 2020-09-20 03:19:52 +0000 |
| commit | 71ab2c82f3894c93b4fcf5265967efcf76bf9883 (patch) | |
| tree | a35889fc2181569ba43b065ccd91135008a8d676 /src/lib/libcrypto/x509/x509_ext.c | |
| parent | 68a66b91bb79ab939073198e9c822a826bd09b96 (diff) | |
| download | openbsd-71ab2c82f3894c93b4fcf5265967efcf76bf9883.tar.gz openbsd-71ab2c82f3894c93b4fcf5265967efcf76bf9883.tar.bz2 openbsd-71ab2c82f3894c93b4fcf5265967efcf76bf9883.zip | |
Fix a memory leak in x509_constraints_extract_names
If the default path of the switch is taken, vname will not be added
to the names list and will leak when it is set to NULL. Simplify the
logic by eliminating the add Boolean. Instead, free and zero vname in
the default case and continue the while loop directly. At the bottom
of the switch, add vname to the names list unconditionally zero it out
since it's now owned by names.
Found by Guido Vranken's cryptofuzzer
ok beck
Diffstat (limited to 'src/lib/libcrypto/x509/x509_ext.c')
0 files changed, 0 insertions, 0 deletions