[Dr. McKay]

Originally Posted by KyleS argc should be < 1 instead of != 1 as people tend to use spaces with targets. You store and set the CmdReplySource, but you never set the value back to what it was after you print your message. You should consider using GetClientSerial instead of GetClientUserID to prevent potential duplicate checks on the same client. You should also consider checking the returns from a few GetClient* Functions (GetClientIP). None of this seems lethal, though.

Thanks, fixed.