On Thu 18.Mar'10 at 19:38:37 +0300, Alexey I. Froloff wrote:

> > >           } else {
> > >                   dir = tmp;
> dir points to tmp.
> 
> > >           }
> > > +         wfree(tmp);
> dir is freed too in "else" case.
> 
> > >           prefix = wstrdup(pos + 1);
> > >           ScanFiles(dir, prefix, (unsigned)-1, 0, variants);
> > >           wfree(dir);
> And now it's freed for the second time in "else" case.

Right you are! Thanks for the review!

Unapplied.


-- 
To unsubscribe, send mail to [email protected].

Reply via email to