Bug 40479

Summary: missing escapes in "univention-lib"
Product: UCS Reporter: Arvid Requate <requate>
Component: UCRAssignee: UCS maintainers <ucs-maintainers>
Status: RESOLVED WONTFIX QA Contact:
Severity: normal    
Priority: P3 CC: best, gohmann, hahn, markus.daehlmann, requate, thorp-hansen, walkenhorst
Version: UCS 4.0Flags: requate: Patch_Available+
Target Milestone: UCS 4.1-x   
Hardware: Other   
OS: Linux   
What kind of report is it?: Bug Report What type of bug is this?: 3: Simply Wrong: The implementation doesn't match the docu
Who will be affected by this bug?: 2: Will only affect a few installed domains How will those affected feel about the bug?: 2: A Pain – users won’t like this once they notice it
User Pain: 0.069 Enterprise Customer affected?:
School Customer affected?: ISV affected?:
Waiting Support: Flags outvoted (downgraded) after PO Review:
Ticket number: Bug group (optional): Troubleshooting
Max CVSS v3 score:
Bug Depends on: 40031    
Bug Blocks:    

Description Arvid Requate univentionstaff 2016-01-20 18:29:56 CET
We might need to backport the fix for "is_ucr_true" for UCS 4.0 too:


+++ This bug was initially created as a clone of Bug #40031 +++

=== Bug #40031 Comment 2 ===

When executing some univention commands the following line appears in the terminal:

tr: misaligned [:upper:] and/or [:lower:] construct

The problem seems to be in the function " is_ucr_true" in the file /usr/share/univention-lib/ucr.sh.

=== Bug #40031 Comment 9 ===

Note: This is the german translation of the error message:

tr: fehlerhaft positioniertes [:upper:]- und/oder [:lower:]-Konstrukt

We have seen this at Ticket#2016011921000162 on one UCS 4.0-4 DC Slave (errata 386 at the time of analysis), where is_ucr_true failed with this message in a joinscript. As a result it took the wrong if/then/else branch.
Comment 1 Markus Dählmann 2016-03-21 10:23:37 CET
This hit me again today on a school DC, which by coincidence/lazy naming had a file called "p" in the /root/ directory. Server changed its machine password although this is disabled via UCR:

root@dc601:~# ucr get server/password/change
false
root@dc601:~# . /usr/share/univention-lib/all.sh
root@dc601:~# eval "$(/usr/sbin/univention-config-registry shell)"
root@dc601:~# is_ucr_true server/password/change
tr: fehlerhaft positioniertes [:upper:]- und/oder [:lower:]-Konstrukt
root@dc601:~# echo $?
2
root@dc601:~# ls -l
insgesamt 144
[...]
-rw-r--r--  1 root root 46553 Feb 15 17:53 p
[...]
Comment 2 Philipp Hahn univentionstaff 2017-02-10 12:54:05 CET
UCS-4.0 is no longer maintained; the bug is fixed in UCS-4.1.
Closing this request for back-port as WONT-FIX.