last-used-folder -> None crash

Bug #1019180 reported by Michael Schwendt
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
SoundConverter
Fix Released
Critical
GautierPortet

Bug Description

Another crash has been reported where the code tries to set "last-used-folder" to "None" in one place where no safety check is applied:

    https://bugzilla.redhat.com/836338

backtrace:
:gconfstore.py:55:set_string:TypeError: GConf.Client.set_string() argument 2
must be string, not None
:
:Traceback (most recent call last):
: File "/usr/lib/soundconverter/python/soundconverter/ui.py", line 1254, in
on_addfolder_activate
: self.addfolderchooser.get_current_folder_uri())
: File "/usr/lib/soundconverter/python/soundconverter/gconfstore.py", line 55,
in set_string
: self.gconf.set_string(self.path(key), value)
:TypeError: GConf.Client.set_string() argument 2 must be string, not None
:
:Local variables in innermost frame:
:self: <soundconverter.ui.PreferencesDialog object at 0xb6b7128c>
:key: 'last-used-folder'
:value: None

Revision history for this message
Michael Schwendt (mschwendt) wrote :
Revision history for this message
GautierPortet (kassoulet) wrote :

Thank you Michael!

Changed in soundconverter:
importance: Undecided → Critical
assignee: nobody → GautierPortet (kassoulet)
milestone: none → 2.0.4
status: New → Fix Committed
Changed in soundconverter:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.