Bug 43273 - local variable 'windomain' referenced before assignment
local variable 'windomain' referenced before assignment
Product: UCS
Classification: Unclassified
Component: UMC - Setup wizard
UCS 4.1
Other Linux
: P5 normal (vote)
: UCS 4.2-0-errata
Assigned To: Florian Best
Richard Ulmer
Depends on:
  Show dependency treegraph
Reported: 2017-01-02 17:50 CET by Florian Best
Modified: 2017-06-19 15:04 CEST (History)
0 users

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: 6: Setup Problem: Issue for the setup process
Who will be affected by this bug?: 1: Will affect a very few installed domains
How will those affected feel about the bug?: 3: A User would likely not purchase the product
User Pain: 0.103
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number: 2016122921000239, 2017050721000096
Bug group (optional): External feedback
Max CVSS v3 score:

patch (486 bytes, patch)
2017-01-02 17:51 CET, Florian Best
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Florian Best univentionstaff 2017-01-02 17:50:04 CET
Version: 4.1-4 errata366 (Vahr)

Ein unerwarteter Fehler trat während des Einrichtungsprozesses auf: local variable 'windomain' referenced before assignment
Comment 1 Florian Best univentionstaff 2017-01-02 17:51:06 CET
Created attachment 8329 [details]

Function was added in svn r52011.
Happens if the entered domainname doesn't contain a dot.
Comment 2 Florian Best univentionstaff 2017-05-10 16:10:40 CEST
Reported again, 4.1-4 errata324 (Vahr)
Comment 3 Florian Best univentionstaff 2017-05-29 17:24:41 CEST
Can be reproduced in a python shell:
# python
>>> from univention.management.console.modules.setup.util import domain2windowdomain
>>> domain2windowdomain('foo')

univention-system-setup (10.0.10-19):
r79939 | Bug #43273: don't crash if given domainname contains no dot

r79941 | YAML Bug #44035 Bug #44394 Bug #43273
Comment 4 Richard Ulmer univentionstaff 2017-05-30 09:38:23 CEST
The change looks reasonable and the function works with inputs without '.'s now. -> Verified.
Comment 5 Janek Walkenhorst univentionstaff 2017-06-19 15:04:48 CEST