Also replace the variable name Result with Error Since UnpackPacket() returns zero on success. This red a bit funny: if(!Result) { // do something with the result } It now is: if(Error) continue; // do something with the result