https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25339

David Roberts <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #104064|0                           |1
        is obsolete|                            |

--- Comment #4 from David Roberts <[email protected]> ---
Created attachment 104850
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=104850&action=edit
Bug 25339: Validate biblionumber on opac-basket.pl and opac-review.pl

This patch runs biblionumber through the int() function before
using it in the application.

Test Plan:
1) Apply patch
2) Go to
/cgi-bin/koha/opac-basket.pl?bib_list=1/abcdefghi'-alert(1)-'j&verbose=1
3) Note that you can't find abcdefghi in the HTML. You only see biblionumber 1.
4) Go to
/cgi-bin/koha/opac-review.pl?biblionumber=1'-alert(1)-'abcdefghi&reviewid=1&review=
5) Note that you can't find abcdefghi in the HTML. You only see biblionumber 1.

Signed-off-by: David Roberts <[email protected]>

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to