FlightSim-Streaming/XPlaneConnectCSP
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fixed bug in DATA message handler.

Browse Source 
- Data validation checks were being executed in the wrong order. This caused most valid DATA commands to be reported as empty.
This commit is contained in:
Jason Watkins
2015-04-24 09:39:15 -07:00
parent d03a6985dc
commit 625482dcdd
1 changed files with 9 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
xpcPlugin/MessageHandlers.cpp
View File

@@ -228,13 +228,6 @@ namespace XPC
Log::FormatLine("[DATA] Message Received (Conn %i)", connection.id); Log::FormatLine("[DATA] Message Received (Conn %i)", connection.id);
#endif #endif
} }
if (numCols > 32) // Error. Will overflow values
{
#if LOG_VERBOSITY > 0
Log::FormatLine("[DATA] ERROR numCols to large.");
#endif
return;
}
else else
{ {
#if LOG_VERBOSITY > 1 #if LOG_VERBOSITY > 1
@@ -242,7 +235,15 @@ namespace XPC
#endif #endif
return; return;
} }
float values[32][9];
if (numCols > 134) // Error. Will overflow values
{
#if LOG_VERBOSITY > 0
Log::FormatLine("[DATA] ERROR numCols to large.");
#endif
return;
}
float values[134][9];
for (int i = 0; i < numCols; ++i) for (int i = 0; i < numCols; ++i)
{ {
values[i][0] = buffer[5 + 36 * i]; values[i][0] = buffer[5 + 36 * i];