lynx-dev [PATCH] Prematurely ending command scripts cause lynx to busy loop

[Ville Herva]

[Please CC me, I'm not subscribed to lynx-dev.]

k.script:
---8<---8<---8<---8<---8<---8<---8<---8<---8<---8<---
# Command logfile created by Lynx 2.8.5dev.3 (06 Oct 2001)
key g
key h
key t
key t
key p
key :
key /
key /
---8<---8<---8<---8<---8<---8<---8<---8<---8<---8<---

Then try

  lynx -cmd_script=k.script http://www.hut.fi

Lynx will busy loop for ever (tried 2.8.5dev8, dev3 and dev1). The same
happens with all prematurely ending command scripts.

The attached patch against 2.8.5dev8 fixes it for me.

 
-- v --

[EMAIL PROTECTED]

--- src/LYStrings.c.ORIG        Fri Jun 28 00:20:25 2002
+++ src/LYStrings.c     Fri Jun 28 00:21:21 2002
@@ -5694,7 +5694,11 @@
                break;
            }
        }
-       FREE(buffer);
+       if (feof(cmd_script)) {
+           fclose(cmd_script);
+           cmd_script = 0;    
+       }
+       FREE(buffer);
     } else {
        ch = LYgetch_for(mode);
     }