It is possible to enter some special search/replace pattern and get an infinite loop on the text view. For example, if we use "[" as search pattern and "[[" for a replacement then we're in trouble: loop will never finish and script will be killed by timeout (if any). So, I've reworked problematic part of code to guarantee single pass of the string, it protects from infinite loop.

Comments

dmitriy.trt’s picture

Title: Infinite loop in text replace pattern component » Infinite loop in text pattern replace component
johannesjvorstermans’s picture

Works well for me. Just had to remove the last /** from the patch.

Be good to get this into the dev version asap.

jarodms’s picture

Status: Needs review » Fixed

committed to dev.

Status: Fixed » Closed (fixed)

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