diff options
author | Randall Spangler <rspangler@chromium.org> | 2016-09-28 11:54:20 -0700 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2016-10-01 00:04:39 -0700 |
commit | 770202f0c745d68f2635dbb8d0f0765aae75a087 (patch) | |
tree | 46d29849be392aed1a168e0f291ab98a210dca8d /tests/vb20_common3_tests.c | |
parent | fefc682bb70d7fd997f4ef0079e8fec0a4937cf7 (diff) | |
download | vboot-770202f0c745d68f2635dbb8d0f0765aae75a087.tar.gz |
Fix more coverity warnings
Assorted minor code issues, which we should fix so any new errors stand
out more.
BUG=chromium:643769
BRANCH=none
TEST=make runtests
Change-Id: I82ece2de948ef224115c408bdfc09445d3da119b
Signed-off-by: Randall Spangler <rspangler@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/390337
Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org>
Diffstat (limited to 'tests/vb20_common3_tests.c')
-rw-r--r-- | tests/vb20_common3_tests.c | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/tests/vb20_common3_tests.c b/tests/vb20_common3_tests.c index fd15892a..24029293 100644 --- a/tests/vb20_common3_tests.c +++ b/tests/vb20_common3_tests.c @@ -224,8 +224,11 @@ static void test_verify_fw_preamble(struct vb2_packed_key *public_key, private_key, 0x5678); TEST_PTR_NEQ(hdr, NULL, "vb2_verify_fw_preamble() prereq test preamble"); - if (!hdr) + if (!hdr) { + free(body_sig); return; + } + hsize = (uint32_t) hdr->preamble_size; h = (struct vb2_fw_preamble *)malloc(hsize + 16384); @@ -366,8 +369,11 @@ static void test_verify_kernel_preamble( private_key); TEST_PTR_NEQ(hdr, NULL, "vb2_verify_kernel_preamble() prereq test preamble"); - if (!hdr) + if (!hdr) { + free(body_sig); return; + } + hsize = (uint32_t) hdr->preamble_size; struct vb2_kernel_preamble *h = (struct vb2_kernel_preamble *)malloc(hsize + 16384); |