Bug 37909 - Missing hyphen/dash in univention-run-join-scripts (UCC 1.0)
Missing hyphen/dash in univention-run-join-scripts (UCC 1.0)
Status: CLOSED FIXED
Product: Z_Univention Corporate Client (UCC)
Classification: Unclassified
Component: Client management
UCC 1.0
Other Linux
: P5 normal
: UCC 1.0-errata
Assigned To: Felix Botner
Erik Damrose
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2015-03-02 10:47 CET by Michael Grandjean
Modified: 2015-07-01 13:48 CEST (History)
2 users (show)

See Also:
What kind of report is it?: ---
What type of bug is this?: ---
Who will be affected by this bug?: ---
How will those affected feel about the bug?: ---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Grandjean univentionstaff 2015-03-02 10:47:16 CET
> 138       if [ -z "$binddn" ]; then
> 139               # Next check is the local ldapi interface
> 140               binddn="$(univention-ssh "$DCPWD" "$DCACCOUNT"@"$ldap_master" \
> 141                       ldapsearch -x LLL -H ldapi:/// "(&(uid=$DCACCOUNT)(objectClass=person))" dn | ldapsearch-decode64 | sed -ne 's|^dn: ||p;s|^DN: ||p')"
> 142       fi

Because there is a '-' missing before 'LLL', ALL DNs are returned and not only the one we are looking for ($DCACCOUNT).
This might let the join fail, produce confusing warnings during join and is probably related to Bug #30222.

This also applies to UCC 2.0. The version in UCS has been fixed with Bug #32005. 
I guess we should regularly adopt the current version from UCS.
Comment 1 Moritz Muehlenhoff univentionstaff 2015-03-02 10:58:41 CET
We should merge the UCS fix from Bug 32005
Comment 2 Felix Botner univentionstaff 2015-04-22 13:57:48 CEST
univention-ucc-join - 1.1.16-7 3.1-0-0-ucc-errata

2015-04-22-univention-ucc-join.yaml
Comment 3 Erik Damrose univentionstaff 2015-04-23 10:39:40 CEST
OK: Fix: merge from ucc2
OK: YAML
Verified
Comment 4 Erik Damrose univentionstaff 2015-07-01 13:48:12 CEST
http://errata.univention.de/ucc/1.0/45.html