]> granicus.if.org Git - musl/commitdiff
use dynamic buffer for getmntent
authorNatanael Copa <ncopa@alpinelinux.org>
Thu, 8 Sep 2016 17:07:31 +0000 (19:07 +0200)
committerRich Felker <dalias@aerifal.cx>
Thu, 20 Oct 2016 05:20:27 +0000 (01:20 -0400)
overlayfs may have fairly long lines so we use getline to allocate a
buffer dynamically. The buffer will be allocated on first use, expand as
needed, but will never be free'ed.

Downstream bug: http://bugs.alpinelinux.org/issues/5703

Signed-off-by: Natanael Copa <ncopa@alpinelinux.org>
src/misc/mntent.c

index a16d65254d5042df3de28ff29dc719ccecc76bc5..eabb8200bf94d2b450971ff54d3951d365b939f9 100644 (file)
@@ -3,6 +3,11 @@
 #include <mntent.h>
 #include <errno.h>
 
+static char *internal_buf;
+static size_t internal_bufsize;
+
+#define SENTINEL (char *)&internal_buf
+
 FILE *setmntent(const char *name, const char *mode)
 {
        return fopen(name, mode);
@@ -16,13 +21,18 @@ int endmntent(FILE *f)
 
 struct mntent *getmntent_r(FILE *f, struct mntent *mnt, char *linebuf, int buflen)
 {
-       int cnt, n[8];
+       int cnt, n[8], use_internal = (linebuf == SENTINEL);
 
        mnt->mnt_freq = 0;
        mnt->mnt_passno = 0;
 
        do {
-               fgets(linebuf, buflen, f);
+               if (use_internal) {
+                       getline(&internal_buf, &internal_bufsize, f);
+                       linebuf = internal_buf;
+               } else {
+                       fgets(linebuf, buflen, f);
+               }
                if (feof(f) || ferror(f)) return 0;
                if (!strchr(linebuf, '\n')) {
                        fscanf(f, "%*[^\n]%*[\n]");
@@ -49,9 +59,8 @@ struct mntent *getmntent_r(FILE *f, struct mntent *mnt, char *linebuf, int bufle
 
 struct mntent *getmntent(FILE *f)
 {
-       static char linebuf[256];
        static struct mntent mnt;
-       return getmntent_r(f, &mnt, linebuf, sizeof linebuf);
+       return getmntent_r(f, &mnt, SENTINEL, 0);
 }
 
 int addmntent(FILE *f, const struct mntent *mnt)