fixup! util: handle uncommon thrown errors

Author: yvele

lib/internal/util/inspect.js

@@ -2557,13 +2557,14 @@ function formatProperty(ctx, value, recurseTimes, key, type, desc,
         }
         ctx.indentationLvl -= 2;
       } catch (err) {
-        let messageSuffix;
+        // Error message itself may be a getter
+        let errMessage;
         try {
-          // Error message itself may be a getter
-          messageSuffix = ` (${err.message})`;
+          errMessage = err.message;
         } catch {
-          messageSuffix = '';
+          errMessage = undefined;
         }
+        const messageSuffix = typeof errMessage === 'string' ? ` (${errMessage})` : '';
         const message = `<Inspection threw${messageSuffix}>`;
         str = `${s(`[${label}:`, sp)} ${message}${s(']', sp)}`;
       }

test/parallel/test-util-inspect.js

@@ -2565,20 +2565,20 @@ assert.strictEqual(
       /* eslint-disable-next-line no-throw-literal */
       get foo() { throw true; }
     }, { getters: true }),
-    '{ foo: [Getter: <Inspection threw (undefined)>] }'
+    '{ foo: [Getter: <Inspection threw>] }'
   );
   assert.strictEqual(
     inspect({
       /* eslint-disable-next-line no-throw-literal */
       get foo() { throw {}; }
     }, { getters: true }),
-    '{ foo: [Getter: <Inspection threw (undefined)>] }'
+    '{ foo: [Getter: <Inspection threw>] }'
   );
   assert.strictEqual(
     inspect({
       get foo() { throw Error; }
     }, { getters: true }),
-    '{ foo: [Getter: <Inspection threw (undefined)>] }'
+    '{ foo: [Getter: <Inspection threw>] }'
   );
 }