From 257b1e3b88f75fa233d69b9a18a421a1d23522ff Mon Sep 17 00:00:00 2001
From: Mike Frysinger <vapier@gentoo.org>
Date: Fri, 18 Feb 2011 20:15:13 +0000
Subject: [PATCH] convert urj_parse_stream parsing to getline so that we dont
 have any arbitrary line length limitations on the inputs

git-svn-id: https://urjtag.svn.sourceforge.net/svnroot/urjtag/trunk@1877 b68d4a1b-bc3d-0410-92ed-d4ac073336b7
---
 urjtag/ChangeLog          |  4 ++
 urjtag/configure.ac       |  1 +
 urjtag/src/global/parse.c | 91 +++++++++++++++++++++++++++------------
 3 files changed, 69 insertions(+), 27 deletions(-)

diff --git a/urjtag/ChangeLog b/urjtag/ChangeLog
index fa719ceb..1da76d8f 100644
--- a/urjtag/ChangeLog
+++ b/urjtag/ChangeLog
@@ -7,6 +7,10 @@
   * include/urjtag/parse.h (urj_parse_line): Constify the "line" argument.
   * src/global/parse.c (urj_parse_line): Likewise.
 
+  * configure.ac: Check for getline().
+  * src/global/parse.c (urj_parse_stream): Get rid of line length limitations
+  by using getline when it is available.
+
 2011-02-17  Mike Frysinger  <vapier@gentoo.org>
 
   * data/analog/bf548/STEPPINGS: Add bf54x-0.4 support.
diff --git a/urjtag/configure.ac b/urjtag/configure.ac
index 8e9ee407..e8cdf8e7 100644
--- a/urjtag/configure.ac
+++ b/urjtag/configure.ac
@@ -119,6 +119,7 @@ fi
 AC_CHECK_FUNCS(m4_flatten([
 	_sleep
 	geteuid
+	getline
 	getuid
 	nanosleep
 	pread
diff --git a/urjtag/src/global/parse.c b/urjtag/src/global/parse.c
index 6d84cfb6..a190d2cb 100644
--- a/urjtag/src/global/parse.c
+++ b/urjtag/src/global/parse.c
@@ -40,10 +40,6 @@
 #include <urjtag/jtag.h>
 #include <urjtag/bsdl.h>
 
-#define MAXINPUTLINE 300        /* Maximum input line length */
-
-
-
 int
 urj_parse_line (urj_chain_t *chain, const char *line)
 {
@@ -151,39 +147,78 @@ urj_parse_line (urj_chain_t *chain, const char *line)
     return r;
 }
 
+#ifndef HAVE_GETLINE
+#define MAXINPUTLINE 300        /* Maximum input line length */
+static ssize_t getline(char **line, size_t *len, FILE *f)
+{
+    static int lnr;
+    char *p;
+
+    ++lnr;
+    if (!*line)
+    {
+        *len = MAXINPUTLINE + 1;
+        *line = malloc (*len);
+    }
+
+    if (fgets (*line, *len, f) == NULL)
+        return -1;
+
+    /* Clip any comments */
+    p = strchr (*line, '#');
+    if (p)
+    {
+        p[0] = '\n';
+        p[1] = '\0';
+    }
+
+    /* Make sure the line wasn't clipped */
+    p = strchr (*line, '\n');
+    if (!p)
+    {
+        urj_warning ("line %d exceeds %zd characters, clipped\n", lnr, *len);
+        while (1) {
+            char c = fgetc (f);
+            if (c == '\n' || c == EOF)
+                break;
+        }
+    }
+
+    return p - *line + 1;
+}
+#endif
 
 int
 urj_parse_stream (urj_log_level_t ll, urj_chain_t *chain, FILE *f)
 {
-    char inputline[MAXINPUTLINE + 1];
-    int go = 1, i, c, lnr, clip, found_comment;
+    char *inputline, *p;
+    size_t len;
+    int go;
 
     /* read the stream line-by-line until EOF or "quit" */
-    lnr = 0;
+    inputline = NULL;
     do
     {
-        i = 0;
-        clip = 0;
-        found_comment = 0;
-
-        /* read stream until '\n' or EOF, copy at most MAXINPUTLINE-1 chars */
-        while (1)
+        if (getline (&inputline, &len, f) == -1)
         {
-            c = fgetc (f);
-            if (c == EOF || c == '\n')
+            if (!feof (f))
+            {
+                go = URJ_STATUS_FAIL;
+                urj_warning ("getline() failed\n");
                 break;
-            if (c == '#')
-                found_comment = 1;
-            if (i < sizeof (inputline) - 1)
-                inputline[i++] = c;
-            else
-                clip = 1;
+            }
+
+            /* Make sure inputline is big enough ... don't rely on getline */
+            inputline = realloc (inputline, 5);
+            sprintf (inputline, "quit");
         }
-        inputline[i] = '\0';
-        lnr++;
-        if (clip && !found_comment)
-            urj_warning ("line %d exceeds %zd characters, clipped\n", lnr,
-                         sizeof (inputline) - 1);
+        else
+        {
+            p = strchr (inputline, '\n');
+            if (p)
+                *p = '\0';
+        }
+
         go = urj_parse_line (chain, inputline);
         if (go == URJ_STATUS_FAIL)
         {
@@ -192,7 +227,9 @@ urj_parse_stream (urj_log_level_t ll, urj_chain_t *chain, FILE *f)
         }
         urj_tap_chain_flush (chain);
     }
-    while (go != URJ_STATUS_MUST_QUIT && c != EOF);
+    while (go != URJ_STATUS_MUST_QUIT);
+
+    free (inputline);
 
     return go;
 }