[PATCH] Fixing Bug 10720 - error: Unable to convert first SID
Stefan (metze) Metzmacher
metze at samba.org
Tue Dec 2 00:48:20 MST 2014
Am 02.12.2014 um 04:35 schrieb Andrew Bartlett:
> On Mon, 2014-12-01 at 08:14 +0100, Stefan (metze) Metzmacher wrote:
>> Hi Garming,
>>> On 26/11/14 16:11, Garming Sam wrote:
>>>> The first two patches improves our current testing to correctly
>>>> identify the error. They simply build upon the existing wbinfo_sid2xid
>>>> tests and are currently in auto-build.
>>>> The other three patches fix the actual issue. The first plumbs a
>>>> pointer to struct unixid instead of uid/gid. The second fixes the
>>>> return of the id type and the last fixes the ABI - which with
>>>> renaming, we can also fix another bug
>>>> (https://bugzilla.samba.org/show_bug.cgi?id=10355) where it has naming
>>>> conflicts with Brookhaven Protein DataBank libpdb.
>>> The first two patches are now in master, with a minor change to flush
>>> the cache for the other tests.
>>> The other three only appear to have failed the duplicate symbol check so
>>> I've reattached the patches with the build scripts slightly modified so
>>> they don't depend on pdb directly.
>> Would it be possible to support ID_TYPE_BOTH in
>> pdb_wbc_sam_id_to_sid() too?
> This isn't required, you don't use that as an input type, only have it
> updated on output, and the underlying wbclient library doesn't support
> this with the required wb* calls. Additionally, I think we should just
> drop this module, it only existed for Likewise winbindd replacements,
> and they don't work with modern Samba anyway.
>> Otherwise it looks good to me.
Ok, than push it with my review!
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 181 bytes
Desc: OpenPGP digital signature
More information about the samba-technical