MonetDB: Jun2016 - Don't use value after it was freed.

[Sjoerd Mullender]

Changeset: 7d4cbc987d83 for MonetDB
URL: http://dev.monetdb.org/hg/MonetDB?cmd=changeset;node=7d4cbc987d83
Modified Files:
        tools/merovingian/daemon/merovingian.c
Branch: Jun2016
Log Message:

Don't use value after it was freed.


diffs (17 lines):

diff --git a/tools/merovingian/daemon/merovingian.c 
b/tools/merovingian/daemon/merovingian.c
--- a/tools/merovingian/daemon/merovingian.c
+++ b/tools/merovingian/daemon/merovingian.c
@@ -715,7 +715,12 @@ main(int argc, char *argv[])
                                                "expected '%s', disabling 
passphrase\n",
                                                h, MONETDB5_PASSWDHASH);
                        } else {
-                               setConfVal(kv, p + 1);
+                               /* p points into kv->val which gets freed 
before p
+                                * gets copied inside setConfVal, hence we need 
to
+                                * make a temporary copy */
+                               p = strdup(p + 1);
+                               setConfVal(kv, p);
+                               free(p);
                        }
                }
        }
_______________________________________________
checkin-list mailing list
checkin-list@monetdb.org
https://www.monetdb.org/mailman/listinfo/checkin-list