Bug 10374 - Samba 4 dies when userParameters is set in primary DC
Summary: Samba 4 dies when userParameters is set in primary DC
Status: RESOLVED FIXED
Alias: None
Product: Samba 4.0
Classification: Unclassified
Component: AD: LDB/DSDB/SAMDB (show other bugs)
Version: 4.0.14
Hardware: x64 Linux
: P5 critical (vote)
Target Milestone: ---
Assignee: Andrew Bartlett
QA Contact: Samba QA Contact
URL:
Keywords:
: 10401 (view as bug list)
Depends on:
Blocks: 8077
  Show dependency tree / graph
 
Reported: 2014-01-13 12:44 UTC by Holger
Modified: 2022-04-22 02:33 UTC (History)
1 user (show)

See Also:

Attachments
Patch for ldb_val to lsa_BinaryString conversation (2.34 KB, patch)
2014-05-27 19:45 UTC, Stefan Gohmann 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Holger 2014-01-13 12:44:02 UTC 
When the LDAP attribute "userParameters" is set in the Primary DC (In my case Windows Server 2008 Small Business Server) than Samba dies when you request Userinformation about the user who has data set in this attribute.

Under SBS the attribute userParameters is set with the following data:

"CtxCfgPresent                                   PCtxCfgPresent㔵攱戰ã¢CtxCfgFlags1〰ã¦ã€²ã€¹"

(I think its UTF-8 encoded)

After deleting this Attributedata I can user the user Account without samba crash.

Best regards,
Holger Martiker
Comment 1 Holger 2014-01-24 10:20:58 UTC 
*** Bug 10401 has been marked as a duplicate of this bug. ***
Comment 2 Stefan Gohmann 2014-05-27 19:45:24 UTC 
Created attachment 9984 [details]
Patch for ldb_val to lsa_BinaryString conversation

The attached patch should fix this issue.
Comment 3 Douglas Bagnall 2022-04-22 02:33:43 UTC 
(In reply to Stefan Gohmann from comment #2)
It looks like that patch was obsoleted by d7b4d10aba90f4a1acf01d1d5ab62161862f62f7 (the case where val->length/2 != (val->length + 1)/2 is never hit).

That also should have stopped the crash.

I will close this. If you disagree, please reopen.