add a missing break, don't double activate sockets

This commit is contained in:
Stephen Birarda 2014-01-23 15:05:09 -08:00
parent e7927ff837
commit 09cd8eb0fd
2 changed files with 3 additions and 3 deletions

View file

@ -148,9 +148,9 @@ void AvatarMixer::processDatagram(const QByteArray& dataByteArray, const HifiSoc
// parse positional data from an node // parse positional data from an node
nodeList->updateNodeWithData(avatarNode.data(), senderSockAddr, nodeList->updateNodeWithData(avatarNode.data(), senderSockAddr,
(unsigned char*) dataByteArray.data(), dataByteArray.size()); (unsigned char*) dataByteArray.data(), dataByteArray.size());
} else {
break;
} }
break;
} }
case PACKET_TYPE_KILL_AVATAR: { case PACKET_TYPE_KILL_AVATAR: {
nodeList->processKillNode(dataByteArray); nodeList->processKillNode(dataByteArray);

View file

@ -186,7 +186,7 @@ int NodeList::updateNodeWithData(Node *node, const HifiSockAddr& senderSockAddr,
node->setLastHeardMicrostamp(usecTimestampNow()); node->setLastHeardMicrostamp(usecTimestampNow());
if (!senderSockAddr.isNull()) { if (!senderSockAddr.isNull() && !node->getActiveSocket()) {
if (senderSockAddr == node->getPublicSocket()) { if (senderSockAddr == node->getPublicSocket()) {
node->activatePublicSocket(); node->activatePublicSocket();
} else if (senderSockAddr == node->getLocalSocket()) { } else if (senderSockAddr == node->getLocalSocket()) {