Bugfix: when X509_NAME_dup(3) failed, X509_NAME_set(3) indicated success
authorschwarze <schwarze@openbsd.org>
Sun, 4 Jul 2021 11:38:37 +0000 (11:38 +0000)
committerschwarze <schwarze@openbsd.org>
Sun, 4 Jul 2021 11:38:37 +0000 (11:38 +0000)
commitb79b571036757ffc2ef7728fb9ec5f9a4e56590c
tree4c3e8fdf021225e36d120a33726f2e240844552c
parent2278327984daf55f73c8a33cd90ffbcd9d7d3132
Bugfix: when X509_NAME_dup(3) failed, X509_NAME_set(3) indicated success
even though it did not actually set the name.
Instead, indicate failure in this case.

This commit sneaks in a small, unrelated change in behaviour.
If the first argument of X509_NAME_set(3) was NULL, the function
used to return failure.  Now it crashes the program by accessing
the NULL pointer, for compatibility with the same change in OpenSSL.

This merges the following two commits from the OpenSSL-1.1.1 branch,
which is still available under a free license:
1. 180794c5  Rich Salz        Sep  3 11:33:34 2017 -0400
2. c1c1783d  Richard Levitte  May 17 09:53:14 2018 +0200

OK tb@
lib/libcrypto/asn1/x_name.c