On 21-12-23 13:14:26, Tobias Heider wrote: > On Mon, Dec 13, 2021 at 10:56:15AM -0700, Joel Sing wrote: > > CVSROOT: /cvs > > Module name: src > > Changes by: [email protected] 2021/12/13 10:56:15 > > > > Modified files: > > lib/libcrypto : Makefile > > lib/libcrypto/asn1: tasn_typ.c > > Removed files: > > lib/libcrypto/asn1: a_bool.c > > > > Log message: > > Clean up d2i_ASN1_BOOLEAN() and i2d_ASN1_BOOLEAN(). > > > > Convert these to templated ASN.1, given we already have ASN1_BOOLEAN_it. > > > > ok inoguchi@ tb@ > > > > It looks like the new d2i_ASN1_BOOLEAN() is broken on several archs, see: > > http://bluhm.genua.de/regress/results/2021-12-22T10%3A17%3A02Z/logs/lib/libcrypto/asn1/make.log > http://bluhm.genua.de/regress/results/2021-12-21T02%3A17%3A18Z/logs/lib/libcrypto/asn1/make.log
What architectures specifically? > The diff below reverts the previous commit which fixes the test. > > ok to backout? Nothing really uses d2i_ASN1_BOOLEAN() directly - if there is a bug in the templated ASN1_BOOLEAN code (or the regress test) it needs to be found and fixed.
