Skip to content

Commit f5791ec

Browse files
committed
Merge pull request TooTallNate#159 from MorganWu/master
fix NullPointerException in WebSocketClient
2 parents 85efa0a + a18d0f7 commit f5791ec

File tree

1 file changed

+10
-2
lines changed

1 file changed

+10
-2
lines changed

src/main/java/org/java_websocket/client/WebSocketClient.java

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -120,7 +120,11 @@ public WebSocketClient( URI serverUri , Draft draft , Map<String,String> headers
120120
conn = (WebSocketImpl) wf.createWebSocket( this, draft, channel.socket() );
121121
} catch ( IOException e ) {
122122
onWebsocketError( null, e );
123-
conn.closeConnection( CloseFrame.NEVER_CONNECTED, e.getMessage() );
123+
124+
if(conn != null) {
125+
conn.closeConnection( CloseFrame.NEVER_CONNECTED, e.getMessage() );
126+
}
127+
124128
return;
125129
}
126130
}
@@ -223,7 +227,11 @@ private final void interruptableRun() {
223227
return;
224228
} catch ( /*IOException | SecurityException | UnresolvedAddressException*/Exception e ) {//
225229
onWebsocketError( conn, e );
226-
conn.closeConnection( CloseFrame.NEVER_CONNECTED, e.getMessage() );
230+
231+
if(conn != null) {
232+
conn.closeConnection( CloseFrame.NEVER_CONNECTED, e.getMessage() );
233+
}
234+
227235
return;
228236
}
229237

0 commit comments

Comments
 (0)