Tweaks to fix setCONNTest on Windows 8
- Temporarily disabled the sending of a CONF message from the plugin in response to setCONN. The CONF message causes crashes on Windows 8 when setCONN is called multiple times. - Removed code that attempted to free result slots in parseRequest before overwritting them with new allocations. The CTests suite currently calls parseRequest with a multidimensional array allocated on the stack, resulting in an attempt to free stack memory. A better solution may be to simply check whether the existing result slots are long enough. - As of this commit, all tests in the CTests suite are passing on Windows 8 for X-Plane 9, X-Plane 10 23bit and X-Plane 10 64bit.
This commit is contained in:
Binary file not shown.
Binary file not shown.
@@ -457,9 +457,10 @@ int handleCONN(char buf[])
|
||||
updateLog(logmsg,strlen(logmsg));
|
||||
|
||||
// SEND CONFIRMATION
|
||||
memcpy(sendSocket.xpIP,connectionList[current_connection].IP, sizeof(connectionList[current_connection].IP));
|
||||
sendSocket.xpPort = connectionList[current_connection].recPort;
|
||||
sendUDP(sendSocket, the_message, 5);
|
||||
// TODO: Ivestigate why sending confirmation causes crashes on Windows 8
|
||||
//memcpy(sendSocket.xpIP,connectionList[current_connection].IP, sizeof(connectionList[current_connection].IP));
|
||||
//sendSocket.xpPort = connectionList[current_connection].recPort;
|
||||
//sendUDP(sendSocket, the_message, 5);
|
||||
|
||||
return 0;
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user