os: migrate node_os.cc to internal/errors

PR-URL: https://github.com/nodejs/node/pull/16567
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>

1 files changed, 28 insertions, 6 deletions

diff --git a/lib/os.js b/lib/os.js
index 6af5d15cbc..7c07a5b0d3 100644
--- a/lib/os.js
+++ b/lib/os.js
@@ -27,21 +27,43 @@ const { deprecate } = require('internal/util');
 const { getCIDRSuffix } = require('internal/os');
 const isWindows = process.platform === 'win32';
 
+const errors = require('internal/errors');
+
 const {
   getCPUs,
   getFreeMem,
-  getHomeDirectory,
-  getHostname,
-  getInterfaceAddresses,
+  getHomeDirectory: _getHomeDirectory,
+  getHostname: _getHostname,
+  getInterfaceAddresses: _getInterfaceAddresses,
   getLoadAvg,
-  getOSRelease,
-  getOSType,
+  getOSRelease: _getOSRelease,
+  getOSType: _getOSType,
   getTotalMem,
-  getUserInfo,
+  getUserInfo: _getUserInfo,
   getUptime,
   isBigEndian
 } = process.binding('os');
 
+function getCheckedFunction(fn) {
+  return function checkError(...args) {
+    const ctx = {};
+    const ret = fn(...args, ctx);
+    if (ret === undefined) {
+      const err = new errors.SystemError(ctx);
+      Error.captureStackTrace(err, checkError);
+      throw err;
+    }
+    return ret;
+  };
+}
+
+const getHomeDirectory = getCheckedFunction(_getHomeDirectory);
+const getHostname = getCheckedFunction(_getHostname);
+const getInterfaceAddresses = getCheckedFunction(_getInterfaceAddresses);
+const getOSRelease = getCheckedFunction(_getOSRelease);
+const getOSType = getCheckedFunction(_getOSType);
+const getUserInfo = getCheckedFunction(_getUserInfo);
+
 getFreeMem[Symbol.toPrimitive] = () => getFreeMem();
 getHostname[Symbol.toPrimitive] = () => getHostname();
 getHomeDirectory[Symbol.toPrimitive] = () => getHomeDirectory();