diff options
| author | Zygmunt Krynicki <zygmunt.krynicki@canonical.com> | 2014-06-17 16:10:34 +0200 |
|---|---|---|
| committer | Zygmunt Krynicki <zygmunt.krynicki@canonical.com> | 2014-06-17 16:10:34 +0200 |
| commit | 2a35e07ac20314fb2d97f7e662ce38284f276cc4 (patch) | |
| tree | eb667ab5577571be9f0f9638b118c39fff20bf76 /jobs | |
| parent | 6fd768729920e4221f4d0815f598fb510acd2de7 (diff) | |
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 <zygmunt.krynicki@canonical.com>
Diffstat (limited to 'jobs')
0 files changed, 0 insertions, 0 deletions