From 19c1c712281e4a0a98ae9d864cae3fa13df9aa0b Mon Sep 17 00:00:00 2001
From: Matheus Marchini <matheus@sthima.com.br>
Date: Mon, 4 Dec 2017 19:08:15 -0200
Subject: [PATCH] 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: https://github.com/nodejs/node/pull/17461
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
---
 node.gyp | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/node.gyp b/node.gyp
index d5cd268c5956cf..4ae9d36a552413 100644
--- a/node.gyp
+++ b/node.gyp
@@ -841,6 +841,9 @@
             '<(OBJ_PATH)<(OBJ_SEPARATOR)node_crypto_clienthello.<(OBJ_SUFFIX)',
             '<(OBJ_PATH)<(OBJ_SEPARATOR)tls_wrap.<(OBJ_SUFFIX)',
           ],
+          'defines': [
+            'HAVE_OPENSSL=1',
+          ],
         }],
         ['v8_enable_inspector==1', {
           'sources': [