Summary: | Error in regexp causes crash | ||
---|---|---|---|
Product: | [Applications] konqueror | Reporter: | ck |
Component: | kjs | Assignee: | Konqueror Developers <konq-bugs> |
Status: | CLOSED FIXED | ||
Severity: | crash | ||
Priority: | NOR | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | Compiled Sources | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: |
Description
ck
2002-04-08 06:56:33 UTC
On 8 Apr 2002 ck@heise.de wrote: > The following HTML page crashes Konqueror reliably. This seems to be due to the erroneous regexp (no character before the + sign). I can confirm the crash on a system without PCRE support compiled in. An other one with that extended support it hangs. Both equally bad of course. > While the Regexp is incorrect it should still not crash the browser. Added as a testcase ("foo+bar".replace(/+/g"%2B");) to the JS test suite. Will look into fixing it later. Harri. On 8 Apr 2002 ck@heise.de wrote: > alert(s1.replace(/+/g"%2B")); Fixed the crash together with another replace() bug for KDE 3.0.1. We still don't accept '+' as that character though but as an invalid regexp. Netscape 4.7 treats it differently. Thanks for the report Harri. |