Patch: An updated version of my script idmapping backend for winbind.

Richard Sharpe realrichardsharpe at gmail.com
Mon Dec 22 13:23:49 MST 2014


On Mon, Dec 22, 2014 at 10:15 AM, Jeremy Allison <jra at samba.org> wrote:
> On Thu, Nov 27, 2014 at 10:32:27AM -0800, Richard Sharpe wrote:
>> Hi folks,
>>
>> Attached is an updated version that:
>>
>> 1. Cleans up some unused variables that only the Samba 3 build seemed to find,
>> 2. Adds a man page.
>>
>> Review and push or feedback appreciated.
>> +
>> +     p = popen(cmd, "r");
>
> Use sys_popen() instead (I know idmap_tdb2.c
> doesn't. It should :-). sys_popen does not
> search PATH (safer).

Thanks. I have been talking to Michael about the other feedback so I
will incorporate this as well.

-- 
Regards,
Richard Sharpe
(何以解憂?唯有杜康。--曹操)


More information about the samba-technical mailing list