]> git.ipfire.org Git - thirdparty/lldpd.git/commitdiff
compat: use getline() instead of fgetln()
authorVincent Bernat <vincent@bernat.im>
Sat, 12 Dec 2015 21:47:41 +0000 (22:47 +0100)
committerVincent Bernat <vincent@bernat.im>
Sat, 12 Dec 2015 21:47:41 +0000 (22:47 +0100)
getline() is POSIX.1-2008 while fgetln() is BSD. Write a simple
implementation from the NetBSD one for too old OSes.

configure.ac
src/client/lldpcli.c
src/compat/compat.h
src/compat/fgetln.c [deleted file]
src/compat/getline.c [new file with mode: 0644]

index 52dbc1b89d6f94b3a8a1b3e1d94ac7bca2f4e835..9743404e7fb229978bad30484bb2554be8fadeb0 100644 (file)
@@ -142,7 +142,7 @@ AC_CHECK_FUNCS([setproctitle_init])
 AC_REPLACE_FUNCS([strlcpy
                   strnlen
                   strndup
-                  fgetln
+                  getline
                   asprintf
                   vsyslog
                   daemon])
index e38256a037e6e563d2591221e8da92f05e0c7c09..f04bdadc2f1210efb879697274f042e6a47e2c3e 100644 (file)
@@ -508,16 +508,17 @@ main(int argc, char *argv[])
                log_debug("lldpctl", "process: %s", first->name);
                FILE *file = fopen(first->name, "r");
                if (file) {
-                       size_t len;
+                       size_t n;
+                       ssize_t len;
                        char *line;
-                       while ((line = fgetln(file, &len))) {
-                               line = strndup(line, len);
+                       while (line = NULL, len = 0, (len = getline(&line, &n, file)) > 0) {
                                if (line[len - 1] == '\n') {
                                        line[len - 1] = '\0';
                                        parse_and_exec(conn, fmt, line);
                                }
                                free(line);
                        }
+                       free(line);
                        fclose(file);
                } else {
                        log_warn("lldpctl", "unable to open %s",
index 5232c530e67ae5165eb7aba03c93ea29ac8345fd..982f9f1bfdbdb0034b53b45ea83028dba8caba46 100644 (file)
@@ -68,8 +68,8 @@ size_t        strnlen(const char *, size_t);
 char   *strndup(const char *, size_t);
 #endif
 
-#if !HAVE_FGETLN
-char *fgetln(FILE *, size_t *);
+#if !HAVE_GETLINE
+ssize_t getline(char **, size_t *, FILE *);
 #endif
 
 #if !HAVE_SETPROCTITLE
diff --git a/src/compat/fgetln.c b/src/compat/fgetln.c
deleted file mode 100644 (file)
index 6590875..0000000
+++ /dev/null
@@ -1,57 +0,0 @@
-/* -*- mode: c; c-file-style: "openbsd" -*- */
-/*
- * Copyright (c) 2001 Marc Espie.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions
- * are met:
- * 1. Redistributions of source code must retain the above copyright
- *    notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- *    notice, this list of conditions and the following disclaimer in the
- *    documentation and/or other materials provided with the distribution.
- *
- * THIS SOFTWARE IS PROVIDED BY THE OPENBSD PROJECT AND CONTRIBUTORS
- * ``AS IS'' AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
- * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
- * A PARTICULAR PURPOSE ARE DISCLAIMED.  IN NO EVENT SHALL THE OPENBSD
- * PROJECT OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
- * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
- * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
- * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
- * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
- * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
- * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-#include <stdlib.h>
-#include <stdio.h>
-#include <sys/types.h>
-#include <string.h>
-
-char *
-fgetln(FILE *stream, size_t *len)
-{
-       static char *buffer = NULL;
-       static size_t buflen = 0;
-       char *newbuf;
-
-       if (buflen == 0) {
-               buflen = 512;
-               if ((buffer = malloc(buflen+1)) == NULL)
-                       return NULL;
-       }
-       if (fgets(buffer, buflen+1, stream) == NULL)
-               return NULL;
-       *len = strlen(buffer);
-       while (*len == buflen && buffer[*len-1] != '\n') {
-               if ((newbuf = realloc(buffer, 2*buflen + 1)) == NULL)
-                       return NULL;
-               buffer = newbuf;
-               if (fgets(buffer + buflen, buflen + 1, stream) == NULL)
-                       return NULL;
-               *len += strlen(buffer + buflen);
-               buflen *= 2;
-       }
-       return buffer;
-}
diff --git a/src/compat/getline.c b/src/compat/getline.c
new file mode 100644 (file)
index 0000000..2e11f23
--- /dev/null
@@ -0,0 +1,112 @@
+/* -*- mode: c; c-file-style: "openbsd" -*- */
+/*
+ * Copyright (c) 2009 The NetBSD Foundation, Inc.
+ *
+ * This code is derived from software contributed to The NetBSD Foundation
+ * by Roy Marples.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ * 1. Redistributions of source code must retain the above copyright
+ *    notice, this list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above copyright
+ *    notice, this list of conditions and the following disclaimer in the
+ *    documentation and/or other materials provided with the distribution.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE AUTHOR ``AS IS'' AND ANY EXPRESS OR
+ * IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES
+ * OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED.
+ * IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR ANY DIRECT, INDIRECT,
+ * INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT
+ * NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+ * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+ * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
+ * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ */
+
+/* Several modifications to make the code more portable (and less robust and far less efficient) */
+
+#include <stdio.h>
+#include <errno.h>
+#include <limits.h>
+#include <stdint.h>
+#include <stdio.h>
+#include <stdlib.h>
+#include <string.h>
+
+#define MINBUF 128
+
+static ssize_t
+___getdelim(char **buf, size_t *buflen,
+    int sep, FILE *fp)
+{
+       int p;
+       size_t len = 0, newlen;
+       char *newb;
+
+       if (buf == NULL || buflen == NULL) {
+               errno = EINVAL;
+               return -1;
+       }
+
+       /* If buf is NULL, we have to assume a size of zero */
+       if (*buf == NULL)
+               *buflen = 0;
+
+       do {
+               p = fgetc(fp);
+               if (ferror(fp))
+                       return -1;
+               if (p == EOF)
+                       break;
+
+               /* Ensure we can handle it */
+               if (len > SSIZE_MAX) {
+                       errno = EOVERFLOW;
+                       return -1;
+               }
+               newlen = len + 2; /* reserve space for NUL terminator */
+               if (newlen > *buflen) {
+                       if (newlen < MINBUF)
+                               newlen = MINBUF;
+#define powerof2(x) ((((x)-1)&(x))==0)
+                       if (!powerof2(newlen)) {
+                               /* Grow the buffer to the next power of 2 */
+                               newlen--;
+                               newlen |= newlen >> 1;
+                               newlen |= newlen >> 2;
+                               newlen |= newlen >> 4;
+                               newlen |= newlen >> 8;
+                               newlen |= newlen >> 16;
+#if SIZE_MAX > 0xffffffffU
+                               newlen |= newlen >> 32;
+#endif
+                               newlen++;
+                       }
+
+                       newb = realloc(*buf, newlen);
+                       if (newb == NULL)
+                               return -1;
+                       *buf = newb;
+                       *buflen = newlen;
+               }
+
+               (*buf)[len++] = p;
+       } while (p != sep);
+
+       /* POSIX demands we return -1 on EOF. */
+       if (len == 0)
+               return -1;
+
+       if (*buf != NULL)
+               (*buf)[len] = '\0';
+       return (ssize_t)len;
+}
+
+ssize_t
+getline(char **buf, size_t *buflen, FILE *fp)
+{
+       return ___getdelim(buf, buflen, '\n', fp);
+}