Re: Permission error when getting cwd

2012-02-06 Thread Ben Boeckel
On Mon, Feb 06, 2012 at 09:16:47 +, Nicholas Marriott wrote: > I'm pretty sure that should be "root != NULL" ;-). Oops, yes. I had originally had it down after the complete_path label and returning (s->cwd) itself, but falling through seems like the better option. Guess I didn't swap the compa

Re: Permission error when getting cwd

2012-02-06 Thread Nicholas Marriott
Hi I'm pretty sure that should be "root != NULL" ;-). On Sun, Feb 05, 2012 at 11:49:39PM -0500, Ben Boeckel wrote: > If you have a process running in a window and create a new window with > it focused, a NULL dereference can occur if the process is not owned by > the user (e.g., a root shell).

Permission error when getting cwd

2012-02-05 Thread Ben Boeckel
If you have a process running in a window and create a new window with it focused, a NULL dereference can occur if the process is not owned by the user (e.g., a root shell). Patch attached. --Ben >From c9d364132951dcba3a46af008d00edbb0ac6355d Mon Sep 17 00:00:00 2001 From: Ben Boeckel Date: Sun,