Commit 8c96e74e authored by kiko%async.com.br's avatar kiko%async.com.br

Fix for bug 252378: Remove $COOKIE from attachment.cgi. r=bugreport,

a=justdave.
parent 4e17422c
...@@ -870,7 +870,7 @@ sub insert ...@@ -870,7 +870,7 @@ sub insert
$comment = Text::Wrap::wrap('', '', $comment); $comment = Text::Wrap::wrap('', '', $comment);
AppendComment($::FORM{'bugid'}, AppendComment($::FORM{'bugid'},
$::COOKIE{"Bugzilla_login"}, Bugzilla->user->login,
$comment, $comment,
$isprivate); $isprivate);
...@@ -934,7 +934,7 @@ sub insert ...@@ -934,7 +934,7 @@ sub insert
} }
# Define the variables and functions that will be passed to the UI template. # Define the variables and functions that will be passed to the UI template.
$vars->{'mailrecipients'} = { 'changer' => $::COOKIE{'Bugzilla_login'}, $vars->{'mailrecipients'} = { 'changer' => Bugzilla->user->login,
'owner' => $owner }; 'owner' => $owner };
my $bugid = $::FORM{'bugid'}; my $bugid = $::FORM{'bugid'};
detaint_natural($bugid); # don't bother with error condition, we know it'll work detaint_natural($bugid); # don't bother with error condition, we know it'll work
...@@ -1156,7 +1156,7 @@ sub update ...@@ -1156,7 +1156,7 @@ sub update
} }
# Define the variables and functions that will be passed to the UI template. # Define the variables and functions that will be passed to the UI template.
$vars->{'mailrecipients'} = { 'changer' => $::COOKIE{'Bugzilla_login'} }; $vars->{'mailrecipients'} = { 'changer' => Bugzilla->user->login };
$vars->{'attachid'} = $::FORM{'id'}; $vars->{'attachid'} = $::FORM{'id'};
$vars->{'bugid'} = $bugid; $vars->{'bugid'} = $bugid;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment