Skip to content

Commit 19370b3

Browse files
authored
crypto: rename X509_NAME_FLAGS
Rename X509_NAME_FLAGS to kX509NameFlagsMultiline to better align with the naming conventions we use for constexpr values, to distinguish it from OpenSSL's built-in X509 constants, and to clarify what specific X509 name flags the constant represents. PR-URL: #42001 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]>
1 parent 5edad38 commit 19370b3

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

src/crypto/crypto_common.cc

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ using v8::Undefined;
4141
using v8::Value;
4242

4343
namespace crypto {
44-
static constexpr int X509_NAME_FLAGS =
44+
static constexpr int kX509NameFlagsMultiline =
4545
ASN1_STRFLGS_ESC_2253 |
4646
ASN1_STRFLGS_ESC_CTRL |
4747
ASN1_STRFLGS_UTF8_CONVERT |
@@ -940,7 +940,11 @@ MaybeLocal<Value> GetIssuerString(
940940
const BIOPointer& bio,
941941
X509* cert) {
942942
X509_NAME* issuer_name = X509_get_issuer_name(cert);
943-
if (X509_NAME_print_ex(bio.get(), issuer_name, 0, X509_NAME_FLAGS) <= 0) {
943+
if (X509_NAME_print_ex(
944+
bio.get(),
945+
issuer_name,
946+
0,
947+
kX509NameFlagsMultiline) <= 0) {
944948
USE(BIO_reset(bio.get()));
945949
return Undefined(env->isolate());
946950
}
@@ -956,7 +960,7 @@ MaybeLocal<Value> GetSubject(
956960
bio.get(),
957961
X509_get_subject_name(cert),
958962
0,
959-
X509_NAME_FLAGS) <= 0) {
963+
kX509NameFlagsMultiline) <= 0) {
960964
USE(BIO_reset(bio.get()));
961965
return Undefined(env->isolate());
962966
}

0 commit comments

Comments
 (0)