MayaTheShy/Starworld
1
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

fix: update sendPing to set source ID before sequence number for proper header resizing

Browse Source
This commit is contained in:
MayaTheShy
2025-11-10 01:41:21 -05:00
parent 89ec0cd200
commit aa8a04590b
1 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
src/OverteClient.cpp
View File

@@ -1538,10 +1538,13 @@ void OverteClient::handlePing(const char* payload, size_t len) {
void OverteClient::sendPing(int fd, const sockaddr_storage& addr, socklen_t addrLen) { void OverteClient::sendPing(int fd, const sockaddr_storage& addr, socklen_t addrLen) {
// Create NLPacket for Ping with correct version // Create NLPacket for Ping with correct version
NLPacket packet(PacketType::Ping, PacketVersions::Ping_IncludeConnectionID, false); NLPacket packet(PacketType::Ping, PacketVersions::Ping_IncludeConnectionID, false);
// Include our local ID if we have one (sourced packet)
// Set source ID first (this resizes header and writes it)
if (m_localID != 0) { if (m_localID != 0) {
packet.setSourceID(m_localID); packet.setSourceID(m_localID);
} }
// Then set sequence number (this updates the header without resizing)
packet.setSequenceNumber(m_sequenceNumber++); packet.setSequenceNumber(m_sequenceNumber++);
// Add timestamp (microseconds since epoch) // Add timestamp (microseconds since epoch)