diff options
author | Ivan Kohler <ivan@freeside.biz> | 2014-01-05 01:43:50 -0800 |
---|---|---|
committer | Ivan Kohler <ivan@freeside.biz> | 2014-01-05 01:43:50 -0800 |
commit | 98b36380fd09ade523de7046e0c67e3e9b35de03 (patch) | |
tree | 9a3af8e1b10b7b0e05c0f6a05884d3fb2e496ce0 | |
parent | e70335c54c0e4dd39485f1c711c1a7cc037df695 (diff) |
fix login with an existing session cookie, RT#25533
-rwxr-xr-x | fs_selfservice/FS-SelfService/cgi/selfservice.cgi | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/fs_selfservice/FS-SelfService/cgi/selfservice.cgi b/fs_selfservice/FS-SelfService/cgi/selfservice.cgi index 13723118c..f84f5d633 100755 --- a/fs_selfservice/FS-SelfService/cgi/selfservice.cgi +++ b/fs_selfservice/FS-SelfService/cgi/selfservice.cgi @@ -105,7 +105,7 @@ unless ( $nologin_actions{$action} ) { my %cookies = CGI::Cookie->fetch; - my $login_rv; + my $login_rv = {}; if ( exists($cookies{'session'}) ) { @@ -178,10 +178,7 @@ unless ( $nologin_actions{$action} ) { } # else session_id ne 'login' - } else { - # there is no session cookie - $login_rv = {}; - } + } # else there is no session cookie if ( !$session_id ) { # XXX why are we getting agentnum from a CGI param? surely it should |