Of course it should be bool. I'm glad that the OP at least made it a const char *password though, since that's also a must-have. Always const input pointers. A

Of course it should be bool. I'm glad that the OP at least made it a const char *password though, since that's also a must-have. Always const input pointers. And local temporaries. And everything else that you can const, except scalar arguments pretty much.