]> git.koha-community.org Git - koha.git/commit
Bug 33339: Prevent Formula Injection (CSV Injection) in CSV files
authorKyle M Hall <kyle@bywatersolutions.com>
Thu, 27 Jul 2023 16:30:54 +0000 (12:30 -0400)
committerFridolin Somers <fridolin.somers@biblibre.com>
Tue, 5 Nov 2024 14:43:13 +0000 (15:43 +0100)
commitc9ffc538de3cb4972d126407d1d251b2090ed8c8
treef538e18a105132967fead055e44d6e7f10d0f851
parent86bb8a3dc9af76827cdfa670af845eb0c052dc6e
Bug 33339: Prevent Formula Injection (CSV Injection) in CSV files

The system is vulnerable to Formula Injection attacks as the data
stored within the database and exported as CSV/Excel is not being
sanitized or validated against implanted formula payloads

This patch modifies all uses of Text::CSV and derived classes to pass
the "formula" parameter with value of "empty" which replaces formulas
by empty string.

Test Plan:
1) Apply this patch
2) For guided_reports.pl, attempt to export CSV where you've set a column to a formula somehow
   ( such as "=1+3" )
3) Export that CSV file
4) Note the formula has not been exported
5) Repeat this plan for the remaining scripts that export CSV files
   where users can define the outputted data

Signed-off-by: Magnus Enger <magnus@libriotech.no>
Fixed two conflicts. I have tested that this works as advertised on:
- Reports (Download > Comma separated text (.csv)) [Text::CSV::Encoded]
- Circulation > Overdues > Download file of all overdues [Text::CSV_XS]
- misc/export_borrowers.pl [Text::CSV]
This covers all modules used, and both GUI and command line.

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
[EDIT] Change none to empty in the commit message ! None is the default,
doing nothing. Empty clears the formulas.
Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
(cherry picked from commit e588f810a521065bf46dc037edeac5c83d537714)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
20 files changed:
C4/Acquisition.pm
C4/ImportExportFramework.pm
C4/Labels/Label.pm
C4/Record.pm
Koha/Patrons/Import.pm
admin/aqplan.pl
circ/overdue.pl
labels/label-create-csv.pl
misc/cronjobs/overdue_notices.pl
misc/cronjobs/runreport.pl
misc/export_borrowers.pl
misc/migration_tools/import_lexile.pl
reports/cash_register_stats.pl
reports/guided_reports.pl
reports/itemslost.pl
serials/lateissues-export.pl
t/db_dependent/Record/marcrecord2csv.t
tools/import_borrowers.pl
tools/inventory.pl
tools/viewlog.pl