GNU bug report logs - #25391
24.5; ffap-guesser "stack overflow in regexp matcher" error may crash emacs.

Previous Next

Package: emacs;

Reported by: Thierry Volpiatto <thierry.volpiatto <at> gmail.com>

Date: Sun, 8 Jan 2017 07:32:01 UTC

Severity: minor

Tags: fixed, patch

Found in version 24.5

Fixed in version 26.1

Done: Noam Postavsky <npostavs <at> users.sourceforge.net>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: npostavs <at> users.sourceforge.net
To: Thierry Volpiatto <thierry.volpiatto <at> gmail.com>
Cc: 25391 <at> debbugs.gnu.org
Subject: bug#25391: 24.5; ffap-guesser "stack overflow in regexp matcher" error may crash emacs.
Date: Fri, 13 Jan 2017 21:25:54 -0500
[Message part 1 (text/plain, inline)]
Thierry Volpiatto <thierry.volpiatto <at> gmail.com> writes:

> npostavs <at> users.sourceforge.net writes:
>
>> Yes, although that's also true before my patch.  So `ffap-gopher-regexp'
>> never actually supported being changed.  I'm not sure if it's a good
>> idea to add support for that.
>
> I think it is, gopher is not widely used AFAIK, and allowing disabling
> it would avoid needless search in most cases, thus it make it consistent
> with other ffap* fns that can be disabled; I have a lot of complains at
> helm with ffap annoyances and having more flexibility on this is interesting.

Oh, yes, if the other regexps can be disabled like that, we should make it
consistent.  I've added a note about it to the docstring and toplevel
comments; I'll push to master in a couple of days.

[v4-0001-Improve-ffap-gopher-at-point-handling-of-long-lin.patch (text/plain, attachment)]

This bug report was last modified 8 years and 130 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.