[#5977] smbldap driver fail in mixed environment
Summary smbldap driver fail in mixed environment
Queue Passwd
Queue Version HEAD
Type Enhancement
State Resolved
Priority 1. Low
Owners Chuck Hagenbuch <chuck (at) horde (dot) org>
Requester marco (at) csita (dot) unige (dot) it
Created 12/09/2007 (154 days ago)
Due
Updated 12/11/2007 (152 days ago)
Assigned
Resolved 12/11/2007 (152 days ago)
Attachments smbldap.php[1].patch Download
Milestone
Patch

History
12/11/2007 Chuck Hagenbuch Comment #6
Assigned to Chuck Hagenbuch
State ⇒ Resolved		 Reply to this comment
Committed, thanks!
12/11/2007 Chuck Hagenbuch Deleted Attachment: smbldap.php.patch
 
12/11/2007 marco (at) csita (dot) unige (dot) it Comment #5
New Attachment: smbldap.php[1].patch Download		 Reply to this comment
I wrongly worked on a previous version, this should be the right one.

Cheers,
  Marco.
12/10/2007 Chuck Hagenbuch Comment #4 Reply to this comment
Can you make sure the patch is up to date against http://cvs.horde.org/co.php?r=1.16&f=passwd%2Flib%2FDriver%2Fsmbldap.php? The patch doesn't apply cleanly for me and it seems to undo some of the work of ignoring commented out attributes (in the array_merge segment at least).

Thanks!
12/10/2007 Chuck Hagenbuch Deleted Attachment: smbldap.php
 
12/10/2007 marco (at) csita (dot) unige (dot) it Comment #3
New Attachment: smbldap.php.patch		 Reply to this comment
> The entire smbldap.php file is attached, not a patch. Can you please
> upload a patch?

Sorry, correct file is attached herewith.

I also realized that every Samba attributes is update separately. I think this is an error, because all attributes must remain in synch and updates should fail or success together.
I have correct this behaviour, taking in account also Request #5937
  http://bugs.horde.org/ticket/?id=5937
12/09/2007 Chuck Hagenbuch Comment #2
State ⇒ Feedback		 Reply to this comment
The entire smbldap.php file is attached, not a patch. Can you please upload a patch?
12/09/2007 marco (at) csita (dot) unige (dot) it Comment #1
Summary ⇒ smbldap driver fail in mixed environment
Type ⇒ Enhancement
State ⇒ New
Queue ⇒ Passwd
Priority ⇒ 1. Low
New Attachment: smbldap.php		 Reply to this comment
If in LDAP tree entries are mixed among Samba and not Ssamba user, when a non Samba user try to change password using smbldap backend he receives an error ("objectclass violation" usually) but operation success on his standard LDAP password.

I attached a patch to apply SMB password update only if entry permit Samba attributes.