summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPatrick Georgi <pgeorgi@google.com>2019-09-02 17:16:09 +0200
committerCommit Bot <commit-bot@chromium.org>2019-09-04 11:53:29 +0000
commit519f9cdf0d50d87111e45227e9f0efea3686a8e1 (patch)
treee5885610704bff6cfc73d92f23eded66f296e4e6
parent43d6a36faa96b7a63a8779906272078dc9161e91 (diff)
downloadchrome-ec-519f9cdf0d50d87111e45227e9f0efea3686a8e1.tar.gz
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>
-rw-r--r--util/ectool.c2
1 files changed, 2 insertions, 0 deletions
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;
}