I think we don't need this method. This is because same() is only used in verifyPassword() to compare two hashes which contain ASCII characters only. We can simply put the for-loop comparison at the end of verifyPassword(). strlen() is also sufficient.