diff options
author | Hanno Böck <hanno@hboeck.de> | 2015-05-11 11:33:37 +0100 |
---|---|---|
committer | Matt Caswell <matt@openssl.org> | 2015-05-13 15:32:23 +0100 |
commit | 5e0ec9012bae4cc261ec300a0bf1432fbb13ee2a (patch) | |
tree | b64e0fba30d5b33f51a59ef5f71496cf15319453 /crypto/cmac/cmac.c | |
parent | 5c122908ab42399df020981d0238e59d4ec38098 (diff) |
Call of memcmp with null pointers in obj_cmp()
The function obj_cmp() (file crypto/objects/obj_dat.c) can in some
situations call memcmp() with a null pointer and a zero length.
This is invalid behaviour. When compiling openssl with undefined
behaviour sanitizer (add -fsanitize=undefined to compile flags) this
can be seen. One example that triggers this behaviour is the pkcs7
command (but there are others, e.g. I've seen it with the timestamp
function):
apps/openssl pkcs7 -in test/testp7.pem
What happens is that obj_cmp takes objects of the type ASN1_OBJECT and
passes their ->data pointer to memcmp. Zero-sized ASN1_OBJECT
structures can have a null pointer as data.
RT#3816
Signed-off-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Rich Salz <rsalz@openssl.org>
(cherry picked from commit 2b8dc08b74fc3c6d4c2fc855cc23bac691d985be)
Diffstat (limited to 'crypto/cmac/cmac.c')
0 files changed, 0 insertions, 0 deletions