I just got the snapshot and see your fix. Thank you. I thought of using a "goto", but having drilled into me that if you can use something else, do it, I choose to restructure. Perhaps this is one of those exceptions where a "goto" is the cleanest way.
Thanks for your work. - Dan Perik On Mon, 2002-03-18 at 20:38, Kostas Kalevras wrote: > On 18 Mar 2002, Dan Perik wrote: > > > > > As promised, here's the patch I threw together for the rlm_ldap module > > to solve the problem of failed auth when the LDAP server disconnects the > > idle connection. > > > > Basically, I took the ldap_connect code out of the perform_search > > function into it's own "search_connect" function. Then, if > > ldap_search_st returns LDAP_SERVER_DOWN, it sets inst->bound to 0, does > > search_connect to try to reconnect to the server, and tries the > > ldap_search_st one more time. > > > > Again, my understanding of all this stuff is very limited. For all I > > know I created a vast memory leak that will rot your hard drive and > > cause your business to go bankrupt. > > > > - Dan > > > > > > -- > > - Dan Perik > > Computer Services Department > > Lapilo Center > > New Tribes Mission - PNG > > > > Bug fixed in cvs. The fix was a little different than your patch but > anyway thanks for the bug note. > > -- > Kostas Kalevras Network Operations Center > [EMAIL PROTECTED] National Technical University of Athens, Greece > Work Phone: +30 10 7721861 > 'Go back to the shadow' Gandalf > > > - > List info/subscribe/unsubscribe? See http://www.freeradius.org/list/users.html -- - Dan Perik Computer Services Department Lapilo Center New Tribes Mission - PNG - List info/subscribe/unsubscribe? See http://www.freeradius.org/list/users.html
