Comment 2 for bug 556824

Revision history for this message
Daniel Manrique (roadmr) wrote :

Hello, the following information should complete this bug as a valid SRU report, to hopefully upload this fix to Lucid.

Solution:
Checkbox revision 783 changes udev_resource script to be more resilient, using a regexp that takes everything before the "=" as a property name, thus allowing dots and other extraneous characters as property names.

TEST CASE:
1 - Install KVM and related packages:
 https://help.ubuntu.com/community/KVM/Installation
2- Obtain a Lucid ISO image
3- Boot KVM using the Lucid ISO image:
  sudo kvm -vga vmware -m 512 -smp 1 -cdrom ubuntu-10.04.2-desktop-i386.iso -boot once=d
4- Choose to "Try Ubuntu without installing"
5- Once on the Desktop, launch System Testing
6- Perform a test run (all tests can be disabled in the selection screen)

Notice that a trace appears in /var/crash/_usr_share_checkbox_scripts_udev_resource.999.crash

Regression potential:
Since the initial assumption was that udev property names cannot contain non-word characters, and with this change they potentially can, there could be a regression if some other parts of the code expect udev property names to not contain dots or other extraneous characters. However none of the code changes since this fix was introduced are related to strange characters in udev property names, so this fix apparently doesn't cause any problems higher up in the process.