Spreadsheets are vulnerable to hidden formulas in CSV files. If webform data is submitted by untrusted users, and the submissions are downloaded in CSV format, maliciously-formatted submission data may be used to create a security vulnerability. The Drupal security team has concluded that this vulnerability does not rest in webform, but rather in the spreadsheets. Nonetheless, a warning about opening downloaded submissions in CSV format with spreadsheets should be added to the webform download page.

For more information, see the headings "Formula Injection" and "Exploiting trust relationships" here: http://www.contextis.com/resources/blog/comma-separated-vulnerabilities/.

Originally reported by zread.

Solution: Add the following warning to the download page when delimited text format is selected:

Warning: Opening delimited text files with spreadsheet applications may expose you to formula injection or other security vulnerabilities. When the submissions contain data from untrusted users and the downloaded file will be used with spreadsheets, choose Microsoft Excel format. Search "spreadsheet formula injection" for more information.

CommentFileSizeAuthor
#3 webform-csv_warning-2788591-2.patch2.83 KBDanChadwick
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

DanChadwick created an issue. See original summary.

  • DanChadwick committed d534392 on 7.x-4.x
    Issue #2788591: Added warning about opening CSV files with spreadsheets.
    
DanChadwick’s picture

Status: Active » Fixed
FileSize
2.83 KB

Committed to 7.x-4.x

DanChadwick’s picture

Version: 7.x-4.x-dev » 8.x-4.x-dev
Category: Feature request » Task
Status: Fixed » Patch (to be ported)
fenstrat’s picture

Version: 8.x-4.x-dev » 7.x-4.x-dev
Category: Task » Feature request
Status: Patch (to be ported) » Fixed

Closing to clear out the old Webform 8.x-4.x branch. See #2827845: [roadmap] YAML Form 8.x-1.x to Webform 8.x-5.x.

  • DanChadwick committed d534392 on 8.x-5.x
    Issue #2788591: Added warning about opening CSV files with spreadsheets.
    

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.