cfgfile: fix parsing of long fields
authorJoseph Richard <joseph.richard@windriver.com>
Fri, 31 Mar 2017 13:52:02 +0000 (09:52 -0400)
committerThomas Monjalon <thomas.monjalon@6wind.com>
Tue, 4 Apr 2017 14:32:06 +0000 (16:32 +0200)
When parsing a ini file with a "key = value" line that has both "key" and
"value" sized to the maximum allowed length causes a parsing failure.  The
internal "buffer" variable should be sized at least as large as the maximum
for both fields.  This commit updates the local array to be sized to hold
the max name, max value, " = ", and the nul terminator.

Signed-off-by: Allain Legacy <allain.legacy@windriver.com>
Acked-by: Keith Wiles <keith.wiles@intel.com>
Acked-by: Cristian Dumitrescu <cristian.dumitrescu@intel.com>
lib/librte_cfgfile/rte_cfgfile.c

index e4a3885..4ef7dec 100644 (file)
@@ -148,7 +148,7 @@ rte_cfgfile_load_with_params(const char *filename, int flags,
        int allocated_entries = 0;
        int curr_section = -1;
        int curr_entry = -1;
-       char buffer[256] = {0};
+       char buffer[CFG_NAME_LEN + CFG_VALUE_LEN + 4] = {0};
        int lineno = 0;
        struct rte_cfgfile *cfg = NULL;