summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorIvan Kohler <ivan@freeside.biz>2014-01-05 01:43:50 -0800
committerIvan Kohler <ivan@freeside.biz>2014-01-05 01:43:50 -0800
commit98b36380fd09ade523de7046e0c67e3e9b35de03 (patch)
tree9a3af8e1b10b7b0e05c0f6a05884d3fb2e496ce0
parente70335c54c0e4dd39485f1c711c1a7cc037df695 (diff)
fix login with an existing session cookie, RT#25533
-rwxr-xr-xfs_selfservice/FS-SelfService/cgi/selfservice.cgi7
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