diff options
| author | Chris Lumens <clumens@redhat.com> | 2009-03-04 18:00:22 -0500 |
|---|---|---|
| committer | David Lehman <dlehman@redhat.com> | 2009-03-04 17:30:05 -0600 |
| commit | 99c80e4eb0c4c4fddfbcd24005db680fa9cb93ff (patch) | |
| tree | 4f4592c8db7dd47cfdf2e837b19de7357697e828 /iw/bootloader_main_gui.py | |
| parent | 4503335c6f5d33bcb236da1757123dde73a842bc (diff) | |
If nothing's selected in the bootloader selection combo, don't traceback.
This fixes a rarely reported, even more rarely reproduced bug indicating a
problem subscripting a TreeIter.
Diffstat (limited to 'iw/bootloader_main_gui.py')
| -rw-r--r-- | iw/bootloader_main_gui.py | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/iw/bootloader_main_gui.py b/iw/bootloader_main_gui.py index fda24c70e..452e880e9 100644 --- a/iw/bootloader_main_gui.py +++ b/iw/bootloader_main_gui.py @@ -80,7 +80,12 @@ class MainBootloaderWindow(InstallWindow): def __driveChange(combo, dxml, choices): if not choices.has_key("mbr"): return - first = combo.get_model()[combo.get_active_iter()][1] + + iter = combo.get_active_iter() + if not iter: + return + + first = combo.get_model()[iter][1] desc = choices["mbr"][1] dxml.get_widget("mbrRadio").set_label("%s - /dev/%s" %(desc, first)) dxml.get_widget("mbrRadio").set_data("bootDevice", first) |