OPENSSL_assert() is not appropriate in this context
authorjob <job@openbsd.org>
Thu, 2 Sep 2021 15:59:05 +0000 (15:59 +0000)
committerjob <job@openbsd.org>
Thu, 2 Sep 2021 15:59:05 +0000 (15:59 +0000)
Feedback from tb@

OK tb@

lib/libcrypto/x509/x509_asid.c

index 56283cb..bcb6e4e 100644 (file)
@@ -218,11 +218,12 @@ static int ASIdOrRange_cmp(const ASIdOrRange *const *a_,
 {
     const ASIdOrRange *a = *a_, *b = *b_;
 
-    OPENSSL_assert((a->type == ASIdOrRange_id && a->u.id != NULL) ||
+    /* XXX: these asserts need to be replaced */
+    assert((a->type == ASIdOrRange_id && a->u.id != NULL) ||
            (a->type == ASIdOrRange_range && a->u.range != NULL &&
             a->u.range->min != NULL && a->u.range->max != NULL));
 
-    OPENSSL_assert((b->type == ASIdOrRange_id && b->u.id != NULL) ||
+    assert((b->type == ASIdOrRange_id && b->u.id != NULL) ||
            (b->type == ASIdOrRange_range && b->u.range != NULL &&
             b->u.range->min != NULL && b->u.range->max != NULL));