rpki-client: rework AIA, SIA, and CRL handling
authortb <tb@openbsd.org>
Tue, 4 Jun 2024 14:17:24 +0000 (14:17 +0000)
committertb <tb@openbsd.org>
Tue, 4 Jun 2024 14:17:24 +0000 (14:17 +0000)
commite7ecba749cd2afac76c9d53c6ad698da604a10d5
treecb35c0db2261dd46c588573e19736c73e3d8fcac
parent78268cf82339967be4573ca1c53c04816d2ee01c
rpki-client: rework AIA, SIA, and CRL handling

Assert that the out parameter is NULL rather than blindly NULL-ing it,
and follow the approach in sbgp_sia() more closely: use a local variable
and warn on any ignored accessMethod/distributionPoint in verbose mode.
AIA is slightly different because there's only a single accessMethod.

Drop the now useless (and previously wrong) warning on *out != NULL in
x509_location(). Instead, assert that the out parameter is NULL. This
way things are a bit less clever, but more explicit, more correct, more
robust, and the code grew only very slightly.

ok claudio
usr.sbin/rpki-client/x509.c