From 5407885d87e772765b49a35d8622012f41a1ec1d Mon Sep 17 00:00:00 2001 From: HanzZ Date: Fri, 9 Mar 2012 19:36:48 +0100 Subject: [PATCH] Do not send available presence as response to subscribed --- src/rostermanager.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/rostermanager.cpp b/src/rostermanager.cpp index 4d8afba9..1bde0173 100644 --- a/src/rostermanager.cpp +++ b/src/rostermanager.cpp @@ -342,7 +342,7 @@ void RosterManager::handleSubscription(Swift::Presence::ref presence) { break; case Swift::Presence::Subscribed: // onBuddyAdded(buddy); - break; + return; // buddy is already there, so nothing to do, just answer case Swift::Presence::Unsubscribe: response->setType(Swift::Presence::Unsubscribed); @@ -397,7 +397,7 @@ void RosterManager::handleSubscription(Swift::Presence::ref presence) { buddy->setSubscription(Buddy::Both); handleBuddyChanged(buddy); } - break; + return; default: return; }