From: tb <tb@openbsd.org>
Date: Thu, 16 Feb 2023 14:34:34 +0000 (+0000)
Subject: Revert r1.63.
X-Git-Url: http://artulab.com/gitweb/?a=commitdiff_plain;h=bcf2dfa57b4c42eca7831b26219d150ece827ada;p=openbsd

Revert r1.63.

GEN_OTHERNAME is the type of a GENERAL_NAMES, not of a DIST_POINT_NAME,
which needs naked numbers as there is no enum nor defines describing it.

claudio agrees
---

diff --git a/usr.sbin/rpki-client/x509.c b/usr.sbin/rpki-client/x509.c
index 688c1c2e7c0..bbfb2695e77 100644
--- a/usr.sbin/rpki-client/x509.c
+++ b/usr.sbin/rpki-client/x509.c
@@ -1,4 +1,4 @@
-/*	$OpenBSD: x509.c,v 1.64 2023/02/16 14:25:27 job Exp $ */
+/*	$OpenBSD: x509.c,v 1.65 2023/02/16 14:34:34 tb Exp $ */
 /*
  * Copyright (c) 2022 Theo Buehler <tb@openbsd.org>
  * Copyright (c) 2021 Claudio Jeker <claudio@openbsd.org>
@@ -610,7 +610,7 @@ x509_get_crl(X509 *x, const char *fn, char **crl)
 		    " disallowed", fn);
 		goto out;
 	}
-	if (dp->distpoint->type != GEN_OTHERNAME) {
+	if (dp->distpoint->type != 0) {
 		warnx("%s: RFC 6487 section 4.8.6: CRL: "
 		    "expected GEN_OTHERNAME, have %d", fn, dp->distpoint->type);
 		goto out;