diff --git a/src/rostermanager.cpp b/src/rostermanager.cpp
index 4d8afba9048fea5c726f1bf0a158853fb9459574..1bde017318445eb439495dc55ac744e47900298b 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;
 			}