Skip to content

Commit ce99600

Browse files
tniessenbengl
authored andcommitted
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 047d4de commit ce99600

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

src/crypto/crypto_common.cc

+7-3
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ using v8::Undefined;
4242
using v8::Value;
4343

4444
namespace crypto {
45-
static constexpr int X509_NAME_FLAGS =
45+
static constexpr int kX509NameFlagsMultiline =
4646
ASN1_STRFLGS_ESC_2253 |
4747
ASN1_STRFLGS_ESC_CTRL |
4848
ASN1_STRFLGS_UTF8_CONVERT |
@@ -988,7 +988,11 @@ MaybeLocal<Value> GetIssuerString(
988988
const BIOPointer& bio,
989989
X509* cert) {
990990
X509_NAME* issuer_name = X509_get_issuer_name(cert);
991-
if (X509_NAME_print_ex(bio.get(), issuer_name, 0, X509_NAME_FLAGS) <= 0) {
991+
if (X509_NAME_print_ex(
992+
bio.get(),
993+
issuer_name,
994+
0,
995+
kX509NameFlagsMultiline) <= 0) {
992996
USE(BIO_reset(bio.get()));
993997
return Undefined(env->isolate());
994998
}
@@ -1004,7 +1008,7 @@ MaybeLocal<Value> GetSubject(
10041008
bio.get(),
10051009
X509_get_subject_name(cert),
10061010
0,
1007-
X509_NAME_FLAGS) <= 0) {
1011+
kX509NameFlagsMultiline) <= 0) {
10081012
USE(BIO_reset(bio.get()));
10091013
return Undefined(env->isolate());
10101014
}

0 commit comments

Comments
 (0)