Bugzilla – Attachment 40045 Details for
Bug 14360
XSS Injection point
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
[SIGNED OFF] Bug 14360: Unescaped variable causes alert
SIGNED-OFF-Bug-14360-Unescaped-variable-causes-ale.patch (text/plain), 3.62 KB, created by
Katrin Fischer
on 2015-06-09 21:38:05 UTC
(
hide
)
Description:
[SIGNED OFF] Bug 14360: Unescaped variable causes alert
Filename:
MIME Type:
Creator:
Katrin Fischer
Created:
2015-06-09 21:38:05 UTC
Size:
3.62 KB
patch
obsolete
>From b28250133d5a1827e635f08c1eb3818fb0b38abd Mon Sep 17 00:00:00 2001 >From: Aleisha <aleishaamohia@hotmail.com> >Date: Mon, 8 Jun 2015 02:30:23 +0000 >Subject: [PATCH] [SIGNED OFF] Bug 14360: Unescaped variable causes alert > >Adding |html to [% resultsperpage %] to escape the variable and get rid of the alert. > >To test: > >1) Go to URL such as ... /cgi-bin/koha/opac-authorities-home.pl?op=do_search&resultsperpage=1%22%3E%3Cscript%3Ealert%28document.cookie%29%3C/script%3E >2) Notice pop-up box with alert >3) Apply patch, refresh page >4) Notice alert is gone > >Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> >--- > .../bootstrap/en/modules/opac-authoritiessearchresultlist.tt | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > >diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-authoritiessearchresultlist.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-authoritiessearchresultlist.tt >index 00d657e..d4a264b 100644 >--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-authoritiessearchresultlist.tt >+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-authoritiessearchresultlist.tt >@@ -6,7 +6,7 @@ > <ul> > [% IF ( displayprev ) %] > <li> >- <a href="opac-authorities-home.pl?startfrom=[% startfromprev %]&[% FOREACH searchdat IN searchdata %][% searchdat.term %]=[% searchdat.val |url %]&[% END %]resultsperpage=[% resultsperpage %]&type=opac&op=do_search&authtypecode=[% authtypecode %]&orderby=[% orderby|html %]"> >+ <a href="opac-authorities-home.pl?startfrom=[% startfromprev %]&[% FOREACH searchdat IN searchdata %][% searchdat.term %]=[% searchdat.val |url %]&[% END %]resultsperpage=[% resultsperpage |html %]&type=opac&op=do_search&authtypecode=[% authtypecode %]&orderby=[% orderby|html %]"> > « > </a> > </li> >@@ -15,12 +15,12 @@ > [% IF ( number.highlight ) %] > <li class="active"><a href="#">[% number.number %]</a></li> > [% ELSE %] >- <li><a href="opac-authorities-home.pl?startfrom=[% number.startfrom %]&[% FOREACH searchdat IN number.searchdata %][% searchdat.term %]=[% searchdat.val |url %]&[% END %]resultsperpage=[% resultsperpage %]&type=opac&op=do_search&authtypecode=[% authtypecode %]&orderby=[% orderby|html %]">[% number.number %]</a></li> >+ <li><a href="opac-authorities-home.pl?startfrom=[% number.startfrom %]&[% FOREACH searchdat IN number.searchdata %][% searchdat.term %]=[% searchdat.val |url %]&[% END %]resultsperpage=[% resultsperpage |html %]&type=opac&op=do_search&authtypecode=[% authtypecode %]&orderby=[% orderby|html %]">[% number.number %]</a></li> > [% END %] > [% END %] > [% IF ( displaynext ) %] > <li> >- <a href="opac-authorities-home.pl?startfrom=[% startfromnext %]&[% FOREACH searchdat IN searchdata %][% searchdat.term %]=[% searchdat.val |url %]&[% END %]&resultsperpage=[% resultsperpage %]&type=opac&op=do_search&authtypecode=[% authtypecode %]&orderby=[% orderby|html %]">»</a> >+ <a href="opac-authorities-home.pl?startfrom=[% startfromnext %]&[% FOREACH searchdat IN searchdata %][% searchdat.term %]=[% searchdat.val |url %]&[% END %]&resultsperpage=[% resultsperpage |html %]&type=opac&op=do_search&authtypecode=[% authtypecode %]&orderby=[% orderby|html %]">»</a> > </li> > [% END %] > </ul> >-- >1.9.1
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 14360
:
39981
|
40009
|
40045
|
40046
|
40066
|
40067