Re: [PATCH] do-lowercase-version for non-uppercase chars

[Chet Ramey]

On 11/30/23 12:20 AM, Grisha Levit wrote:
> On Tue, May 30, 2023, 10:43 Chet Ramey <chet.ramey@case.edu> wrote:
> > On 5/27/23 3:04 PM, Grisha Levit wrote:
> > > If do-lowercase-version is bound to a key that is either not an uppercase
> > > character or is a character that is its own lowercase version, then
> > > readline will either loop forever with no way to interrupt (if the compiler
> > > optimized out the recursion) or will exceed the stack depth and segfault.
> >
> > This sounds more like a "my arm hurts when I do that" kind of situation,
> > but readline still shouldn't crash.
>
> One more for completeness:
>
>      $ bind '"ab": "c"' '"a": do-lowercase-version'
>      $ ax
>      Segmentation fault

Thanks for the report and patch.

Chet

--
``The lyf so short, the craft so long to lerne.'' - Chaucer
                 ``Ars longa, vita brevis'' - Hippocrates
Chet Ramey, UTech, CWRU    chet@case.edu    http://tiswww.cwru.edu/~chet/

OpenPGP_signature.asc
Description: OpenPGP digital signature