[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#13802: stack overflow in mm-add-meta-html-tag
From: |
Thien-Thi Nguyen |
Subject: |
bug#13802: stack overflow in mm-add-meta-html-tag |
Date: |
Sun, 24 Feb 2013 10:17:53 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.1 (gnu/linux) |
I see a "Stack overflow in regexp matcher" error traceable back to
lisp/gnus/mm-decode.el func ‘mm-add-meta-html-tag’ fragment:
(re-search-forward "\
<meta\\s-+http-equiv=[\"']?content-type[\"']?\\s-+content=[\"']\
text/\\(\\sw+\\)\\(?:\;\\s-*charset=\\(.+\\)\\)?[\"'][^>]*>" nil t)
To allow the user (not me) to continue, i kludged the form to be:
(ignore-errors
(re-search-forward "..." nil t))
that is, wrapping w/ ‘ignore-errors’. Is there a better solution?
One idea (untested) is to replace the ".+" (used to match the charset)
with a more specific pattern. Perhaps "[^<>]+" or "\\sw+"?
Thinking more systematically, maybe Emacs should add a condition
‘stack-overflow/regexp’ (or something like that) such that code can
‘condition-case’ for it and try a fallback path.
--
Thien-Thi Nguyen ..................................... GPG key: 4C807502
. NB: ttn at glug dot org is not me .
. (and has not been since 2007 or so) .
. ACCEPT NO SUBSTITUTES .
........... please send technical questions to mailing lists ...........
pgpCK49uqBbG5.pgp
Description: PGP signature
- bug#13802: stack overflow in mm-add-meta-html-tag,
Thien-Thi Nguyen <=