svn commit: r334016 - head/usr.sbin/bhyve

Marcelo Araujo araujo at FreeBSD.org
Tue May 22 04:08:09 UTC 2018


Author: araujo
Date: Tue May 22 04:08:08 2018
New Revision: 334016
URL: https://svnweb.freebsd.org/changeset/base/334016

Log:
  We must free the variable str.
  
  Spotted by:	clang's static analyzer
  Submitted by:	Tom Rix <trix_juniper.net>
  Reviewed by:	grehan
  MFC after:	4 weeks
  Sponsored by:	iXsystems Inc.
  Differential Revision:	https://reviews.freebsd.org/D10009

Modified:
  head/usr.sbin/bhyve/pci_emul.c

Modified: head/usr.sbin/bhyve/pci_emul.c
==============================================================================
--- head/usr.sbin/bhyve/pci_emul.c	Tue May 22 03:57:32 2018	(r334015)
+++ head/usr.sbin/bhyve/pci_emul.c	Tue May 22 04:08:08 2018	(r334016)
@@ -231,8 +231,7 @@ pci_parse_slot(char *opt)
 	si->si_funcs[fnum].fi_param = config;
 
 done:
-	if (error)
-		free(str);
+	free(str);
 
 	return (error);
 }


More information about the svn-src-all mailing list