But Im getting no ware with it.
Is this right, or is there a better way please.
If it were the right way, it would work, wouldn't it. So, clearly it is not.
Look at the strcmp() function, if MsgField is NULL terminated. If it isn't, make it NULL terminated.