From: Eugeniu Rosca <roscaeugeniu@gmail.com>
Date: Tue, 14 Aug 2018 00:43:08 +0000 (+0200)
Subject: common: avb_verify: Fix never-occurring avb_free(ops_data)
X-Git-Url: http://git.dujemihanovic.xyz/html/%7B%7B%20.RelPermalink%20%7D%7D?a=commitdiff_plain;h=47e41631bbb9348eac0ee7771b08d5ba7fdd6d5f;p=u-boot.git

common: avb_verify: Fix never-occurring avb_free(ops_data)

Cppcheck (v1.85) reports w/o this patch:

[common/avb_verify.c:738] -> [common/avb_verify.c:741]: (warning) \
  Either the condition 'ops' is redundant or there is possible null \
  pointer dereference: ops.

Signed-off-by: Eugeniu Rosca <erosca@de.adit-jv.com>
Reviewed-by: Igor Opaniuk <igor.opaniuk@linaro.org>
---

diff --git a/common/avb_verify.c b/common/avb_verify.c
index 0bc1c2369f..0234f98422 100644
--- a/common/avb_verify.c
+++ b/common/avb_verify.c
@@ -766,7 +766,7 @@ void avb_ops_free(AvbOps *ops)
 {
 	struct AvbOpsData *ops_data;
 
-	if (ops)
+	if (!ops)
 		return;
 
 	ops_data = ops->user_data;