Enable enforcing of network packet constraints. Did I mention asn1c is really cool? Anyway, some constraints are still wrong, so this breaks game creation. I will fix it this evening.
This commit is contained in:
@@ -54,7 +54,7 @@ ReceiverHelper::ScanPackets(ReceiveBuffer &buf)
|
|||||||
// This call will also handle the memmove stuff, i.e.
|
// This call will also handle the memmove stuff, i.e.
|
||||||
// buffering for partial packets.
|
// buffering for partial packets.
|
||||||
tmpPacket = NetPacket::Create(buf.recvBuf, buf.recvBufUsed);
|
tmpPacket = NetPacket::Create(buf.recvBuf, buf.recvBufUsed);
|
||||||
} catch (const NetException &e)
|
} catch (const exception &e)
|
||||||
{
|
{
|
||||||
// Reset buffer on error.
|
// Reset buffer on error.
|
||||||
buf.recvBufUsed = 0;
|
buf.recvBufUsed = 0;
|
||||||
@@ -64,7 +64,10 @@ ReceiverHelper::ScanPackets(ReceiveBuffer &buf)
|
|||||||
if (tmpPacket)
|
if (tmpPacket)
|
||||||
{
|
{
|
||||||
//cerr << "IN:" << endl << tmpPacket->ToString() << endl;
|
//cerr << "IN:" << endl << tmpPacket->ToString() << endl;
|
||||||
buf.receivedPackets.push_back(tmpPacket);
|
if (asn_check_constraints(&asn_DEF_PokerTHMessage, tmpPacket->GetMsg(), NULL, NULL) == 0)
|
||||||
|
buf.receivedPackets.push_back(tmpPacket);
|
||||||
|
else
|
||||||
|
LOG_ERROR("Invalid packet: " << endl << tmpPacket->ToString());
|
||||||
}
|
}
|
||||||
else
|
else
|
||||||
dataAvailable = false;
|
dataAvailable = false;
|
||||||
|
|||||||
Reference in New Issue
Block a user