lightdm.conf boolean values false if have trailing whitespace
Bug #1507033 reported by
Daniel E. Shub
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Light Display Manager |
Fix Released
|
Medium
|
Robert Ancell | ||
1.10 |
Fix Committed
|
Medium
|
Unassigned | ||
1.14 |
Fix Released
|
Medium
|
Robert Ancell | ||
1.16 |
Fix Released
|
Medium
|
Robert Ancell | ||
1.2 |
Fix Committed
|
Medium
|
Unassigned | ||
lightdm (Ubuntu) |
Fix Released
|
Medium
|
Robert Ancell | ||
Trusty |
Fix Released
|
Medium
|
Robert Ancell | ||
Vivid |
Fix Released
|
Medium
|
Robert Ancell | ||
Wily |
Fix Released
|
Medium
|
Robert Ancell |
Bug Description
[Impact]
Adding trailing whitespace to a boolean true value in LightDM configuration has it read as false.
[Test Case]
1. Set a boolean configuration value in lightdm.conf to true_ (_ is a space character)
2. Run LightDM
Expected result:
The configured value is interpreted as true.
Observed result:
The configured value is interpreted as false.
[Regression potential]
Low. We just strip trailing whitespace when checking for the value "true".
Changed in lightdm: | |
milestone: | none → 1.17.0 |
status: | Triaged → Fix Committed |
Changed in lightdm: | |
status: | Fix Committed → Fix Released |
affects: | ubuntu → lightdm (Ubuntu) |
Changed in lightdm (Ubuntu): | |
status: | New → Fix Released |
Changed in lightdm (Ubuntu Vivid): | |
status: | New → Triaged |
importance: | Undecided → Medium |
Changed in lightdm (Ubuntu): | |
importance: | Undecided → Medium |
summary: |
- lightdm.conf is sensitive to trailing whitespace + lightdm.conf boolean values false if have trailing whitespace |
description: | updated |
Changed in lightdm (Ubuntu Vivid): | |
status: | Triaged → In Progress |
assignee: | nobody → Robert Ancell (robert-ancell) |
Changed in lightdm (Ubuntu Wily): | |
importance: | Undecided → Medium |
status: | New → Fix Committed |
tags: |
added: verification-done removed: verification-needed |
tags: |
added: verification-done-vivid removed: verification-done |
Changed in lightdm (Ubuntu Trusty): | |
importance: | Undecided → Medium |
To post a comment you must log in.
Yeah, we should definitely be ignoring whitespace.