wicd-curses: Crash and raise IndexError, "No widget at position %s"

I also met this problem, the version is:
 $ wicd-curses --version -I-
wicd-curses-uimod (using wicd 1.7.2.4)

It's easy to reproduce in a environment with many access points, continue refresh and move the focus to the access point at the bottom each time, when the numbers of access points scanned this time is less than last time, it can't focus on a non-exist access point, crash comes out.

I've view some code and get a solution, change

    self.wlessLB.set_focus(self.focusloc[1])
into
    self.wlessLB.set_focus(min(self.focusloc[1], len(self.wlessLB.body) -1))

in line 703 of wicd-curses.py, it will focus on last access point when get less access points then last scan.

I found my fix is bad, will cause other problems, I wrote another dirty fix for myself, but hope to get better fixes:

  File "/usr/share/wicd/curses/wicd-curses.py", line 703, in update_netlist
    self.wlessLB.set_focus(min(self.focusloc[1], len(self.wlessLB.body) -1))
TypeError: object of type 'PollingListWalker' has no len()

You can indeed reproduce this as @radaiming said.
This fix I wrote seems to work:

if self.focusloc[1] < len(wlessL):
    self.wlessLB.set_focus(self.focusloc[1])
else:
    self.wlessLB.set_focus(len(wlessL) - 1)

or as @radaiming would do it

self.wlessLB.set_focus(min(self.focusloc[1], len(wlessL) - 1))

Patch (based on main branch) for my above comment.
Edited file: http://bazaar.launchpad.net/~wicd-devel/wicd/experimental/view/head:/curses/wicd-curses.py

Improved and applied your patch. Thanks for making wicd more stable!

=== modified file 'curses/wicd-curses.py'
--- curses/wicd-curses.py 2014-12-19 14:40:39 +0000
+++ curses/wicd-curses.py 2014-12-20 09:30:43 +0000
@@ -872,7 +872,8 @@
                         get_body().set_focus(self.focusloc[1])
                 else:
                     if self.wlessLB != self.no_wlan:
- self.thePile.get_focus().set_focus(self.focusloc[1])
+ # Set the focus to the last selected item, but never past the length of the list
+ self.thePile.get_focus().set_focus(min(self.focusloc[1], len(wlessL) - 1))
                     else:
                         self.thePile.set_focus(self.wiredCB)
             else:
@@ -885,7 +886,8 @@
                 if self.focusloc[1] is None:
                     self.focusloc[1] = 0
                 if self.wlessLB != self.no_wlan:
- self.wlessLB.set_focus(self.focusloc[1])
+ # Set the focus to the last selected item, but never past the length of the list
+ self.wlessLB.set_focus(min(self.focusloc[1], len(wlessL) - 1))

         self.prev_state = state
         if not firstrun:

Hi all,

i use wicd-curses and I notice the exact bug claimed here. I saw above the patch by tivervac and its improvement by tomvanbraeckel, and I have two questions: do I have to only apply the last patch, or both two (first the tivervac one, then tomvanbraeckel one)? And does it work with wicd 1.7.3-1?

Thanks very much, and sorry for the behave!
Regards

Only the improved patch (the last one) will do. Try it on wicd 1.7.3-1!