svn commit: r344235 - head/sys/contrib/libnv

Mariusz Zaborski oshogbo at FreeBSD.org
Sun Feb 17 18:26:28 UTC 2019


Author: oshogbo
Date: Sun Feb 17 18:26:27 2019
New Revision: 344235
URL: https://svnweb.freebsd.org/changeset/base/344235

Log:
  libnv: fix double free
  
  In r343986 we introduced a double free. The structure was already
  freed fixed in the r302966. This problem was introduced
  because the GitHub version was out of sync with the FreeBSD one.
  
  Submitted by:	Mindaugas Rasiukevicius <rmind at netbsd.org>
  MFC with:	r343986

Modified:
  head/sys/contrib/libnv/nvpair.c

Modified: head/sys/contrib/libnv/nvpair.c
==============================================================================
--- head/sys/contrib/libnv/nvpair.c	Sun Feb 17 17:47:08 2019	(r344234)
+++ head/sys/contrib/libnv/nvpair.c	Sun Feb 17 18:26:27 2019	(r344235)
@@ -229,14 +229,6 @@ nvpair_remove_nvlist_array(nvpair_t *nvp)
 	nvlarray = __DECONST(nvlist_t **,
 	    nvpair_get_nvlist_array(nvp, &count));
 	for (i = 0; i < count; i++) {
-		nvlist_t *nvl;
-		nvpair_t *nnvp;
-
-		nvl = nvlarray[i];
-		nnvp = nvlist_get_array_next_nvpair(nvl);
-		if (nnvp != NULL) {
-			nvpair_free_structure(nnvp);
-		}
 		nvlist_set_array_next(nvl, NULL);
 		nvlist_set_parent(nvl, NULL);
 	}


More information about the svn-src-all mailing list