Closed Bug 596180 Opened 10 years ago Closed 9 years ago
Kit's loop limits on the adoption agency algorithm
WebKit's new adoption agency algorithm implementation limits the inner and outer loop to 10 iterations per loop entry. We should probably have limits there, too, and lacking data on what the best limits would be, it makes sense to use the same arbitrary limits as WebKit.
Assignee: nobody → hsivonen
Status: NEW → ASSIGNED
Attachment #475030 - Flags: review?(jonas)
Comment on attachment 475030 [details] [diff] [review] Limit the loops to 10 iterations per loop entry rs=me, but we need to keep pushing for the ability to align with spec here.
Attachment #475030 - Flags: review?(jonas) → review+
Nominating as a blocker, since not fixing this exposes a DoS vector for making the parser do an amount of work that is disproportionate to the amount of input text.
blocking2.0: --- → ?
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b8
You need to log in before you can comment on or make changes to this bug.