[#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 (2020 days ago)
Due
Updated 12/11/2007 (2018 days ago)
Assigned
Resolved 12/11/2007 (2018 days ago)
Attachments smbldap.php[1].patch Download
Milestone
Patch No

History
12/11/2007 07:26:04 PM Chuck Hagenbuch Comment #6
Assigned to Chuck Hagenbuch
State ⇒ Resolved		 Reply to this comment
Committed, thanks!
12/11/2007 07:20:46 PM Chuck Hagenbuch Deleted Attachment: smbldap.php.patch
 
12/11/2007 02:49:59 PM 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 10:25:48 PM 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 08:16:00 PM Chuck Hagenbuch Deleted Attachment: smbldap.php
 
12/10/2007 06:54:09 PM 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 11:07:18 PM 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 01:00:50 PM marco (at) csita (dot) unige (dot) it Comment #1
New Attachment: smbldap.php
State ⇒ New
Priority ⇒ 1. Low
Type ⇒ Enhancement
Summary ⇒ smbldap driver fail in mixed environment
Queue ⇒ Passwd		 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.