From 4cc361a1e7383b9607508d97fcb25cc285442b1d Mon Sep 17 00:00:00 2001 From: andpal Date: Mon, 23 Mar 2015 11:00:39 +0100 Subject: [PATCH] Changed how clientRooms map is cleaned up after a client leaves. It should be removed from the map even if the client was connected to one or more rooms. Otherwise the entry will stay in memory, causing a leak. --- .../java/com/corundumstudio/socketio/namespace/Namespace.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/main/java/com/corundumstudio/socketio/namespace/Namespace.java b/src/main/java/com/corundumstudio/socketio/namespace/Namespace.java index 95912b4..1ea1c00 100644 --- a/src/main/java/com/corundumstudio/socketio/namespace/Namespace.java +++ b/src/main/java/com/corundumstudio/socketio/namespace/Namespace.java @@ -299,7 +299,7 @@ public class Namespace implements SocketIONamespace { public void leave(String room, UUID sessionId) { leave(roomClients, room, sessionId); - leave(clientRooms, sessionId, room); + clientRooms.remove(sessionId); } public Set getRooms(SocketIOClient client) {