This is an automated email from the ASF dual-hosted git repository.

johnnyv pushed a commit to branch 2.2.X
in repository https://gitbox.apache.org/repos/asf/mina.git

commit 667517901d27ff9f342917bf1966aba150a427af
Author: Jonathan Valliere <john...@apache.org>
AuthorDate: Tue Sep 14 13:57:24 2021 -0400

    applies null check
---
 .../org/apache/mina/proxy/handlers/socks/Socks4LogicHandler.java     | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git 
a/mina-core/src/main/java/org/apache/mina/proxy/handlers/socks/Socks4LogicHandler.java
 
b/mina-core/src/main/java/org/apache/mina/proxy/handlers/socks/Socks4LogicHandler.java
index bb1354b..a03bf53 100644
--- 
a/mina-core/src/main/java/org/apache/mina/proxy/handlers/socks/Socks4LogicHandler.java
+++ 
b/mina-core/src/main/java/org/apache/mina/proxy/handlers/socks/Socks4LogicHandler.java
@@ -72,9 +72,8 @@ public class Socks4LogicHandler extends 
AbstractSocksLogicHandler {
     protected void writeRequest(final NextFilter nextFilter, final 
SocksProxyRequest request) {
         try {
             boolean isV4ARequest = Arrays.equals(request.getIpAddress(), 
SocksProxyConstants.FAKE_IP);
-            byte[] userID = request.getUserName().getBytes("ASCII");
-            byte[] host = isV4ARequest ? request.getHost().getBytes("ASCII") : 
null;
-
+            byte[] userID = request.getUserName() != null ? 
request.getUserName().getBytes("ASCII") : null;
+            byte[] host = request.getHost() != null ? 
request.getHost().getBytes("ASCII") : null;           
             int len = 9 + userID.length;
 
             if (isV4ARequest) {

Reply via email to