On 05/13/2014 11:45 PM, Bernhard Voelker wrote:
> On 05/13/2014 05:14 PM, Pádraig Brady wrote:
>> From 9c1bad82852cec8403ead49f12f53280c468a2cf Mon Sep 17 00:00:00 2001
>> From: =?UTF-8?q?P=C3=A1draig=20Brady?= <[email protected]>
>> Date: Tue, 13 May 2014 15:56:34 +0100
>> Subject: [PATCH] chroot: don't chdir() if not changing root
>>
>> This allows chroot to use used as a light weight tool
>> to change user identification for a command,
>> while not changing the current working directory.
>>
>> * src/chroot.c (main): If the same root is specified. i.e. '/'
>> then don't change the current working directory, and avoid the
>> overhead of the other redundant calls.
> 
> Looks good, but deserves a NEWS entry, doesn't it?

Yep as a "Change in behavior".
Also needs tests.

Just popping it out as an RFC at this stage.

Thanks for the opinion.

cheers,
Pádraig.

Reply via email to