dm: fix some possible memory leak

tianhuas · acrnsi · commit b96a3555bc7e · 2019-07-12T09:41:15.000+08:00
free memory allocated by strdup() Tracked-On: #3395 Signed-off-by: Tianhua Sun <tianhuax.s.sun@intel.com> Reviewed-by: Yonghua Huang <yonghua.huang@intel.com>
diff --git a/devicemodel/hw/block_if.c b/devicemodel/hw/block_if.c
@@ -660,7 +660,7 @@ blockif_open(const char *optstr, const char *ident)
 		if (size < DEV_BSIZE || (size & (DEV_BSIZE - 1))) {
 			WPRINTF(("%s size not corret, should be multiple of %d\n",
 						nopt, DEV_BSIZE));
-			return 0;
+			goto err;
 		}
 		psectsz = sbuf.st_blksize;
 	}

Original file line number	Diff line number	Diff line change
`@@ -660,7 +660,7 @@ blockif_open(const char optstr, const char ident)`
`660`	`660`	`if (size < DEV_BSIZE \|\| (size & (DEV_BSIZE - 1))) {`
`661`	`661`	`WPRINTF(("%s size not corret, should be multiple of %d\n",`
`662`	`662`	`nopt, DEV_BSIZE));`
`663`		`- return 0;`
	`663`	`+ goto err;`
`664`	`664`	`}`
`665`	`665`	`psectsz = sbuf.st_blksize;`
`666`	`666`	`}`