Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Fix serialization of std::time_t by casting to u64 first (#8353)
Fixes #8332
  • Loading branch information
rubenwardy committed Mar 10, 2019
1 parent 77961aa commit 3b25b80
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 22 deletions.
4 changes: 3 additions & 1 deletion src/network/clientpackethandler.cpp
Expand Up @@ -408,7 +408,9 @@ void Client::handleCommand_ChatMessage(NetworkPacket *pkt)
return;
}

*pkt >> chatMessage->sender >> chatMessage->message >> chatMessage->timestamp;
u64 timestamp;
*pkt >> chatMessage->sender >> chatMessage->message >> timestamp;
chatMessage->timestamp = static_cast<std::time_t>(timestamp);

chatMessage->type = (ChatMessageType) message_type;

Expand Down
16 changes: 0 additions & 16 deletions src/network/networkpacket.cpp
Expand Up @@ -281,12 +281,6 @@ NetworkPacket& NetworkPacket::operator<<(u64 src)
return *this;
}

NetworkPacket& NetworkPacket::operator<<(std::time_t src)
{
*this << (u64) src;
return *this;
}

NetworkPacket& NetworkPacket::operator<<(float src)
{
checkDataSize(4);
Expand Down Expand Up @@ -372,16 +366,6 @@ NetworkPacket& NetworkPacket::operator>>(u64& dst)
return *this;
}

NetworkPacket& NetworkPacket::operator>>(std::time_t& dst)
{
checkReadOffset(m_read_offset, 8);

dst = readU64(&m_data[m_read_offset]);

m_read_offset += 8;
return *this;
}

NetworkPacket& NetworkPacket::operator>>(float& dst)
{
checkReadOffset(m_read_offset, 4);
Expand Down
4 changes: 0 additions & 4 deletions src/network/networkpacket.h
Expand Up @@ -19,7 +19,6 @@ with this program; if not, write to the Free Software Foundation, Inc.,

#pragma once

#include <ctime>
#include "util/pointer.h"
#include "util/numeric.h"
#include "networkprotocol.h"
Expand Down Expand Up @@ -88,9 +87,6 @@ class NetworkPacket
NetworkPacket &operator>>(u64 &dst);
NetworkPacket &operator<<(u64 src);

NetworkPacket &operator>>(std::time_t &dst);
NetworkPacket &operator<<(std::time_t src);

NetworkPacket &operator>>(float &dst);
NetworkPacket &operator<<(float src);

Expand Down
2 changes: 1 addition & 1 deletion src/server.cpp
Expand Up @@ -1559,7 +1559,7 @@ void Server::SendChatMessage(session_t peer_id, const ChatMessage &message)
NetworkPacket pkt(TOCLIENT_CHAT_MESSAGE, 0, peer_id);
u8 version = 1;
u8 type = message.type;
pkt << version << type << std::wstring(L"") << message.message << message.timestamp;
pkt << version << type << std::wstring(L"") << message.message << (u64)message.timestamp;

if (peer_id != PEER_ID_INEXISTENT) {
RemotePlayer *player = m_env->getPlayer(peer_id);
Expand Down

0 comments on commit 3b25b80

Please sign in to comment.