godotengine
/
godot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #95194 from Faless/mp/fix_relay_negative_target 

[MP] Fix relay protocol routing with negative targets
This commit is contained in:
Rémi Verschelde 2024-08-08 10:16:31 +02:00
parent 32eb97fd2a c5fa7354bb
commit 03afb92efa
No known key found for this signature in database
GPG Key ID: C3336907360768E1
1 changed files with 8 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
modules/multiplayer/scene_multiplayer.cpp
View File

@ -321,21 +321,24 @@ void SceneMultiplayer::_process_sys(int p_from, const uint8_t *p_packet, int p_p
multiplayer_peer->set_transfer_mode(p_mode); multiplayer_peer->set_transfer_mode(p_mode);
multiplayer_peer->set_transfer_channel(p_channel); multiplayer_peer->set_transfer_channel(p_channel);
if (peer > 0) { if (peer > 0) {
// Single destination.
multiplayer_peer->set_target_peer(peer); multiplayer_peer->set_target_peer(peer);
_send(data.ptr(), relay_buffer->get_position()); _send(data.ptr(), relay_buffer->get_position());
} else { } else {
// Multiple destinations.
for (const int &P : connected_peers) { for (const int &P : connected_peers) {
// Not to sender, nor excluded. // Not to sender, nor excluded.
if (P == p_from || (peer < 0 && P != -peer)) { if (P == p_from || P == -peer) {
continue; continue;
} }
multiplayer_peer->set_target_peer(P); multiplayer_peer->set_target_peer(P);
_send(data.ptr(), relay_buffer->get_position()); _send(data.ptr(), relay_buffer->get_position());
} }
} if (peer != -1) {
if (peer == 0 || peer == -1) { // The server is one of the targets, process the packet with sender as source.
should_process = true; should_process = true;
peer = p_from; // Process as the source. peer = p_from;
}
} }
} else { } else {
ERR_FAIL_COND(p_from != 1); // Bug. ERR_FAIL_COND(p_from != 1); // Bug.