I'm beginning to wonder how many times it'll take me to get the

password check right.  But this time I actually ran it through
the test harness before checking it in.

diff --git a/Sip/MsgType.pm b/Sip/MsgType.pm
index fb42f00..e963562 100644 (file)
--- a/Sip/MsgType.pm
+++ b/Sip/MsgType.pm
@@ -1160,7 +1160,7 @@ sub handle_patron_enable {
        $resp .= add_field(FID_VALID_PATRON_PWD, 'N');
     } else {
        # valid patron
-       if (!defined($patron_pwd) || !$patron->check_password($patron_pwd)) {
+       if (!defined($patron_pwd) || $patron->check_password($patron_pwd)) {
            # Don't enable the patron if there was an invalid password
            $status = $patron->enable;
        }