correctly return error and emit warning
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
liboobs (Debian) |
Fix Released
|
Unknown
|
|||
liboobs (Ubuntu) |
Fix Released
|
Undecided
|
The ubuntu geek's | ||
Trusty |
Fix Released
|
Medium
|
Brian Murray | ||
Utopic |
Fix Released
|
Medium
|
Brian Murray |
Bug Description
[Impact]
When a user tries to change his full name via users-admin or mate-users-admin, and some error happens in the backends (either liboobs or system-
The issues are already fixed in Debian and Ubuntu 15.04, but the package remains unpatched in 14.04 and 14.10.
The attached patches make users-admin (and mate-users-admin) show the error message dialog when the mentioned error happens, and also show the unchanged full name in this case.
[Test Case]
You'll need the unpatched version of system-
Run users-admin or mate-users-admin from the terminal. Click on the button for changing your full name, change the name to something else and click "OK".
You'll see a warning message in the terminal:
Liboobs-WARNING **: There was an unknown error communicating with the backends: Launch helper exited with unknown return code 255
No error message dialog will appear though.
Also the full name in the GUI will be changed to the name you entered. This is not correct because some error happened in the backends, and the name hasn't been actually changed. You can see that for yourself by simply restarting users-admin or mate-users-admin - it will show the old name then.
[Regression Potential]
I consider the regression risk to be low here. The patches have been already tested by me and liboobs maintainer in Debian (see the corresponding Debian bug report).
Changed in liboobs (Debian): | |
status: | Unknown → Fix Released |
Changed in liboobs (Ubuntu): | |
assignee: | nobody → The ubuntu geek's (the-ubuntu-geek) |
Changed in liboobs (Ubuntu Trusty): | |
assignee: | nobody → Brian Murray (brian-murray) |
Changed in liboobs (Ubuntu Utopic): | |
assignee: | nobody → Brian Murray (brian-murray) |
Changed in liboobs (Ubuntu Trusty): | |
status: | Triaged → In Progress |
Changed in liboobs (Ubuntu Utopic): | |
status: | Triaged → In Progress |
The attachment "Patch 1 - correctly return error code on unknown dbus errors." seems to be a patch. If it isn't, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are a member of the ~ubuntu-reviewers, unsubscribe the team.
[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issues please contact him.]