flightsimulatorplatfrom/pilotclient
1
0
Fork 0
mirror of https://github.com/swift-project/pilotclient.git synced 2026-05-06 02:16:04 +08:00
Code Issues Packages Projects Releases Wiki Activity

[FS9] Add missing null termination when serializing QStrings

Browse Source 
The produced QByteArray was missing the null terminating character,
since QByteArray::append() only added up to and not including '\0'.
This was also the reason why sometimes planes did not change their
aircraft models properly.
This commit is contained in:
Roland Winklmeier
2016-11-06 00:32:17 +01:00
committed by Klaus Basan
parent ea561cf723
commit a13f7eb1d4
2 changed files with 1 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
src/plugins/simulator/fs9/fs9client.cpp
View File

@@ -316,9 +316,6 @@ namespace BlackSimPlugin
MultiPlayerPacketParser::writeSize(message, mpChangePlayerPlane.size()); MultiPlayerPacketParser::writeSize(message, mpChangePlayerPlane.size());
message = MultiPlayerPacketParser::writeMessage(message, mpChangePlayerPlane); message = MultiPlayerPacketParser::writeMessage(message, mpChangePlayerPlane);
CLogMessage(this).debug() << m_callsign << " connected to session."; CLogMessage(this).debug() << m_callsign << " connected to session.";
// Send it several times, since one got missed several times.
sendMessage(message);
sendMessage(message);
sendMessage(message); sendMessage(message);
m_timerId = startTimer(m_updateInterval.value(CTimeUnit::ms())); m_timerId = startTimer(m_updateInterval.value(CTimeUnit::ms()));

2
src/plugins/simulator/fs9/multiplayerpacketparser.cpp
View File

@@ -29,7 +29,7 @@ namespace BlackSimPlugin
QByteArray writeValue(QByteArray data, const QString &value) QByteArray writeValue(QByteArray data, const QString &value)
{ {
data.append(qPrintable(value)); data.append(qPrintable(value), value.size() + 1);
return data; return data;
} }