dgram: fix possibly deoptimizing use of arguments

This commit adds a guard against an out of bounds access of arguments, and replaces another use of arguments with a named function parameter. Refs: #10323 PR-URL: #11242 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]>

Author: vsemozhetbyt

lib/dgram.js

@@ -131,7 +131,7 @@ function replaceHandle(self, newHandle){
 self._handle=newHandle;
 }
 
-Socket.prototype.bind=function(port_/*, address, callback*/){
+Socket.prototype.bind=function(port_,address_/*, callback*/){
 letport=port_;
 
 this._healthCheck();
@@ -141,7 +141,7 @@ Socket.prototype.bind = function(port_ /*, address, callback*/){
 
 this._bindState=BIND_STATE_BINDING;
 
-if(typeofarguments[arguments.length-1]==='function')
+if(arguments.length&&typeofarguments[arguments.length-1]==='function')
 this.once('listening',arguments[arguments.length-1]);
 
 if(portinstanceofUDP){
@@ -158,7 +158,7 @@ Socket.prototype.bind = function(port_ /*, address, callback*/){
 exclusive=!!port.exclusive;
 port=port.port;
 }else{
-address=typeofarguments[1]==='function' ? '' : arguments[1];
+address=typeofaddress_==='function' ? '' : address_;
 exclusive=false;
 }