Samba-4.0-rc2 samba-tool ntacl sysvolreset
Andrew Bartlett
abartlet at samba.org
Sat Oct 6 22:57:48 MDT 2012
On Sun, 2012-10-07 at 06:40 +0200, Gémes Géza wrote:
> 2012-10-07 02:12 keltezéssel, Andrew Bartlett írta:
> > On Sat, 2012-10-06 at 17:51 +0200, Gémes Géza wrote:
> >> Hi,
> >>
> >> My saga with RC2 continues :-)
> >>
> >> I have successfully joined a new RC2 install to my old beta6-GIT-4631723
> >> domain (from classicupgrade)
> >> The rc2 works fine this time. Except one minor (or not so) problem:
> >> I've copied (the tar-ed version) of the sysvol folder from the old
> >> install (deliberately chose to drop acls at the origin)
> >> then I ran samba-tool ntacl sysvolreset on the RC2 install, which gave:
> >>
> >> set_nt_acl_no_snum: fset_nt_acl returned NT_STATUS_INVALID_OWNER.
> >> ERROR(runtime): uncaught exception - (-1073741734,
> >> 'NT_STATUS_INVALID_OWNER')
> > Can you please try this patch?
> >
> > This is my current proposal to work around this situation - we just
> > don't change the posix owner of the file in this case.
> >
> > (I might evolve this patch to have it manually chown the files to root,
> > but it won't go via this failing codepath)
> >
> > Andrew Bartlett
> >
> Hi,
>
> Unfortunately the ntacl.py (from master, the patch doesn't apply to RC2)
> with the patch applied fails on RC2 with:
> samba-tool ntacl sysvolreset yields:
> ERROR(<type 'exceptions.AttributeError'>): uncaught exception -
> 'NoneType' object has no attribute 'sid_to_id'
> File
> "/usr/local/samba/lib/python2.7/site-packages/samba/netcmd/__init__.py",
> line 168, in _run
> return self.run(*args, **kwargs)
> File
> "/usr/local/samba/lib/python2.7/site-packages/samba/netcmd/ntacl.py",
> line 214, in run
> lp, use_ntvfs=use_ntvfs)
> File
> "/usr/local/samba/lib/python2.7/site-packages/samba/provision/__init__.py",
> line 1450, in setsysvolacl
> setntacl(lp,sysvol, SYSVOL_ACL, str(domainsid), use_ntvfs=use_ntvfs)
> File "/usr/local/samba/lib/python2.7/site-packages/samba/ntacls.py",
> line 90, in setntacl
> (owner_id, owner_type) = passdb.sid_to_id(sd.owner_sid)
As far as I can tell, you don't have my patch correctly applied, as
those line numbers don't line up with current master+my patch.
Andrew Bartlett
--
Andrew Bartlett http://samba.org/~abartlet/
Authentication Developer, Samba Team http://samba.org
More information about the samba-technical
mailing list