Zsh Mailing List Archive
Messages sorted by:
Reverse Date,
Date,
Thread,
Author
Re: PATCH: fix a memory leak and potential heap corruption when rotating lines
- X-seq: zsh-workers 44875
- From: Mikael Magnusson <mikachu@xxxxxxxxx>
- To: Roman Perepelitsa <roman.perepelitsa@xxxxxxxxx>
- Subject: Re: PATCH: fix a memory leak and potential heap corruption when rotating lines
- Date: Mon, 28 Oct 2019 12:50:43 +0100
- Cc: Zsh hackers list <zsh-workers@xxxxxxx>
- Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=4oq6+paFGHzZCteA1yV171rooE6xuvbwF+JKmDCrjvY=; b=cDRhm0+1ER+FalneA4aMlrdvu99BH6wBLEwhKafmubz2Ap95xCuR+NQHZmeCeRdv67 j+EHCiMQxn160IlIIfdo/k1Kagou+qujKp8bEQwUGCXCP++q2pR2YM+uXOIDeivHO14W 0w00fYbnt1cY3EqxRQDpmhCMY92tG3HhRMpw2F7Dy8wm+KRL3HJ1xobI/aUA8sdgBtiV N4b/imrVBmAO6e8/Vkbb7p+Y8AhoSs5yTJ61+vw/oNhF0m57WYoxYLk0gkwb6z9oNsgk FvfuHpFpY3oGhYE8Tobr8nemy9l+JmbkAZOiF4jFGPKRD5/6V0opoFU+Rc6fB6ZsQOfO fyWg==
- In-reply-to: <CAN=4vMou-FFjD5J0kxs6OLZq-GkUGztOUY-sOt3kL2uXQ_0n2Q@mail.gmail.com>
- List-help: <mailto:zsh-workers-help@zsh.org>
- List-id: Zsh Workers List <zsh-workers.zsh.org>
- List-post: <mailto:zsh-workers@zsh.org>
- List-unsubscribe: <mailto:zsh-workers-unsubscribe@zsh.org>
- Mailing-list: contact zsh-workers-help@xxxxxxx; run by ezmlm
- References: <CAN=4vMou-FFjD5J0kxs6OLZq-GkUGztOUY-sOt3kL2uXQ_0n2Q@mail.gmail.com>
On 10/28/19, Roman Perepelitsa <roman.perepelitsa@xxxxxxxxx> wrote:
> I've bumped into this code accidentally while trying to do something
> unrelated. It looks wrong to me although there is a good chance I'm
> misreading it. I'm attaching a patch that either fixes two bugs,
> introduces two new bugs, or does something in between. I'll appreciate
> if someone more experienced can take a look at it.
>
> All test pass with this patch and my zsh seems to work fine. I haven't
> verified that this code actually runs either during testing or when
> I'm using zsh normally.
Did you accidentally sent a reversed patch? That line has been --olnct
since the initial cvs revision.
--
Mikael Magnusson
Messages sorted by:
Reverse Date,
Date,
Thread,
Author