From cd752e8463fad7c4805951d9ba47cd2f39691f2d Mon Sep 17 00:00:00 2001
From: "Steven R. Loomis" <srloomis@us.ibm.com>
Date: Fri, 15 Apr 2016 21:35:53 -0700
Subject: [PATCH] intl: Don't crash if v8BreakIterator not available

If the undocumented v8BreakIterator does not have data available,
monkeypatch it to throw an error instead of crashing.

Fixes: https://github.com/nodejs/node/issues/3111
PR-URL: https://github.com/nodejs/node/pull/4253
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
---
 lib/internal/process.js                    | 15 +++++++++++++++
 src/node.cc                                |  7 +++++++
 test/parallel/test-intl-v8BreakIterator.js | 15 +++++++++++++++
 3 files changed, 37 insertions(+)
 create mode 100644 test/parallel/test-intl-v8BreakIterator.js

diff --git a/lib/internal/process.js b/lib/internal/process.js
index 881d91b11e..c435c2e871 100644
--- a/lib/internal/process.js
+++ b/lib/internal/process.js
@@ -117,6 +117,21 @@ function setupConfig(_source) {
     if (value === 'false') return false;
     return value;
   });
+  const processConfig = process.binding('config');
+  // Intl.v8BreakIterator() would crash w/ fatal error, so throw instead.
+  if (processConfig.hasIntl &&
+      processConfig.hasSmallICU &&
+      Intl.hasOwnProperty('v8BreakIterator') &&
+      !process.icu_data_dir) {
+    const des = Object.getOwnPropertyDescriptor(Intl, 'v8BreakIterator');
+    des.value = function v8BreakIterator() {
+      throw new Error('v8BreakIterator: full ICU data not installed. ' +
+                      'See https://github.com/nodejs/node/wiki/Intl');
+    };
+    Object.defineProperty(Intl, 'v8BreakIterator', des);
+  }
+  // Don’t let icu_data_dir leak through.
+  delete process.icu_data_dir;
 }
 
 
diff --git a/src/node.cc b/src/node.cc
index 6f155a64a3..eb8805b219 100644
--- a/src/node.cc
+++ b/src/node.cc
@@ -2976,6 +2976,13 @@ void SetupProcessObject(Environment* env,
   READONLY_PROPERTY(versions,
                     "icu",
                     OneByteString(env->isolate(), U_ICU_VERSION));
+
+  if (icu_data_dir != nullptr) {
+    // Did the user attempt (via env var or parameter) to set an ICU path?
+    READONLY_PROPERTY(process,
+                      "icu_data_dir",
+                      OneByteString(env->isolate(), icu_data_dir));
+  }
 #endif
 
   const char node_modules_version[] = NODE_STRINGIFY(NODE_MODULE_VERSION);
diff --git a/test/parallel/test-intl-v8BreakIterator.js b/test/parallel/test-intl-v8BreakIterator.js
new file mode 100644
index 0000000000..6e25c32cd7
--- /dev/null
+++ b/test/parallel/test-intl-v8BreakIterator.js
@@ -0,0 +1,15 @@
+'use strict';
+require('../common');
+const assert = require('assert');
+
+if (global.Intl === undefined || Intl.v8BreakIterator === undefined) {
+  return console.log('1..0 # Skipped: no Intl');
+}
+
+try {
+  new Intl.v8BreakIterator();
+  // May succeed if data is available - OK
+} catch (e) {
+  // May throw this error if ICU data is not available - OK
+  assert.throws(() => new Intl.v8BreakIterator(), /ICU data/);
+}