On Mon, May 01, 2017 at 01:36:51PM +0100, Nicholas Marriott wrote:
>
> looks good, ok nicm
Anyone willing to commit?
> On Mon, May 01, 2017 at 10:35:59AM +0200, Tobias Stoeckmann wrote:
> > Hi,
> >
> > On Mon, May 01, 2017 at 09:15:45AM +0200, Anton Lindqvist wrote:
> > > While freeing tag entries, make sure to free the copied strings.
> >
> > this patch looks good to me.
> >
> > Have you reported this to the upstream less maintainers, as well?
> > The original less and the forked one from Garrett D'Amore, which we
> > use as foundation, have the same issue in them.
> >
> >
> > Tobias
> >
> > > Index: tags.c
> > > ===================================================================
> > > RCS file: /cvs/src/usr.bin/less/tags.c,v
> > > retrieving revision 1.18
> > > diff -u -p -r1.18 tags.c
> > > --- tags.c 17 Sep 2016 15:06:41 -0000 1.18
> > > +++ tags.c 30 Apr 2017 18:13:24 -0000
> > > @@ -77,6 +77,8 @@ cleantags(void)
> > > */
> > > while ((tp = taglist.tl_first) != TAG_END) {
> > > TAG_RM(tp);
> > > + free(tp->tag_file);
> > > + free(tp->tag_pattern);
> > > free(tp);
> > > }
> > > curtag = NULL;
> > >
> >
>
