Re: [Nano-devel] [PATCH] browser: repalce newpath by path and present_path

[Benno Schulenberg]

> On Tue, Jul 5, 2016 at 11:28 PM, Rishabh Dave <address@hidden> wrote:
> > Attached patch replaces all occurrences of newpath in do_browser().

Sorry.  Have had no time.  Busy.

But from a quick look:

-               free(newpath);
-               newpath = NULL;
+               free(path);
+               path = NULL;

Are you sure you want to make path null?  Don't you want to
restore it from present_path?  Have you tested this route
through the code?  You know: --operatingdir, ^R ^T, try to
go outside, ^L.

Benno

-- 
http://www.fastmail.com - Choose from over 50 domains or use your own