Re: [LINUX5250] lib5250 0.17.3 infinite loop

[James Rich <james@xxxxxxxxxxx>]

On Fri, 6 May 2005, James Rich wrote:

> On Fri, 6 May 2005, Scott Klement wrote:
>
> > A little more info.... it appears that the loop that it's getting stuck in 
> > is the one in the tn5250_display_set_cursor_prev_progression_field() 
> > function.
>
> This is code I wrote.  I'm working on it right now.  The weird part is that 
> tn5250_display_prev_field() will never return the same field as that which is 
> obtained here:
>
>  origfield = tn5250_display_current_field (This);

Figured it out.  In this particular case origfield is a bypass field.  So 
tn5250_display_prev_field() was never returning it and the checks were 
failing.  I now check if it is a bypass field and if so call 
tn5250_display_prev_field() immediately without entering the loop.  Fix is 
in cvs.

James Rich

It's not the software that's free; it's you.
        - billyskank on Groklaw