h4ckx0r/engine.io-client-java
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[Fix] "must call close()" crash

Browse Source
This commit is contained in:
Stefan Haacker
2016-07-26 11:12:55 -04:00
parent edc8626496
commit 1c166b25d9
1 changed files with 7 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/main/java/io/socket/engineio/client/transports/WebSocket.java
View File

@@ -165,6 +165,11 @@ public class WebSocket extends Transport {
final int[] total = new int[]{packets.length}; final int[] total = new int[]{packets.length};
for (Packet packet : packets) { for (Packet packet : packets) {
if (this.readyState != ReadyState.OPENING && this.readyState != ReadyState.OPEN) {
// Ensure we don't try to send anymore packets if the socket ends up being closed due to an exception
break;
}
Parser.encodePacket(packet, new Parser.EncodeCallback() { Parser.encodePacket(packet, new Parser.EncodeCallback() {
@Override @Override
public void call(Object packet) { public void call(Object packet) {
@@ -176,6 +181,7 @@ public class WebSocket extends Transport {
} }
} catch (IOException e) { } catch (IOException e) {
logger.fine("websocket closed before onclose event"); logger.fine("websocket closed before onclose event");
close();
} }
if (0 == --total[0]) done.run(); if (0 == --total[0]) done.run();