View Issue Details

This bug affects 1 person(s).
 4
IDProjectCategoryView StatusLast Update
18940Bug reportsSurvey editingpublic2024-11-27 17:03
Reporterollehar Assigned Toc_schmitz  
PrioritynormalSeverityminor 
Status closedResolutionduplicate 
Product Version6.1.x 
Summary18940: Out of memory if question text is too big
Description

When question text is huge and contains lots of expression script, viewing question overview can consume 1GB of memory or more and also crash the browser.

TagsNo tags attached.
Bug heat4
Complete LimeSurvey version number (& build)master
I will donate to the project if issue is resolvedNo
Browser-
Database type & version-
Server OS (if known)-
Webserver software & version (if known)-
PHP Version-

Relationships

duplicate of 19853 ready for testingtibor.pacalat Data images crashing survey 

Users monitoring this issue

There are no users monitoring this issue.

Activities

ollehar

ollehar

2023-07-05 16:37

administrator   ~75949

Already pushed a fix for 3.x here: https://github.com/LimeSurvey/LimeSurvey/commit/a6afa978fa54802078e4d10f1ba0bfccb23f5679

Missing for master and 5.x

DenisChenu

DenisChenu

2023-07-05 17:39

developer   ~75951

Crash browser because of expression script ?

You have a sample ?

DenisChenu

DenisChenu

2023-07-06 09:54

developer   ~75955

Comments on github

  1. need to adapt according to $config['memory_limit'] (with real one : ini_get('memory_limit'))
  2. Maybe we can check memory_get_usage ?
  3. Need to be done in ExpressionManager class directly
  4. Must show a warning in HTML content not a alert
ollehar

ollehar

2023-07-06 09:56

administrator   ~75956

You have a sample ?

Only customer, protected data sorry. But it was 1,5M characters in that expression. :)

DenisChenu

DenisChenu

2023-07-06 10:02

developer   ~75957

Only customer, protected data sorry. But it was 1,5M characters in that expression. :)

Ow ! I have some user using copy/paste image (then data:image/png) but have only 11 410.
With or without expression ?

Maybe need to minimize before send it to ExpressionManager ?

ollehar

ollehar

2023-07-06 10:14

administrator   ~75960

With or without expression ?

The question text was one huge expression, nothing else.

DenisChenu

DenisChenu

2023-07-06 12:46

developer   ~75968

{ 1,5M characters }

8-o Oh F**

Issue History

Date Modified Username Field Change
2023-07-05 16:36 ollehar New Issue
2023-07-05 16:37 ollehar Priority none => normal
2023-07-05 16:37 ollehar Product Version => 6.1.x
2023-07-05 16:37 ollehar Note Added: 75949
2023-07-05 16:37 ollehar Bug heat 0 => 2
2023-07-05 16:38 ollehar Description Updated
2023-07-05 16:38 ollehar Steps to Reproduce Updated
2023-07-05 16:38 ollehar Description Updated
2023-07-05 17:39 DenisChenu Note Added: 75951
2023-07-05 17:39 DenisChenu Bug heat 2 => 4
2023-07-06 09:54 DenisChenu Note Added: 75955
2023-07-06 09:56 ollehar Note Added: 75956
2023-07-06 10:02 DenisChenu Note Added: 75957
2023-07-06 10:14 ollehar Note Added: 75960
2023-07-06 12:46 DenisChenu Note Added: 75968
2023-07-07 22:08 gabrieljenik Status new => confirmed
2024-11-27 17:03 c_schmitz Relationship added duplicate of 19853
2024-11-27 17:03 c_schmitz Assigned To => c_schmitz
2024-11-27 17:03 c_schmitz Status confirmed => closed
2024-11-27 17:03 c_schmitz Resolution open => duplicate