Fix a logic error in ASN1_item_sign_ctx()
authortb <tb@openbsd.org>
Thu, 15 Jun 2023 12:44:17 +0000 (12:44 +0000)
committertb <tb@openbsd.org>
Thu, 15 Jun 2023 12:44:17 +0000 (12:44 +0000)
commita6297c0aa7403f7aeb9c181ee9296ebd43873aec
treeb9a6d7aabd82da2b07cc42b48db4ac8c36299a1f
parentc83bf4c80970526c99a3b61304819bc635b08f70
Fix a logic error in ASN1_item_sign_ctx()

If the item_sign() ASN.1 method returns 1, it supposedly handles everything
and the goto err prior to r1.5 was actually a success path. Go figure.

This is fortunately inconsequential since there are only two item_sign()
methods, one for RSA and one for Ed25519, neither of which can return 1.
They only return 0, 2, and 3.

Pointed out by and ok jsing
lib/libcrypto/asn1/asn1_item.c