bug-readline
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Bug-readline] [PATCH] Fix sv_isrchterm (set isearch-terminators) to


From: Sergio Durigan Junior
Subject: Re: [Bug-readline] [PATCH] Fix sv_isrchterm (set isearch-terminators) to check for NUL on the value string
Date: Mon, 18 May 2015 01:22:09 -0400
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.4 (gnu/linux)

On Sunday, May 17 2015, Chet Ramey wrote:

> On 5/16/15 11:26 PM, Sergio Durigan Junior wrote:
>
>> However, it is not checking to see if v[end] is not NUL, which leads to
>> a segmentation fault depending on what is on the stack when v gets
>> allocated.  FWIW, I managed to reproduce the failure using the inputrc
>> file attached to this message.
>> 
>> The patch to fix this issue is trivial.  It just checks to see if v[end]
>> is not NUL before proceeding with the whitespace check.
>
> Thanks for the report.  You have found one of the oldest bugs in readline:
> that code has existed in more or less its current form for over 15 years.

Hah, neat :-).

>> It seems that you are not using a ChangeLog file anymore, so I am
>> sending only the patch.  Please let me know if I need to send anything
>> else.
>
> Readline development is done along with bash, so the bash changelog (and
> devel branch) contains the readline development history.

Alright, I remember seeing someone commenting this on gdb-patches
indeed.  So, should I send the patch to bug-bash, or is bug-readline
already enough?

Cheers,

-- 
Sergio
GPG key ID: 237A 54B1 0287 28BF 00EF  31F4 D0EB 7628 65FC 5E36
Please send encrypted e-mail if possible
http://sergiodj.net/



reply via email to

[Prev in Thread] Current Thread [Next in Thread]