[PATCH] Please review my session-info branch
nivanova at samba.org
Mon Aug 16 11:48:06 MDT 2010
I checked the descriptors in your session_info branch and they are OK.
On Mon, Aug 16, 2010 at 1:36 AM, Andrew Bartlett <abartlet at samba.org> wrote:
> On Sun, 2010-08-15 at 17:45 +0300, Nadezhda Ivanova wrote:
> > Hi Andrew,
> > I do not understand enough the purpose of your patch, but replacing
> > anonymous_session with system_session in acl.py is not acceptable -
> > recently I added some tests for denying anonymous access, which I will
> > activate when I am able to get this feature in master. The tests do
> > work against windows and are tested :). If you tell me what else I can
> > use to establish an anonymous connection, then I'll use it instead and
> > the patch would be OK, otherwise I need this binding.
> To make an LDB connection with anonymous (or other) credentials, simply
> specify those separately to the session_info. Explicitly specified
> credentials are always used in preference to those associated with the
> > Also, what is the purpose of
> Are you sure it doesn't change the resulting token? We use this session
> during provision to make sure the object's descriptors are generated
> properly. I'll compile your branch and take a look at the resulting
> descriptors to make sure they look as they should...
> Please do check, but I couldn't see any difference in the code.
> Andrew Bartlett
> Andrew Bartlett http://samba.org/~abartlet/<http://samba.org/%7Eabartlet/>
> Authentication Developer, Samba Team http://samba.org
> Samba Developer, Cisco Inc.
More information about the samba-technical