Page MenuHomePhabricator

Evaluator freeze/timeout on unclosed comment
Closed, ResolvedPublic


Filter evaluator seems to freeze/timeout on filter snippet, inadvertently containing an indented unclosed comment

It can be reproduced by applying filter code as simple as the following (there has to be some whitespace preceding /*)

/* unclosed comment...

on some log entry, e.g.:


Related Gerrit Patches:
mediawiki/extensions/AbuseFilter : masterProperly detect unclosed comments

Event Timeline

Teslaton created this task.May 1 2016, 6:18 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMay 1 2016, 6:18 PM
Daimona triaged this task as Medium priority.Mar 25 2018, 2:22 PM
Daimona moved this task from Backlog to Internal bugs on the AbuseFilter board.

Change 425321 had a related patch set uploaded (by Daimona Eaytoy; owner: Daimona Eaytoy):
[mediawiki/extensions/AbuseFilter@master] Properly detect unclosed comments

Daimona claimed this task.Apr 10 2018, 5:28 PM
Daimona added a subscriber: Daimona.

A bit of explanation on gerrit. Basically we have a while loop which never ends while tokenizing the rules, thus causing the timeout.

Change 425321 merged by jenkins-bot:
[mediawiki/extensions/AbuseFilter@master] Properly detect unclosed comments

Huji closed this task as Resolved.Apr 11 2018, 2:15 AM
Huji removed a project: Patch-For-Review.