vbGore Free Online RPG Engine

Would it be a good idea to put an error handler in each sub that recieves packet that overflows the buffer so nothing is left in it?

Overflow is only used to make the buffer's read position higher so it looks like the packet is empty. Its just used on packets that have invalid information, but to cut out that information takes too much CPU usage (which would make it a good DOS packet, which we don't want). So not exactly sure how putting an error check will do much or help - maybe I'm missing something you're saying?