net: check pipe mode and path

PR-URL: #50770
Reviewed-By: Vinícius Lourenço Claro Cardoso <contact@viniciusl.com.br>
Reviewed-By: James M Snell <jasnell@gmail.com>

Author: theanarkh

lib/net.js

@@ -2014,6 +2014,12 @@ Server.prototype.listen = function(...args) {
 // (path[, backlog][, cb]) or (options[, cb])
 // where path or options.path is a UNIX domain socket or Windows pipe
 if(options.path&&isPipeName(options.path)){
+// We can not call fchmod on abstract unix socket
+if(options.path[0]==='\0'&&
+(options.readableAll||options.writableAll)){
+constmsg='can not set readableAll or writableAllt to true when path is abstract unix socket';
+thrownewERR_INVALID_ARG_VALUE('options',options,msg);
+}
 constpipeName=this._pipeName=options.path;
 backlog=options.backlog||backlogFromArgs;
 listenInCluster(this,

test/parallel/test-pipe-abstract-socket.js

@@ -0,0 +1,34 @@
+'use strict';
+constcommon=require('../common');
+constassert=require('assert');
+constnet=require('net');
+
+if(!common.isLinux)common.skip();
+
+constpath='\0abstract';
+constmessage=/cannotsetreadableAllorwritableAllttotruewhenpathisabstractunixsocket/;
+
+assert.throws(()=>{
+constserver=net.createServer(common.mustNotCall());
+server.listen({
+ path,
+readableAll: true
+});
+},message);
+
+assert.throws(()=>{
+constserver=net.createServer(common.mustNotCall());
+server.listen({
+ path,
+writableAll: true
+});
+},message);
+
+assert.throws(()=>{
+constserver=net.createServer(common.mustNotCall());
+server.listen({
+ path,
+readableAll: true,
+writableAll: true
+});
+},message);