Turn the validation_err() macro into a function
authortb <tb@openbsd.org>
Wed, 5 Jan 2022 17:36:32 +0000 (17:36 +0000)
committertb <tb@openbsd.org>
Wed, 5 Jan 2022 17:36:32 +0000 (17:36 +0000)
commit65d9c3c1cf116f0376aa951c159ef3eaaaae465b
tree7d7ea65c865c0b2a4bf1f15fb11cdba5847a4e49
parent97ab7993c79b8c163c5cdbf094cf64c3f54ead5e
Turn the validation_err() macro into a function

validation_err() is an ugly macro with side effects and a goto in it.
At the cost of a few lines of code we can turn this into a function
where the side effects are explicit and ret is now explicitly set in
the main body of addr_validate_path_internal().

We get to a point where it is halfway possible to reason about the
convoluted control flow in this function.

ok inoguchi jsing
lib/libcrypto/x509/x509_addr.c