desktop-packages team mailing list archive

[Robert Ancell <robert.ancell@xxxxxxxxxxxxx>]

** Also affects: ubuntu
   Importance: Undecided
       Status: New

** Also affects: Ubuntu Vivid
   Importance: Undecided
       Status: New

** Package changed: ubuntu => lightdm (Ubuntu)

** Changed in: lightdm (Ubuntu)
       Status: New => Fix Released

** Changed in: lightdm (Ubuntu Vivid)
       Status: New => Triaged

** Changed in: lightdm (Ubuntu Vivid)
   Importance: Undecided => Medium

** Changed in: lightdm (Ubuntu)
   Importance: Undecided => Medium

** Summary changed:

- lightdm.conf is sensitive to trailing whitespace
+ lightdm.conf boolean values false if have trailing whitespace

** Description changed:

- It appears that lightdm.conf is sensitive to trailing spaces. I get
- different behavior with
+ [Impact]
+ Adding trailing whitespace to a boolean true value in LightDM configuration has it read as false.
  
- greeter-hide-users=true
+ [Test Case]
+ 1. Set a boolean configuration value in lightdm.conf to true_ (_ is a space character)
+ 2. Run LightDM
  
- and
+ Expected result:
+ The configured value is interpreted as true.
  
- greeter-hide-users=true_
+ Observed result:
+ The configured value is interpreted as false.
  
- where the _ in the second example represents a trailing space. Without
- the space, the greeter hides the list of users as I expected. With the
- space, the greeter displays the list of users as if the greeter-hide-
- users parameter is not set to true.
+ [Regression potential]
+ Low. We just strip trailing whitespace when checking for the value "true".

-- 
You received this bug notification because you are a member of Desktop
Packages, which is subscribed to lightdm in Ubuntu.
https://bugs.launchpad.net/bugs/1507033

Title:
  lightdm.conf boolean values false if have trailing whitespace

Status in Light Display Manager:
  Fix Released
Status in Light Display Manager 1.10 series:
  Fix Committed
Status in Light Display Manager 1.14 series:
  Fix Committed
Status in Light Display Manager 1.16 series:
  Fix Committed
Status in Light Display Manager 1.2 series:
  Fix Committed
Status in lightdm package in Ubuntu:
  Fix Released
Status in lightdm source package in Vivid:
  Triaged

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".

To manage notifications about this bug go to:
https://bugs.launchpad.net/lightdm/+bug/1507033/+subscriptions