]> granicus.if.org Git - pdns/commitdiff
rec: Fix a memory leak in the negative cache unit tests
authorRemi Gacogne <remi.gacogne@powerdns.com>
Mon, 28 May 2018 16:05:52 +0000 (18:05 +0200)
committerRemi Gacogne <remi.gacogne@powerdns.com>
Mon, 28 May 2018 16:05:52 +0000 (18:05 +0200)
pdns/recursordist/test-negcache_cc.cc

index aba53fa26368f83b84c94552362127c8cd4d1d3c..2431794657e9d0031a4cd29fb9588145b727b81a 100644 (file)
@@ -383,6 +383,14 @@ BOOST_AUTO_TEST_CASE(test_dumpToFile) {
       BOOST_FAIL("Unable to read a line from the temp file");
     BOOST_CHECK_EQUAL(line, str);
   }
+
+  if (line != nullptr) {
+    /* getline() allocates a buffer then called with a nullptr,
+       then reallocates it when needed, but we need to free the
+       last allocation if any. */
+    free(line);
+  }
+
   fclose(fp);
 }