Skip to content

Commit

Permalink
x509_acert: Add and retrieve certificate extensions
Browse files Browse the repository at this point in the history
Add API to manage attribute certificate extensions

Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Neil Horman <nhorman@openssl.org>
(Merged from #15857)
  • Loading branch information
dhobsong authored and mattcaswell committed Apr 24, 2024
1 parent b97fb22 commit 1eeec94
Show file tree
Hide file tree
Showing 4 changed files with 39 additions and 4 deletions.
16 changes: 16 additions & 0 deletions crypto/x509/x509_acert.c
Original file line number Diff line number Diff line change
Expand Up @@ -242,3 +242,19 @@ int X509_ACERT_add1_attr_by_txt(X509_ACERT *x, const char *attrname, int type,

return X509at_add1_attr_by_txt(attrs, attrname, type, bytes, len) != NULL;
}

void *X509_ACERT_get_ext_d2i(const X509_ACERT *x, int nid, int *crit, int *idx)
{
return X509V3_get_d2i(x->acinfo->extensions, nid, crit, idx);
}

int X509_ACERT_add1_ext_i2d(X509_ACERT *x, int nid, void *value, int crit,
unsigned long flags)
{
return X509V3_add1_i2d(&x->acinfo->extensions, nid, value, crit, flags);
}

const STACK_OF(X509_EXTENSION) *X509_ACERT_get0_extensions(const X509_ACERT *x)
{
return x->acinfo->extensions;
}
19 changes: 15 additions & 4 deletions doc/man3/X509V3_get_d2i.pod
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,10 @@

X509V3_get_d2i, X509V3_add1_i2d, X509V3_EXT_d2i, X509V3_EXT_i2d,
X509_get_ext_d2i, X509_add1_ext_i2d,
X509_ACERT_get_ext_d2i, X509_ACERT_add1_ext_i2d,
X509_CRL_get_ext_d2i, X509_CRL_add1_ext_i2d,
X509_REVOKED_get_ext_d2i, X509_REVOKED_add1_ext_i2d,
X509_get0_extensions, X509_CRL_get0_extensions,
X509_get0_extensions, X509_ACERT_get0_extensions, X509_CRL_get0_extensions,
X509_REVOKED_get0_extensions - X509 extension decode and encode functions

=head1 SYNOPSIS
Expand All @@ -25,6 +26,10 @@ X509_REVOKED_get0_extensions - X509 extension decode and encode functions
int X509_add1_ext_i2d(X509 *x, int nid, void *value, int crit,
unsigned long flags);

void *X509_ACERT_get_ext_d2i(const X509_ACERT *x, int nid, int *crit, int *idx);
int X509_ACERT_add1_ext_i2d(X509_ACERT *x, int nid, void *value, int crit,
unsigned long flags);

void *X509_CRL_get_ext_d2i(const X509_CRL *crl, int nid, int *crit, int *idx);
int X509_CRL_add1_ext_i2d(X509_CRL *crl, int nid, void *value, int crit,
unsigned long flags);
Expand All @@ -34,6 +39,7 @@ X509_REVOKED_get0_extensions - X509 extension decode and encode functions
unsigned long flags);

const STACK_OF(X509_EXTENSION) *X509_get0_extensions(const X509 *x);
const STACK_OF(X509_EXTENSION) *X509_ACERT_get0_extensions(const X509 *x);
const STACK_OF(X509_EXTENSION) *X509_CRL_get0_extensions(const X509_CRL *crl);
const STACK_OF(X509_EXTENSION) *X509_REVOKED_get0_extensions(const X509_REVOKED *r);

Expand Down Expand Up @@ -64,6 +70,10 @@ X509_get_ext_d2i() and X509_add1_ext_i2d() operate on the extensions of
certificate I<x>. They are otherwise identical to X509V3_get_d2i() and
X509V3_add1_i2d().

X509_ACERT_get_ext_d2i() and X509_ACERT_add1_ext_i2d() operate on the extensions
of B<X509_ACERT> structure I<x>. They are otherwise identical to X509V3_get_d2i()
and X509V3_add1_i2d().

X509_CRL_get_ext_d2i() and X509_CRL_add1_ext_i2d() operate on the extensions
of CRL I<crl>. They are otherwise identical to X509V3_get_d2i() and
X509V3_add1_i2d().
Expand All @@ -72,9 +82,10 @@ X509_REVOKED_get_ext_d2i() and X509_REVOKED_add1_ext_i2d() operate on the
extensions of B<X509_REVOKED> structure I<r> (i.e for CRL entry extensions).
They are otherwise identical to X509V3_get_d2i() and X509V3_add1_i2d().

X509_get0_extensions(), X509_CRL_get0_extensions() and
X509_REVOKED_get0_extensions() return a STACK of all the extensions
of a certificate, a CRL or a CRL entry respectively.
X509_get0_extensions(), X509_ACERT_get0_extensions(),
X509_CRL_get0_extensions() and X509_REVOKED_get0_extensions() return a
STACK of all the extensions of a certificate, an attribute certificate,
a CRL or a CRL entry respectively.

=head1 NOTES

Expand Down
5 changes: 5 additions & 0 deletions include/openssl/x509_acert.h.in
Original file line number Diff line number Diff line change
Expand Up @@ -77,6 +77,11 @@ int X509_ACERT_get_attr_by_OBJ(const X509_ACERT *x, const ASN1_OBJECT *obj,
X509_ATTRIBUTE *X509_ACERT_get_attr(const X509_ACERT *x, int loc);
X509_ATTRIBUTE *X509_ACERT_delete_attr(X509_ACERT *x, int loc);

void *X509_ACERT_get_ext_d2i(const X509_ACERT *x, int nid, int *crit, int *idx);
int X509_ACERT_add1_ext_i2d(X509_ACERT *x, int nid, void *value, int crit,
unsigned long flags);
const STACK_OF(X509_EXTENSION) *X509_ACERT_get0_extensions(const X509_ACERT *x);

# define OSSL_OBJECT_DIGEST_INFO_PUBLIC_KEY 0
# define OSSL_OBJECT_DIGEST_INFO_PUBLIC_KEY_CERT 1
# define OSSL_OBJECT_DIGEST_INFO_OTHER 2 /* must not be used in RFC 5755 profile */
Expand Down
3 changes: 3 additions & 0 deletions util/libcrypto.num
Original file line number Diff line number Diff line change
Expand Up @@ -5615,3 +5615,6 @@ X509_ACERT_add1_attr_by_txt ? 3_4_0 EXIST::FUNCTION:
X509_ACERT_sign ? 3_4_0 EXIST::FUNCTION:
X509_ACERT_sign_ctx ? 3_4_0 EXIST::FUNCTION:
X509_ACERT_verify ? 3_4_0 EXIST::FUNCTION:
X509_ACERT_get_ext_d2i ? 3_4_0 EXIST::FUNCTION:
X509_ACERT_add1_ext_i2d ? 3_4_0 EXIST::FUNCTION:
X509_ACERT_get0_extensions ? 3_4_0 EXIST::FUNCTION:

0 comments on commit 1eeec94

Please sign in to comment.