freeipa

Clone
Source Code
GIT

#5921 Local variable s_indent might be referenced before deinfed in textui.print_attribute
Opened 7 years ago by mbasti. Modified 6 years ago

Open
Close issue as:
fixed wontfix worksforme duplicate insufficientinfo invalid

Attribute s_indent is not defined in cases:

  • if terminal length is unknown
  • if text to be printed is unspecified (???)

That code is hard to understand and have poor quality, it is storm of if and else statements and IMO it is not possible to get empty text in that place.

a workaround should be merged in 4.4 timeframe. Proper fix later.

safety fix was pushed 4.4:

  • 493ae1e Fix: Local variable s_indent might be referenced before defined

But this method needs more love in 4.5

Metadata Update from @mbasti:
- Issue assigned to mbasti
- Issue set to the milestone: FreeIPA 4.5 backlog
6 years ago

Metadata Update from @mbasti:
- Assignee reset
6 years ago

Login to comment on this ticket.

Metadata
None
None
None
None
normal
None
None
None
None
defect
None
None
IPA
None
1
None
None
None
None
0
None
None
None
Powered by Pagure 5.13.3
Documentation • File an Issue • About • SSH Hostkey/Fingerprint
© Red Hat, Inc. and others.