Bug 19108: Fix Stored XSS in fieldmapping.pl
authorAmit Gupta <amit.gupta@informaticsglobal.com>
Tue, 15 Aug 2017 08:40:43 +0000 (14:10 +0530)
committerJonathan Druart <jonathan.druart@bugs.koha-community.org>
Fri, 29 Sep 2017 15:20:50 +0000 (12:20 -0300)
To Test
1. Hit the page /cgi-bin/koha/admin/fieldmapping.pl
2. Add a text in the field Field name that contains js
3. Save the page.
4. Notice js is execute
5. Apply patch and reload, the js is escaped

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
koha-tmpl/intranet-tmpl/prog/en/modules/admin/fieldmapping.tt

index d182750..238555a 100644 (file)
@@ -69,7 +69,7 @@ $(document).ready(function() {
                                                                        </tr>
                                                                        [% FOREACH field IN fields %]
                                                                        <tr>
-                                                                               <td>[% field.field %]</td>
+                                        <td>[% field.field |html %]</td>
                                                                                <td>[% field.fieldcode %]</td>
                                                                                <td>[% field.subfieldcode %]</td>
                                         <td><a class="btn btn-default btn-xs" href="?op=delete&amp;id=[% field.id %]&amp;framework=[% field.frameworkcode %]"><i class="fa fa-trash"></i> Delete</a></td>