lp:~zyga/checkbox/fix-1235426

Branch merges

Propose for merging

No branches dependent on this one.

Merged into lp:checkbox at revision 3080

Daniel Manrique (community): Approve on 2014-06-17

Diff: 10 lines (+0/-1)

1 file modified

providers/plainbox-provider-checkbox/bin/camera_test (+0/-1)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Bug #1235426: /usr/lib/checkbox/bin/camera_test:KeyError:/usr/share/checkbox/scripts/camera_test@558:detect:_supported_resolutions_to_string

Undecided

Fix Released

Link a bug report

Related blueprints

3079. By Zygmunt Krynicki on 2014-06-17

provider:checkbox: fix bad error case in camera_test

This patch changes CameraTest._get_supported_resolutions() not to
return early from the per-format loop when an IOError occurs while
manipulating the device. This allows the 'resolutions' element
to be assigned which ensures that every dictionary in supported_formats
has that element set.

This bug can happen, for example, when the current user has no permission
to manipulate the camera device and gets an IOError with EPERM inside.

Fixes: https://bugs.launchpad.net/plainbox-provider-checkbox/+bug/1235426

Signed-off-by: Zygmunt Krynicki <email address hidden>

3078. By Po-Hsu Lin on 2014-06-17

"automatic merge by tarmac [r=zkrynicki][bug=1327949][author=cypressyew]"

3077. By Zygmunt Krynicki on 2014-06-17

"automatic merge by tarmac [r=zkrynicki,sylvain-pineau][bug=][author=zkrynicki]"

3076. By Zygmunt Krynicki on 2014-06-17

"automatic merge by tarmac [r=zkrynicki][bug=][author=zkrynicki]"

3075. By Zygmunt Krynicki on 2014-06-16

"automatic merge by tarmac [r=sylvain-pineau][bug=][author=zkrynicki]"

3074. By Sylvain Pineau on 2014-06-16

"automatic merge by tarmac [r=sylvain-pineau][bug=][author=sylvain-pineau]"

3073. By Daniel Manrique on 2014-06-16

"test-in-lxc: fix recommended sudo configuration for missing commands [r=zkrynicki][bug=][author=roadmr]"

3072. By Po-Hsu Lin on 2014-06-16

"automatic merge by tarmac [r=roadmr][bug=][author=cypressyew]"

3071. By Po-Hsu Lin on 2014-06-16

"automatic merge by tarmac [r=zkrynicki][bug=1330361][author=cypressyew]"

3070. By Chris Gregan on 2014-06-13

"automatic merge by tarmac [r=roadmr][bug=][author=cgregan]"