From 519f9cdf0d50d87111e45227e9f0efea3686a8e1 Mon Sep 17 00:00:00 2001
From: Patrick Georgi <pgeorgi@google.com>
Date: Mon, 2 Sep 2019 17:16:09 +0200
Subject: util/ectool: Free memory before giving up

Found by Coverity Scan #203652, #203653

BUG=none
BRANCH=none
TEST=none

Change-Id: I195773c42d4d0dfa35836e3e442cca15125ac9ea
Signed-off-by: Patrick Georgi <pgeorgi@google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1781471
Tested-by: Patrick Georgi <pgeorgi@chromium.org>
Commit-Queue: Patrick Georgi <pgeorgi@chromium.org>
Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org>
---
 util/ectool.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/util/ectool.c b/util/ectool.c
index 4ea3b94815..d20e0acaad 100644
--- a/util/ectool.c
+++ b/util/ectool.c
@@ -8657,6 +8657,8 @@ int cmd_tp_frame_get(int argc, char* argv[])
 
 	if (data == NULL || r == NULL) {
 		fprintf(stderr, "Couldn't allocate memory.\n");
+		free(r);
+		free(data);
 		return EC_ERROR_UNKNOWN;
 	}
 
-- 
cgit v1.2.1