Skip to content

Commit c1689dc

Browse files
Matheus Marchiniapapirovski
authored andcommitted
build: define HAVE_OPENSSL macro for cctest
cctest build target wasn't defining the HAVE_OPENSSL macro when node_use_openssl was true, causing inconsistencies on most `node::Environment` member's addresses. For example, if someone wanted to access the context of an environment by using `node::Environment::context()`, the object returned by the function was pointing to an invalid address. PR-URL: #17461 Reviewed-By: Joyee Cheung <[email protected]> Reviewed-By: Daniel Bevenius <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Anna Henningsen <[email protected]>
1 parent 987b1e4 commit c1689dc

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

node.gyp

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -864,6 +864,9 @@
864864
'<(OBJ_PATH)<(OBJ_SEPARATOR)node_crypto_clienthello.<(OBJ_SUFFIX)',
865865
'<(OBJ_PATH)<(OBJ_SEPARATOR)tls_wrap.<(OBJ_SUFFIX)',
866866
],
867+
'defines': [
868+
'HAVE_OPENSSL=1',
869+
],
867870
}],
868871
['v8_enable_inspector==1', {
869872
'sources': [

0 commit comments

Comments
 (0)