diff options
author | Vincent Palatin <vpalatin@chromium.org> | 2015-10-15 17:46:58 -0700 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2015-10-17 08:18:48 -0700 |
commit | 4d47243c9088ef295892fbc25b9c3622e43ad639 (patch) | |
tree | 07df0c98c2819fbdf0d77fc349eb6802e69fc0b6 /tests/futility/test_file_types.sh | |
parent | 02ac2885fd797fba7f12ef040f0eb041dda7af20 (diff) | |
download | vboot-4d47243c9088ef295892fbc25b9c3622e43ad639.tar.gz |
futility: fix segfault on short files
Verify the size of the buffer read from the file before trying to use 1KB of
it for the new rwsig format detection.
Add a new test case with a short file containing only 4 bytes of unknown
data and run "futility show" on it.
BRANCH=smaug
BUG=none
TEST=futility show foobar.pub.pem
where foobar.pub.pem is a 451-byte file.
check that "make runtests" passes with the fix
and fails without it with the following message :
test_file_types.sh ... failed
FAIL: 13 / 14 passed
Change-Id: Ia9d68c6b528c2b3a595ea6791c907374616d051f
Reviewed-on: https://chromium-review.googlesource.com/306682
Commit-Ready: Vincent Palatin <vpalatin@chromium.org>
Tested-by: Vincent Palatin <vpalatin@chromium.org>
Reviewed-by: Bill Richardson <wfrichar@chromium.org>
Diffstat (limited to 'tests/futility/test_file_types.sh')
-rwxr-xr-x | tests/futility/test_file_types.sh | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/tests/futility/test_file_types.sh b/tests/futility/test_file_types.sh index 8acbcf72..470d6313 100755 --- a/tests/futility/test_file_types.sh +++ b/tests/futility/test_file_types.sh @@ -26,6 +26,7 @@ fail_case() { } # Known types +test_case "unknown" "tests/futility/data/short_junk.bin" test_case "unknown" "tests/futility/data/random_noise.bin" test_case "pubkey" "tests/devkeys/root_key.vbpubk" test_case "keyblock" "tests/devkeys/kernel.keyblock" |