From 8b9199ed09acc6cce742b8c0ddc57cc4c0b96a3c Mon Sep 17 00:00:00 2001 From: Michael Bayne Date: Sun, 7 Sep 2003 23:30:13 +0000 Subject: [PATCH] Make sure we don't have any dangling subscriptions if we receive a message after having closed our client connection. git-svn-id: svn+ssh://src.earth.threerings.net/narya/trunk@2791 542714f4-19e9-0310-aa3c-eee0fc999fb1 --- .../threerings/presents/server/PresentsClient.java | 14 ++++++++++---- 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/src/java/com/threerings/presents/server/PresentsClient.java b/src/java/com/threerings/presents/server/PresentsClient.java index 6a9b22c4f..31d5deef5 100644 --- a/src/java/com/threerings/presents/server/PresentsClient.java +++ b/src/java/com/threerings/presents/server/PresentsClient.java @@ -1,5 +1,5 @@ // -// $Id: PresentsClient.java,v 1.56 2003/05/24 01:50:33 mdb Exp $ +// $Id: PresentsClient.java,v 1.57 2003/09/07 23:30:13 mdb Exp $ package com.threerings.presents.server; @@ -672,10 +672,16 @@ public class PresentsClient conn.postMessage(msg); _messagesOut++; // count 'em up! return true; - } else { - Log.info("Dropped message [client=" + this + ", msg=" + msg + "]."); - return false; } + + Log.info("Dropped message [client=" + this + ", msg=" + msg + "]."); + + // make darned sure we don't have any remaining subscriptions + if (_subscrips.size() > 0) { + Log.warning("Clearing stale subscriptions [client=" + this + "]."); + clearSubscrips(); + } + return false; } /**