Hello Fabio, or anyone else affected,

Accepted xdiagnose into precise-proposed. The package will build now and
be available at http://launchpad.net/ubuntu/+source/xdiagnose/2.5.3 in a
few hours, and then in the -proposed repository.

Please help us by testing this new package.  See
https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to
enable and use -proposed.  Your feedback will aid us getting this update
out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug,
mentioning the version of the package you tested, and change the tag
from verification-needed to verification-done. If it does not fix the
bug for you, please add a comment stating that, and change the tag to
verification-failed.  In either case, details of your testing will help
us make a better decision.

Further information regarding the verification process can be found at
https://wiki.ubuntu.com/QATeam/PerformingSRUVerification .  Thank you in
advance!

** Tags added: verification-needed

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

Title:
  xdiagnose needs root privilege to create /etc/default/grub.bak

Status in xdiagnose:
  Triaged
Status in “xdiagnose” package in Ubuntu:
  Fix Released
Status in “xdiagnose” source package in Precise:
  Fix Committed
Status in “xdiagnose” source package in Quantal:
  Fix Committed
Status in “xdiagnose” source package in Raring:
  Fix Released

Bug description:
  [Impact]
  xdiagnose requires root permission to run, otherwise it crashes when it tries 
to update the grub configuration.

  This is showing up in the automatically collected errors:
  
https://errors.ubuntu.com/bucket/?id=%2Fusr%2Fbin%2Fxdiagnose%3AIOError%3Amove%3Ahandle_enable_apport%3Amove%3Acopy2%3Acopyfile

  This shows the issue affects precise, quantal, and raring users, with
  quantal being the most heavily affected.

  [Testcase]
  1.  Open xterm window
  2.  Run `xdiagnose` as a non-root user
  3.  Change some grub settings
  4.  Click apply

  Expected behavior:  At step 2, remind user to run app as superuser.
  Actual behavior: Crashes at step 4

  [Fix]
  A partial fix was added to the .desktop file, which made xdiagnose use 
gksu/pkexec to request root permissions when run from the menus.  However, 
xdiagnose can also be run directly from X terminals.  To address this case it 
needs to check if it's run as root at startup, and if not print an error so the 
user knows to run it with sudo or similar.

  [Regression Risk]
  The patch is very trivial and non-invasive so not likely to cause 
regressions.  Also, the application is for diagnostic purposes so the impact of 
a regression in this tool is pretty minor.

  [Original Report]
  Hello
  Launching the xdiagnose gui from the icon, it start, accept the checkboxes 
variation but when i click apply it don't require me the password to read the 
grub file.
  Launching it from terminal it return the permission denied error.
  Thanks
  Fabio

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

-- 
Mailing list: https://launchpad.net/~desktop-packages
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~desktop-packages
More help   : https://help.launchpad.net/ListHelp

Reply via email to