]> arthur.barton.de Git - netdata.git/blobdiff - src/procfile.c
Merge pull request #1998 from ktsaou/master
[netdata.git] / src / procfile.c
old mode 100755 (executable)
new mode 100644 (file)
index 31b97ae..3a89e83
@@ -1,24 +1,4 @@
-#ifdef HAVE_CONFIG_H
-#include <config.h>
-#endif
-#include <stdio.h>
-#include <stdlib.h>
-#include <unistd.h>
-#include <errno.h>
-#include <fcntl.h>
-#include <string.h>
-#include <malloc.h>
-#include <ctype.h>
-#include <time.h>
-#include <sys/time.h>
-#include <sys/wait.h>
-
-#include <sys/types.h>
-#include <sys/stat.h>
-#include <sys/mman.h>
-
 #include "common.h"
-#include "log.h"
 #include "procfile.h"
 
 #define PF_PREFIX "PROCFILE"
@@ -31,369 +11,463 @@ int procfile_adaptive_initial_allocation = 0;
 
 // if adaptive allocation is set, these store the
 // max values we have seen so far
-uint32_t procfile_max_lines = PFLINES_INCREASE_STEP;
-uint32_t procfile_max_words = PFWORDS_INCREASE_STEP;
+size_t procfile_max_lines = PFLINES_INCREASE_STEP;
+size_t procfile_max_words = PFWORDS_INCREASE_STEP;
 size_t procfile_max_allocation = PROCFILE_INCREMENT_BUFFER;
 
-// ----------------------------------------------------------------------------
-// An array of words
 
+// ----------------------------------------------------------------------------
 
-pfwords *pfwords_add(pfwords *fw, char *str) {
-       // debug(D_PROCFILE, PF_PREFIX ":       adding word No %d: '%s'", fw->len, str);
+char *procfile_filename(procfile *ff) {
+    if(ff->filename[0]) return ff->filename;
 
-       if(unlikely(fw->len == fw->size)) {
-               // debug(D_PROCFILE, PF_PREFIX ":       expanding words");
+    char buffer[FILENAME_MAX + 1];
+    snprintfz(buffer, FILENAME_MAX, "/proc/self/fd/%d", ff->fd);
 
-               pfwords *new = realloc(fw, sizeof(pfwords) + (fw->size + PFWORDS_INCREASE_STEP) * sizeof(char *));
-               if(unlikely(!new)) {
-                       error(PF_PREFIX ":      failed to expand words");
-                       free(fw);
-                       return NULL;
-               }
-               fw = new;
-               fw->size += PFWORDS_INCREASE_STEP;
-       }
+    ssize_t l = readlink(buffer, ff->filename, FILENAME_MAX);
+    if(unlikely(l == -1))
+        snprintfz(ff->filename, FILENAME_MAX, "unknown filename for fd %d", ff->fd);
+    else
+        ff->filename[l] = '\0';
 
-       fw->words[fw->len++] = str;
+    // on non-linux systems, something like this will be needed
+    // fcntl(ff->fd, F_GETPATH, ff->filename)
 
-       return fw;
+    return ff->filename;
 }
 
-pfwords *pfwords_new(void) {
-       // debug(D_PROCFILE, PF_PREFIX ":       initializing words");
+// ----------------------------------------------------------------------------
+// An array of words
+
+static inline pfwords *pfwords_add(pfwords *fw, char *str) NEVERNULL;
+static inline pfwords *pfwords_add(pfwords *fw, char *str) {
+    // debug(D_PROCFILE, PF_PREFIX ":   adding word No %d: '%s'", fw->len, str);
+
+    if(unlikely(fw->len == fw->size)) {
+        // debug(D_PROCFILE, PF_PREFIX ":   expanding words");
 
-       uint32_t size = (procfile_adaptive_initial_allocation) ? procfile_max_words : PFWORDS_INCREASE_STEP;
+        fw = reallocz(fw, sizeof(pfwords) + (fw->size + PFWORDS_INCREASE_STEP) * sizeof(char *));
+        fw->size += PFWORDS_INCREASE_STEP;
+    }
 
-       pfwords *new = malloc(sizeof(pfwords) + size * sizeof(char *));
-       if(unlikely(!new)) return NULL;
+    fw->words[fw->len++] = str;
 
-       new->len = 0;
-       new->size = size;
-       return new;
+    return fw;
 }
 
-void pfwords_reset(pfwords *fw) {
-       // debug(D_PROCFILE, PF_PREFIX ":       reseting words");
-       fw->len = 0;
+static inline pfwords *pfwords_new(void) NEVERNULL;
+static inline pfwords *pfwords_new(void) {
+    // debug(D_PROCFILE, PF_PREFIX ":   initializing words");
+
+    size_t size = (procfile_adaptive_initial_allocation) ? procfile_max_words : PFWORDS_INCREASE_STEP;
+
+    pfwords *new = mallocz(sizeof(pfwords) + size * sizeof(char *));
+    new->len = 0;
+    new->size = size;
+    return new;
+}
+
+static inline void pfwords_reset(pfwords *fw) {
+    // debug(D_PROCFILE, PF_PREFIX ":   reseting words");
+    fw->len = 0;
 }
 
-void pfwords_free(pfwords *fw) {
-       // debug(D_PROCFILE, PF_PREFIX ":       freeing words");
+static inline void pfwords_free(pfwords *fw) {
+    // debug(D_PROCFILE, PF_PREFIX ":   freeing words");
 
-       free(fw);
+    freez(fw);
 }
 
 
 // ----------------------------------------------------------------------------
 // An array of lines
 
-pflines *pflines_add(pflines *fl, uint32_t first_word) {
-       // debug(D_PROCFILE, PF_PREFIX ":       adding line %d at word %d", fl->len, first_word);
+static inline pflines *pflines_add(pflines *fl, size_t first_word) NEVERNULL;
+static inline pflines *pflines_add(pflines *fl, size_t first_word) {
+    // debug(D_PROCFILE, PF_PREFIX ":   adding line %d at word %d", fl->len, first_word);
 
-       if(unlikely(fl->len == fl->size)) {
-               // debug(D_PROCFILE, PF_PREFIX ":       expanding lines");
+    if(unlikely(fl->len == fl->size)) {
+        // debug(D_PROCFILE, PF_PREFIX ":   expanding lines");
 
-               pflines *new = realloc(fl, sizeof(pflines) + (fl->size + PFLINES_INCREASE_STEP) * sizeof(ffline));
-               if(unlikely(!new)) {
-                       error(PF_PREFIX ":      failed to expand lines");
-                       free(fl);
-                       return NULL;
-               }
-               fl = new;
-               fl->size += PFLINES_INCREASE_STEP;
-       }
+        fl = reallocz(fl, sizeof(pflines) + (fl->size + PFLINES_INCREASE_STEP) * sizeof(ffline));
+        fl->size += PFLINES_INCREASE_STEP;
+    }
 
-       fl->lines[fl->len].words = 0;
-       fl->lines[fl->len++].first = first_word;
+    fl->lines[fl->len].words = 0;
+    fl->lines[fl->len++].first = first_word;
 
-       return fl;
+    return fl;
 }
 
-pflines *pflines_new(void) {
-       // debug(D_PROCFILE, PF_PREFIX ":       initializing lines");
+static inline pflines *pflines_new(void) NEVERNULL;
+static inline pflines *pflines_new(void) {
+    // debug(D_PROCFILE, PF_PREFIX ":   initializing lines");
 
-       uint32_t size = (unlikely(procfile_adaptive_initial_allocation)) ? procfile_max_words : PFLINES_INCREASE_STEP;
+    size_t size = (unlikely(procfile_adaptive_initial_allocation)) ? procfile_max_words : PFLINES_INCREASE_STEP;
 
-       pflines *new = malloc(sizeof(pflines) + size * sizeof(ffline));
-       if(unlikely(!new)) return NULL;
-
-       new->len = 0;
-       new->size = size;
-       return new;
+    pflines *new = mallocz(sizeof(pflines) + size * sizeof(ffline));
+    new->len = 0;
+    new->size = size;
+    return new;
 }
 
-void pflines_reset(pflines *fl) {
-       // debug(D_PROCFILE, PF_PREFIX ":       reseting lines");
+static inline void pflines_reset(pflines *fl) {
+    // debug(D_PROCFILE, PF_PREFIX ":   reseting lines");
 
-       fl->len = 0;
+    fl->len = 0;
 }
 
-void pflines_free(pflines *fl) {
-       // debug(D_PROCFILE, PF_PREFIX ":       freeing lines");
+static inline void pflines_free(pflines *fl) {
+    // debug(D_PROCFILE, PF_PREFIX ":   freeing lines");
 
-       free(fl);
+    freez(fl);
 }
 
 
 // ----------------------------------------------------------------------------
 // The procfile
 
-#define PF_CHAR_IS_SEPARATOR   ' '
-#define PF_CHAR_IS_NEWLINE             'N'
-#define PF_CHAR_IS_WORD                        'W'
-
 void procfile_close(procfile *ff) {
-       debug(D_PROCFILE, PF_PREFIX ": Closing file '%s'", ff->filename);
+    if(unlikely(!ff)) return;
 
-       if(likely(ff->lines)) pflines_free(ff->lines);
-       if(likely(ff->words)) pfwords_free(ff->words);
+    debug(D_PROCFILE, PF_PREFIX ": Closing file '%s'", procfile_filename(ff));
 
-       if(likely(ff->fd != -1)) close(ff->fd);
-       free(ff);
-}
+    if(likely(ff->lines)) pflines_free(ff->lines);
+    if(likely(ff->words)) pfwords_free(ff->words);
 
-procfile *procfile_parser(procfile *ff) {
-       debug(D_PROCFILE, PF_PREFIX ": Parsing file '%s'", ff->filename);
-
-       char *s = ff->data, *e = ff->data, *t = ff->data;
-       uint32_t l = 0, w = 0;
-       e += ff->len;
-       
-       ff->lines = pflines_add(ff->lines, w);
-       if(unlikely(!ff->lines)) goto cleanup;
+    if(likely(ff->fd != -1)) close(ff->fd);
+    freez(ff);
+}
 
-       while(likely(s < e)) {
-               switch(ff->separators[(int)(*s)]) {
-                       case PF_CHAR_IS_SEPARATOR:
-                               if(likely(s == t)) {
-                                       // skip all leading white spaces
-                                       t = ++s;
-                                       continue;
-                               }
+static inline void procfile_parser(procfile *ff) {
+    // debug(D_PROCFILE, PF_PREFIX ": Parsing file '%s'", ff->filename);
+
+    char  *s = ff->data                 // our current position
+        , *e = &ff->data[ff->len]       // the terminating null
+        , *t = ff->data;                // the first character of a quoted or a parenthesized string
+
+                                        // the look up array to find our type of character
+    PF_CHAR_TYPE *separators = ff->separators;
+
+    char quote = 0;                     // the quote character - only when in quoted string
+
+    size_t
+          l = 0                         // counts the number of lines we added
+        , w = 0                         // counts the number of words we added
+        , opened = 0;                   // counts the number of open parenthesis
+
+    ff->lines = pflines_add(ff->lines, w);
+
+    while(likely(s < e)) {
+        // we are not at the end
+        PF_CHAR_TYPE ct = separators[(unsigned char)(*s)];
+
+        // this is faster than a switch()
+        if(likely(ct == PF_CHAR_IS_WORD)) {
+            s++;
+        }
+        else if(likely(ct == PF_CHAR_IS_SEPARATOR)) {
+            if(unlikely(quote || opened)) {
+                // we are inside a quote
+                s++;
+                continue;
+            }
+
+            if(unlikely(s == t)) {
+                // skip all leading white spaces
+                t = ++s;
+                continue;
+            }
+
+            // end of word
+            *s = '\0';
+
+            ff->words = pfwords_add(ff->words, t);
+            ff->lines->lines[l].words++;
+            w++;
+
+            t = ++s;
+        }
+        else if(likely(ct == PF_CHAR_IS_NEWLINE)) {
+            // end of line
+            *s = '\0';
+
+            ff->words = pfwords_add(ff->words, t);
+            ff->lines->lines[l].words++;
+            w++;
+
+            // debug(D_PROCFILE, PF_PREFIX ":   ended line %d with %d words", l, ff->lines->lines[l].words);
+
+            ff->lines = pflines_add(ff->lines, w);
+            l++;
+
+            t = ++s;
+        }
+        else if(likely(ct == PF_CHAR_IS_QUOTE)) {
+            if(unlikely(!quote && s == t)) {
+                // quote opened at the beginning
+                quote = *s;
+                t = ++s;
+            }
+            else if(unlikely(quote && quote == *s)) {
+                // quote closed
+                quote = 0;
+
+                *s = '\0';
+                ff->words = pfwords_add(ff->words, t);
+                ff->lines->lines[l].words++;
+                w++;
+
+                t = ++s;
+            }
+            else
+                s++;
+        }
+        else if(likely(ct == PF_CHAR_IS_OPEN)) {
+            if(s == t) {
+                opened++;
+                t = ++s;
+            }
+            else if(opened) {
+                opened++;
+                s++;
+            }
+            else
+                s++;
+        }
+        else if(likely(ct == PF_CHAR_IS_CLOSE)) {
+            if(opened) {
+                opened--;
+
+                if(!opened) {
+                    *s = '\0';
+                    ff->words = pfwords_add(ff->words, t);
+                    ff->lines->lines[l].words++;
+                    w++;
+
+                    t = ++s;
+                }
+                else
+                    s++;
+            }
+            else
+                s++;
+        }
+        else
+            fatal("Internal Error: procfile_readall() does not handle all the cases.");
+    }
+
+    if(likely(s > t && t < e)) {
+        // the last word
+        if(likely(ff->len < ff->size))
+            *s = '\0';
+        else {
+            // we are going to loose the last byte
+            ff->data[ff->size - 1] = '\0';
+        }
+
+        ff->words = pfwords_add(ff->words, t);
+        ff->lines->lines[l].words++;
+    }
+}
 
-                               // end of word
-                               *s = '\0';
+procfile *procfile_readall(procfile *ff) {
+    // debug(D_PROCFILE, PF_PREFIX ": Reading file '%s'.", ff->filename);
+
+    ff->len = 0;    // zero the used size
+    ssize_t r = 1;  // read at least once
+    while(r > 0) {
+        ssize_t s = ff->len;
+        ssize_t x = ff->size - s;
+
+        if(unlikely(!x)) {
+            debug(D_PROCFILE, PF_PREFIX ": Expanding data buffer for file '%s'.", procfile_filename(ff));
+            ff = reallocz(ff, sizeof(procfile) + ff->size + PROCFILE_INCREMENT_BUFFER);
+            ff->size += PROCFILE_INCREMENT_BUFFER;
+        }
+
+        debug(D_PROCFILE, "Reading file '%s', from position %zd with length %zd", procfile_filename(ff), s, (ssize_t)(ff->size - s));
+        r = read(ff->fd, &ff->data[s], ff->size - s);
+        if(unlikely(r == -1)) {
+            if(unlikely(!(ff->flags & PROCFILE_FLAG_NO_ERROR_ON_FILE_IO))) error(PF_PREFIX ": Cannot read from file '%s'", procfile_filename(ff));
+            procfile_close(ff);
+            return NULL;
+        }
+
+        ff->len += r;
+    }
+
+    // debug(D_PROCFILE, "Rewinding file '%s'", ff->filename);
+    if(unlikely(lseek(ff->fd, 0, SEEK_SET) == -1)) {
+        if(unlikely(!(ff->flags & PROCFILE_FLAG_NO_ERROR_ON_FILE_IO))) error(PF_PREFIX ": Cannot rewind on file '%s'.", procfile_filename(ff));
+        procfile_close(ff);
+        return NULL;
+    }
+
+    pflines_reset(ff->lines);
+    pfwords_reset(ff->words);
+    procfile_parser(ff);
+
+    if(unlikely(procfile_adaptive_initial_allocation)) {
+        if(unlikely(ff->len > procfile_max_allocation)) procfile_max_allocation = ff->len;
+        if(unlikely(ff->lines->len > procfile_max_lines)) procfile_max_lines = ff->lines->len;
+        if(unlikely(ff->words->len > procfile_max_words)) procfile_max_words = ff->words->len;
+    }
+
+    // debug(D_PROCFILE, "File '%s' updated.", ff->filename);
+    return ff;
+}
 
-                               ff->words = pfwords_add(ff->words, t);
-                               if(unlikely(!ff->words)) goto cleanup;
+static inline void procfile_set_separators(procfile *ff, const char *separators) {
+    static PF_CHAR_TYPE def[256];
+    static char initilized = 0;
+
+    if(unlikely(!initilized)) {
+        // this is thread safe
+        // if initialized is zero, multiple threads may be executing
+        // this code at the same time, setting in def[] the exact same values
+        int i = 256;
+        while(i--) {
+            if(unlikely(i == '\n' || i == '\r'))
+                def[i] = PF_CHAR_IS_NEWLINE;
+
+            else if(unlikely(isspace(i) || !isprint(i)))
+                def[i] = PF_CHAR_IS_SEPARATOR;
+
+            else
+                def[i] = PF_CHAR_IS_WORD;
+        }
+
+        initilized = 1;
+    }
+
+    // copy the default
+    PF_CHAR_TYPE *ffs = ff->separators, *ffd = def, *ffe = &def[256];
+    while(ffd != ffe)
+        *ffs++ = *ffd++;
+
+    // set the separators
+    if(unlikely(!separators))
+        separators = " \t=|";
+
+    ffs = ff->separators;
+    const char *s = separators;
+    while(*s)
+        ffs[(int)*s++] = PF_CHAR_IS_SEPARATOR;
+}
 
-                               ff->lines->lines[l].words++;
-                               w++;
+void procfile_set_quotes(procfile *ff, const char *quotes) {
+    PF_CHAR_TYPE *ffs = ff->separators;
 
-                               t = ++s;
-                               continue;
+    // remove all quotes
+    int i = 256;
+    while(i--)
+        if(unlikely(ffs[i] == PF_CHAR_IS_QUOTE))
+            ffs[i] = PF_CHAR_IS_WORD;
 
-                       case PF_CHAR_IS_NEWLINE:
-                               // end of line
-                               *s = '\0';
+    // if nothing given, return
+    if(unlikely(!quotes || !*quotes))
+        return;
 
-                               ff->words = pfwords_add(ff->words, t);
-                               if(unlikely(!ff->words)) goto cleanup;
+    // set the quotes
+    const char *s = quotes;
+    while(*s)
+        ffs[(int)*s++] = PF_CHAR_IS_QUOTE;
+}
 
-                               ff->lines->lines[l].words++;
-                               w++;
+void procfile_set_open_close(procfile *ff, const char *open, const char *close) {
+    PF_CHAR_TYPE *ffs = ff->separators;
 
-                               // debug(D_PROCFILE, PF_PREFIX ":       ended line %d with %d words", l, ff->lines->lines[l].words);
+    // remove all open/close
+    int i = 256;
+    while(i--)
+        if(unlikely(ffs[i] == PF_CHAR_IS_OPEN || ffs[i] == PF_CHAR_IS_CLOSE))
+            ffs[i] = PF_CHAR_IS_WORD;
 
-                               ff->lines = pflines_add(ff->lines, w);
-                               if(unlikely(!ff->lines)) goto cleanup;
-                               l++;
+    // if nothing given, return
+    if(unlikely(!open || !*open || !close || !*close))
+        return;
 
-                               t = ++s;
-                               continue;
+    // set the openings
+    const char *s = open;
+    while(*s)
+        ffs[(int)*s++] = PF_CHAR_IS_OPEN;
 
-                       default:
-                               s++;
-                               continue;
-               }
-       }
+    // set the closings
+    s = close;
+    while(*s)
+        ffs[(int)*s++] = PF_CHAR_IS_CLOSE;
+}
 
-       if(likely(s != t)) {
-               // the last word
-               if(likely(ff->len < ff->size)) *s = '\0';
-               else {
-                       // we are going to loose the last byte
-                       ff->data[ff->size - 1] = '\0';
-               }
+procfile *procfile_open(const char *filename, const char *separators, uint32_t flags) {
+    debug(D_PROCFILE, PF_PREFIX ": Opening file '%s'", filename);
 
-               ff->words = pfwords_add(ff->words, t);
-               if(unlikely(!ff->words)) goto cleanup;
+    int fd = open(filename, O_RDONLY, 0666);
+    if(unlikely(fd == -1)) {
+        if(unlikely(!(flags & PROCFILE_FLAG_NO_ERROR_ON_FILE_IO))) error(PF_PREFIX ": Cannot open file '%s'", filename);
+        return NULL;
+    }
 
-               ff->lines->lines[l].words++;
-               w++;
-       }
+    size_t size = (unlikely(procfile_adaptive_initial_allocation)) ? procfile_max_allocation : PROCFILE_INCREMENT_BUFFER;
+    procfile *ff = mallocz(sizeof(procfile) + size);
 
-       return ff;
+    //strncpyz(ff->filename, filename, FILENAME_MAX);
+    ff->filename[0] = '\0';
 
-cleanup:
-       error(PF_PREFIX ": Failed to parse file '%s'", ff->filename);
-       procfile_close(ff);
-       return NULL;
-}
+    ff->fd = fd;
+    ff->size = size;
+    ff->len = 0;
+    ff->flags = flags;
 
-procfile *procfile_readall(procfile *ff) {
-       debug(D_PROCFILE, PF_PREFIX ": Reading file '%s'.", ff->filename);
-
-       ssize_t s = 0, r = ff->size, x = ff->size;
-       ff->len = 0;
-
-       while(likely(r == x)) {
-               if(s) {
-                       debug(D_PROCFILE, PF_PREFIX ": Expanding data buffer for file '%s'.", ff->filename);
-
-                       procfile *new = realloc(ff, sizeof(procfile) + ff->size + PROCFILE_INCREMENT_BUFFER);
-                       if(unlikely(!new)) {
-                               error(PF_PREFIX ": Cannot allocate memory for file '%s'", ff->filename);
-                               procfile_close(ff);
-                               return NULL;
-                       }
-                       ff = new;
-                       ff->size += PROCFILE_INCREMENT_BUFFER;
-                       x = PROCFILE_INCREMENT_BUFFER;
-               }
-
-               debug(D_PROCFILE, "Reading file '%s', from position %ld with length %ld", ff->filename, s, ff->size - s);
-               r = read(ff->fd, &ff->data[s], ff->size - s);
-               if(unlikely(r == -1)) {
-                       if(unlikely(!(ff->flags & PROCFILE_FLAG_NO_ERROR_ON_FILE_IO))) error(PF_PREFIX ": Cannot read from file '%s'", ff->filename);
-                       procfile_close(ff);
-                       return NULL;
-               }
-
-               ff->len += r;
-               s = ff->len;
-       }
-
-       debug(D_PROCFILE, "Rewinding file '%s'", ff->filename);
-       if(unlikely(lseek(ff->fd, 0, SEEK_SET) == -1)) {
-               if(unlikely(!(ff->flags & PROCFILE_FLAG_NO_ERROR_ON_FILE_IO))) error(PF_PREFIX ": Cannot rewind on file '%s'.", ff->filename);
-               procfile_close(ff);
-               return NULL;
-       }
-
-       pflines_reset(ff->lines);
-       pfwords_reset(ff->words);
-
-       ff = procfile_parser(ff);
-
-       if(unlikely(procfile_adaptive_initial_allocation)) {
-               if(unlikely(ff->len > procfile_max_allocation)) procfile_max_allocation = ff->len;
-               if(unlikely(ff->lines->len > procfile_max_lines)) procfile_max_lines = ff->lines->len;
-               if(unlikely(ff->words->len > procfile_max_words)) procfile_max_words = ff->words->len;
-       }
-
-       debug(D_PROCFILE, "File '%s' updated.", ff->filename);
-       return ff;
-}
+    ff->lines = pflines_new();
+    ff->words = pfwords_new();
 
-static void procfile_set_separators(procfile *ff, const char *separators) {
-       static char def[256] = { [0 ... 255] = 0 };
-       int i;
-
-       if(unlikely(!def[255])) {
-               // this is thread safe
-               // we check that the last byte is non-zero
-               // if it is zero, multiple threads may be executing this at the same time
-               // setting in def[] the exact same values
-               for(i = 0; likely(i < 256) ;i++) {
-                       if(unlikely(i == '\n' || i == '\r')) def[i] = PF_CHAR_IS_NEWLINE;
-                       else if(unlikely(isspace(i) || !isprint(i))) def[i] = PF_CHAR_IS_SEPARATOR;
-                       else def[i] = PF_CHAR_IS_WORD;
-               }
-       }
-
-       // copy the default
-       char *ffs = ff->separators, *ffd = def, *ffe = &def[256];
-       while(likely(ffd != ffe)) *ffs++ = *ffd++;
-
-       // set the separators
-       if(unlikely(!separators)) separators = " \t=|";
-       ffs = ff->separators;
-       const char *s = separators;
-       while(likely(*s)) ffs[(int)*s++] = PF_CHAR_IS_SEPARATOR;
-}
+    procfile_set_separators(ff, separators);
 
-procfile *procfile_open(const char *filename, const char *separators, uint32_t flags) {
-       debug(D_PROCFILE, PF_PREFIX ": Opening file '%s'", filename);
-
-       int fd = open(filename, O_RDONLY, 0666);
-       if(unlikely(fd == -1)) {
-               if(unlikely(!(flags & PROCFILE_FLAG_NO_ERROR_ON_FILE_IO))) error(PF_PREFIX ": Cannot open file '%s'", filename);
-               return NULL;
-       }
-
-       size_t size = (unlikely(procfile_adaptive_initial_allocation)) ? procfile_max_allocation : PROCFILE_INCREMENT_BUFFER;
-       procfile *ff = malloc(sizeof(procfile) + size);
-       if(unlikely(!ff)) {
-               error(PF_PREFIX ": Cannot allocate memory for file '%s'", filename);
-               close(fd);
-               return NULL;
-       }
-
-       strncpy(ff->filename, filename, FILENAME_MAX);
-       ff->filename[FILENAME_MAX] = '\0';
-
-       ff->fd = fd;
-       ff->size = size;
-       ff->len = 0;
-       ff->flags = flags;
-
-       ff->lines = pflines_new();
-       ff->words = pfwords_new();
-
-       if(unlikely(!ff->lines || !ff->words)) {
-               error(PF_PREFIX ": Cannot initialize parser for file '%s'", filename);
-               procfile_close(ff);
-               return NULL;
-       }
-
-       procfile_set_separators(ff, separators);
-
-       debug(D_PROCFILE, "File '%s' opened.", filename);
-       return ff;
+    debug(D_PROCFILE, "File '%s' opened.", filename);
+    return ff;
 }
 
 procfile *procfile_reopen(procfile *ff, const char *filename, const char *separators, uint32_t flags) {
-       if(unlikely(!ff)) return procfile_open(filename, separators, flags);
+    if(unlikely(!ff)) return procfile_open(filename, separators, flags);
 
-       if(likely(ff->fd != -1)) close(ff->fd);
+    if(likely(ff->fd != -1)) close(ff->fd);
 
-       ff->fd = open(filename, O_RDONLY, 0666);
-       if(unlikely(ff->fd == -1)) {
-               procfile_close(ff);
-               return NULL;
-       }
+    ff->fd = open(filename, O_RDONLY, 0666);
+    if(unlikely(ff->fd == -1)) {
+        procfile_close(ff);
+        return NULL;
+    }
 
-       strncpy(ff->filename, filename, FILENAME_MAX);
-       ff->filename[FILENAME_MAX] = '\0';
+    //strncpyz(ff->filename, filename, FILENAME_MAX);
+    ff->filename[0] = '\0';
 
-       ff->flags = flags;
+    ff->flags = flags;
 
-       // do not do the separators again if NULL is given
-       if(likely(separators)) procfile_set_separators(ff, separators);
+    // do not do the separators again if NULL is given
+    if(likely(separators)) procfile_set_separators(ff, separators);
 
-       return ff;
+    return ff;
 }
 
 // ----------------------------------------------------------------------------
 // example parsing of procfile data
 
 void procfile_print(procfile *ff) {
-       uint32_t lines = procfile_lines(ff), l;
-       uint32_t words, w;
-       char *s;
+    size_t lines = procfile_lines(ff), l;
+    char *s;
 
-       debug(D_PROCFILE, "File '%s' with %d lines and %d words", ff->filename, ff->lines->len, ff->words->len);
+    debug(D_PROCFILE, "File '%s' with %zu lines and %zu words", procfile_filename(ff), ff->lines->len, ff->words->len);
 
-       for(l = 0; likely(l < lines) ;l++) {
-               words = procfile_linewords(ff, l);
+    for(l = 0; likely(l < lines) ;l++) {
+        size_t words = procfile_linewords(ff, l);
 
-               debug(D_PROCFILE, "     line %d starts at word %d and has %d words", l, ff->lines->lines[l].first, ff->lines->lines[l].words);
+        debug(D_PROCFILE, " line %zu starts at word %zu and has %zu words", l, ff->lines->lines[l].first, ff->lines->lines[l].words);
 
-               for(w = 0; likely(w < words) ;w++) {
-                       s = procfile_lineword(ff, l, w);
-                       debug(D_PROCFILE, "             [%d.%d] '%s'", l, w, s);
-               }
-       }
+        size_t w;
+        for(w = 0; likely(w < words) ;w++) {
+            s = procfile_lineword(ff, l, w);
+            debug(D_PROCFILE, "     [%zu.%zu] '%s'", l, w, s);
+        }
+    }
 }