nodejs
diff --git a/‎lib/internal/validators.js‎
Lines changed: 1 addition & 1 deletion b/‎lib/internal/validators.js‎
Lines changed: 1 addition & 1 deletion
diff --git a/‎test/parallel/test-internal-validators-validateport.js‎
Lines changed: 14 additions & 0 deletions b/‎test/parallel/test-internal-validators-validateport.js‎
Lines changed: 14 additions & 0 deletions
@@ -438,7 +438,7 @@ const validatePort = hideStackFrames((port, name = 'Port', allowZero = true) =>
       (typeof port === 'string' && StringPrototypeTrim(port).length === 0) ||
       +port !== (+port >>> 0) ||
       port > 0xFFFF ||
-      (port === 0 && !allowZero)) {
+      (+port === 0 && !allowZero)) {
     throw new ERR_SOCKET_BAD_PORT(name, port, allowZero);
   }
   return port | 0;
 
@@ -21,3 +21,17 @@ for (let n = 0; n <= 0xFFFF; n++) {
 ].forEach((i) => assert.throws(() => validatePort(i), {
   code: 'ERR_SOCKET_BAD_PORT'
 }));
+
+// When allowZero is false, every form of zero must be rejected, including
+// the string forms that coerce to 0. Refs: the zero check must coerce the
+// value the same way the rest of the validation does (`+port`).
+[
+  0, '0', ' 0 ', '00', '0x0', '0o0', '0b0',
+].forEach((i) => assert.throws(() => validatePort(i, 'Port', false), {
+  code: 'ERR_SOCKET_BAD_PORT'
+}));
+
+// With allowZero left at its default (true), those same values are accepted.
+[
+  0, '0', ' 0 ', '00', '0x0', '0o0', '0b0',
+].forEach((i) => assert.strictEqual(validatePort(i), 0));
Original file line number	Diff line number	Diff line change
`@@ -438,7 +438,7 @@ const validatePort = hideStackFrames((port, name = 'Port', allowZero = true) =>`
`438`	`438`	`(typeof port === 'string' && StringPrototypeTrim(port).length === 0) \|\|`
`439`	`439`	`+port !== (+port >>> 0) \|\|`
`440`	`440`	`port > 0xFFFF \|\|`
`441`		`- (port === 0 && !allowZero)) {`
	`441`	`+ (+port === 0 && !allowZero)) {`
`442`	`442`	`throw new ERR_SOCKET_BAD_PORT(name, port, allowZero);`
`443`	`443`	`}`
`444`	`444`	`return port \| 0;`