Attachment 161888 Details for Bug 36034 – Bug 36034: fix capture of return values from checkpw

[patch] Bug 36034: fix capture of return values from checkpw

Bug-36034-fix-capture-of-return-values-from-checkp.patch (text/plain), 2.06 KB, created by Andreas Jonsson on 2024-02-08 11:20:07 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Andreas Jonsson

Created: 2024-02-08 11:20:07 UTC

Size: 2.06 KB

patch

obsolete

>From 41be1d76a8d2789ecce1e206153f30a7409cc63b Mon Sep 17 00:00:00 2001
>From: Andreas Jonsson <andreas.jonsson@kreablo.se>
>Date: Thu, 8 Feb 2024 11:57:03 +0100
>Subject: [PATCH] Bug 36034: fix capture of return values from checkpw
>
>Adapt code to the change of return value type of checkpw
>introduced in bug 34893
>---
> C4/Auth.pm | 9 ++++++---
> 1 file changed, 6 insertions(+), 3 deletions(-)
>
>diff --git a/C4/Auth.pm b/C4/Auth.pm
>index 6940a7ca25..dc18314d10 100644
>--- a/C4/Auth.pm
>+++ b/C4/Auth.pm
>@@ -1031,7 +1031,8 @@ sub checkauth {
>             unless ($shibSuccess) {
>                 if ( $cas && $query->param('ticket') ) {
>                     my $retuserid;
>-                    ( $return, $cardnumber, $retuserid, $cas_ticket ) =
>+                    my $patron;
>+                    ( $return, $cardnumber, $retuserid, $patron, $cas_ticket ) =
>                       checkpw( $userid, $password, $query, $type );
>                     $userid = $retuserid;
>                     $info{'invalidCasLogin'} = 1 unless ($return);
>@@ -1099,8 +1100,9 @@ sub checkauth {
>                             && $q_userid eq C4::Context->preference('AutoSelfCheckID') )
>                       )
>                     {
>+                        my $patron;
> 
>-                        ( $return, $cardnumber, $retuserid, $cas_ticket ) =
>+                        ( $return, $cardnumber, $retuserid, $patron, $cas_ticket ) =
>                           checkpw( $q_userid, $password, $query, $type );
>                         $userid = $retuserid if ($retuserid);
>                         $info{'invalid_username_or_password'} = 1 unless ($return);
>@@ -1593,7 +1595,8 @@ sub check_api_auth {
>                 return ( "failed", undef, undef );
>             }
>             my $newuserid;
>-            ( $return, $cardnumber, $newuserid, $cas_ticket ) = checkpw( $userid, $password, $query );
>+            my $patron;
>+            ( $return, $cardnumber, $newuserid, $patron, $cas_ticket ) = checkpw( $userid, $password, $query );
>         }
> 
>         if ( $return and haspermission( $userid, $flagsrequired ) ) {
>-- 
>2.39.2

Actions: View | Diff | Splinter Review

Attachments on bug 36034: 161888 | 161926 | 162130 | 162131 | 162132 | 162529