Checkbox

non-utf8 characters in Xorg.0.log cause color_depth_info and graphics_driver to throw UnicodeDecodeError exception

Bug #1048096 reported by Daniel Manrique on 2012-09-09

This bug affects 2 people

Affects		Status	Importance	Assigned to	Milestone
	Checkbox	Fix Released	Undecided	Daniel Manrique

Bug Description

One system contains weird characters in its Xorg.0.log file, this information is probably sent by the graphics chipset or firmware. This causes the mentioned tests to fail with this exception:

./color_depth_info
Traceback (most recent call last):
  File "./color_depth_info", line 58, in <module>
    main()
  File "./color_depth_info", line 55, in main
    print('Color Depth: %s\nPixmap Format: %s bpp' % get_color_depth("/home/roadmr/Desktop"))
  File "./color_depth_info", line 47, in get_color_depth
    f.read()):
  File "/usr/lib/python3.2/codecs.py", line 300, in decode
    (result, consumed) = self._buffer_decode(data, self.errors, final)
UnicodeDecodeError: 'utf-8' codec can't decode byte 0x81 in position 32: invalid start byte

I'm attaching an example excerpt from Xorg.0.log which triggers the failure. Apologies for not including the complete file; this is a prerelease system and the file may contain confidential information.

Related branches

lp:~roadmr/checkbox/fix-xorg-parsing

Merged into lp:checkbox at revision 1649

Marc Tardif (community): Approve on 2012-09-10

Daniel Manrique (community): Needs Resubmitting on 2012-09-10

lp:checkbox

lp:~roadmr/ubuntu/quantal/checkbox/0.14.7

Merged into lp:ubuntu/quantal/checkbox at revision 52

Ubuntu branches: Pending requested 2012-09-26

Diff: 114866 lines (+22292/-18992)

125 files modified

checkbox_qt/qt_interface.py (+7/-2)
debian/changelog (+51/-0)
debian/po/ast.po (+2/-2)
debian/po/cs.po (+2/-2)
debian/po/de.po (+2/-2)
debian/po/en_AU.po (+2/-2)
debian/po/en_GB.po (+2/-2)
debian/po/es.po (+2/-2)
debian/po/fr.po (+2/-2)
debian/po/he.po (+2/-2)
debian/po/hu.po (+2/-2)
debian/po/id.po (+2/-2)
debian/po/it.po (+2/-2)
debian/po/ja.po (+2/-2)
debian/po/nl.po (+2/-2)
debian/po/oc.po (+2/-2)
debian/po/pl.po (+2/-2)
debian/po/pt_BR.po (+2/-2)
debian/po/ro.po (+2/-2)
debian/po/ru.po (+2/-2)
debian/po/tr.po (+2/-2)
debian/po/uk.po (+2/-2)
debian/po/zh_CN.po (+2/-2)
debian/po/zh_TW.po (+2/-2)
jobs/audio.txt.in (+6/-6)
jobs/networking.txt.in (+4/-1)
jobs/suspend.txt.in (+5/-4)
patches/0.14.2 (+2/-1)
po/ace.po (+199/-198)
po/af.po (+199/-198)
po/am.po (+199/-198)
po/ar.po (+219/-203)
po/ast.po (+199/-198)
po/az.po (+199/-198)
po/be.po (+199/-198)
po/bg.po (+199/-198)
po/bn.po (+199/-198)
po/bo.po (+199/-198)
po/br.po (+199/-198)
po/bs.po (+199/-198)
po/ca.po (+199/-198)
po/ca@valencia.po (+199/-198)
po/ckb.po (+199/-198)
po/cs.po (+199/-198)
po/cy.po (+199/-198)
po/da.po (+199/-198)
po/de.po (+179/-178)
po/dv.po (+199/-198)
po/el.po (+199/-198)
po/en_AU.po (+415/-414)
po/en_CA.po (+199/-198)
po/en_GB.po (+891/-504)
po/eo.po (+202/-201)
po/es.po (+462/-216)
po/et.po (+199/-198)
po/eu.po (+199/-198)
po/fa.po (+199/-198)
po/fi.po (+202/-201)
po/fr.po (+1576/-489)
po/ga.po (+199/-198)
po/gd.po (+199/-198)
po/gl.po (+261/-208)
po/he.po (+199/-198)
po/hi.po (+199/-198)
po/hr.po (+199/-198)
po/hu.po (+199/-198)
po/hy.po (+199/-198)
po/id.po (+199/-198)
po/is.po (+199/-198)
po/it.po (+199/-198)
po/ja.po (+199/-198)
po/jbo.po (+199/-198)
po/ka.po (+199/-198)
po/kk.po (+199/-198)
po/km.po (+199/-198)
po/kn.po (+199/-198)
po/ko.po (+199/-198)
po/ku.po (+199/-198)
po/ky.po (+199/-198)
po/lt.po (+199/-198)
po/lv.po (+199/-198)
po/mk.po (+199/-198)
po/ml.po (+199/-198)
po/mr.po (+199/-198)
po/ms.po (+666/-251)
po/my.po (+199/-198)
po/nb.po (+199/-198)
po/nds.po (+199/-198)
po/ne.po (+199/-198)
po/nl.po (+199/-198)
po/nn.po (+199/-198)
po/oc.po (+204/-199)
po/pl.po (+199/-198)
po/ps.po (+199/-198)
po/pt.po (+266/-257)
po/pt_BR.po (+626/-236)
po/ro.po (+199/-198)
po/ru.po (+629/-222)
po/sd.po (+199/-198)
po/shn.po (+199/-198)
po/si.po (+199/-198)
po/sk.po (+199/-198)
po/sl.po (+288/-225)
po/sq.po (+217/-198)
po/sr.po (+199/-198)
po/sv.po (+199/-198)
po/ta.po (+199/-198)
po/te.po (+199/-198)
po/th.po (+199/-198)
po/tr.po (+204/-203)
po/ug.po (+199/-198)
po/uk.po (+218/-217)
po/ur.po (+199/-198)
po/uz.po (+199/-198)
po/vi.po (+199/-198)
po/zh_CN.po (+199/-198)
po/zh_HK.po (+199/-198)
po/zh_TW.po (+199/-198)
qt/frontend/qtfront.ui (+2/-2)
scripts/audio_settings (+12/-2)
scripts/color_depth_info (+1/-1)
scripts/frequency_governors_test (+5/-2)
scripts/graphics_driver (+4/-4)
scripts/graphics_modes_info (+1/-1)
scripts/network_device_info (+95/-44)

Revision history for this message

Daniel Manrique (roadmr) wrote on 2012-09-09:

Xorg.0.log Edit (35 bytes, text/plain)

Revision history for this message

Daniel Manrique (roadmr) wrote on 2012-09-09:

This is the error for graphics_driver:

$ ./graphics_driver
Traceback (most recent call last):
  File "./graphics_driver", line 323, in <module>
    sys.exit(main())
  File "./graphics_driver", line 314, in main
    xlog = XorgLog("/home/roadmr/Desktop/Xorg.0.log")
  File "./graphics_driver", line 63, in __init__
    self.parse(logfile)
  File "./graphics_driver", line 73, in parse
    for line in in_file.readlines():
  File "/usr/lib/python3.2/codecs.py", line 300, in decode
    (result, consumed) = self._buffer_decode(data, self.errors, final)
UnicodeDecodeError: 'utf-8' codec can't decode byte 0x81 in position 32: invalid start byte