Okay, here are my results after retesting.

It appears that CVS and the modified version of CompareVerString return the correct value, and the original CompareVerString and CompareVersions are returning the incorrect value. (CompareVersions inputs are reversed so I flipped the inputs below...)

 Code:
? cvs("10.1.183.7","10.1.183.10") ; returns 1
? cvs("10.1.183.10","10.1.183.7") ; returns -1
?
? compareverstringMOD("10.1.183.7","10.1.183.10")  ; returns 1
? compareverstringMOD("10.1.183.10","10.1.183.7")  ; returns -1
?
? compareverstring("10.1.183.7","10.1.183.10") ; returns -1
? compareverstring("10.1.183.10","10.1.183.7") ; returns 1
?
? compareversions("10.1.183.10","10.1.183.7") ; returns -1
? compareversions("10.1.183.7","10.1.183.10") ; returns 1