libubox/tests/cram/test_blob_parse.t
Petr Štetiar 478597b9f9 blob: fix OOB access in blob_check_type
Found by fuzzer:

 ERROR: AddressSanitizer: SEGV on unknown address 0x602100000455
 The signal is caused by a READ memory access.
     #0 in blob_check_type blob.c:214:43
     #1 in blob_parse_attr blob.c:234:9
     #2 in blob_parse_untrusted blob.c:272:12
     #3 in fuzz_blob_parse tests/fuzzer/test-blob-parse-fuzzer.c:34:2
     #4 in LLVMFuzzerTestOneInput tests/fuzzer/test-blob-parse-fuzzer.c:39:2

Caused by following line:

	if (type == BLOB_ATTR_STRING && data[len - 1] != 0)

where len was pointing outside of the data buffer.

Signed-off-by: Petr Štetiar <ynezz@true.cz>
2019-12-25 10:31:58 +01:00

78 lines
3.4 KiB
Perl

check that blob_parse is producing expected results:
$ [ -n "$TEST_BIN_DIR" ] && export PATH="$TEST_BIN_DIR:$PATH"
$ export TEST_INPUTS="$TESTDIR/inputs"
$ export FUZZ_CORPUS="$TESTDIR/../fuzz/corpus"
$ valgrind --quiet --leak-check=full test-blob-parse $TEST_INPUTS/key-build.ucert
=== CHAIN ELEMENT 01 ===
signature:
---
untrusted comment: signed by key 84bfc88a17166577
RWSEv8iKFxZld+bQ+NTqCdDlHOuVYNw5Qw7Q8shjfMgFJcTqrzaqO0bysjIQhTadmcwvWiWvHlyMcwAXSix2BYdfghz/zhDjvgU=
---
payload:
---
"ucert": {
\t"certtype": 1, (esc)
\t"validfrom": 1546188410, (esc)
\t"expiresat": 1577724410, (esc)
\t"pubkey": "untrusted comment: Local build key\\nRWSEv8iKFxZld6vicE1icWhYNfEV9PM7C9MKUKl+YNEKB+PdAWGDF5Z9\\n" (esc)
}
---
$ valgrind --quiet --leak-check=full test-blob-parse $TEST_INPUTS/invalid.ucert
cannot parse cert invalid.ucert
$ test-blob-parse-san $TEST_INPUTS/key-build.ucert
=== CHAIN ELEMENT 01 ===
signature:
---
untrusted comment: signed by key 84bfc88a17166577
RWSEv8iKFxZld+bQ+NTqCdDlHOuVYNw5Qw7Q8shjfMgFJcTqrzaqO0bysjIQhTadmcwvWiWvHlyMcwAXSix2BYdfghz/zhDjvgU=
---
payload:
---
"ucert": {
\t"certtype": 1, (esc)
\t"validfrom": 1546188410, (esc)
\t"expiresat": 1577724410, (esc)
\t"pubkey": "untrusted comment: Local build key\\nRWSEv8iKFxZld6vicE1icWhYNfEV9PM7C9MKUKl+YNEKB+PdAWGDF5Z9\\n" (esc)
}
---
$ test-blob-parse-san $TEST_INPUTS/invalid.ucert
cannot parse cert invalid.ucert
$ for blob in $(LC_ALL=C find $FUZZ_CORPUS -type f | sort ); do
> valgrind --quiet --leak-check=full test-blob-parse $blob; \
> test-blob-parse-san $blob; \
> done
cannot parse cert 71520a5c4b5ca73903216857abbad54a8002d44a
cannot parse cert 71520a5c4b5ca73903216857abbad54a8002d44a
cannot parse cert c1dfd96eea8cc2b62785275bca38ac261256e278
cannot parse cert c1dfd96eea8cc2b62785275bca38ac261256e278
cannot parse cert c42ac1c46f1d4e211c735cc7dfad4ff8391110e9
cannot parse cert c42ac1c46f1d4e211c735cc7dfad4ff8391110e9
cannot parse cert crash-1b8fb1be45db3aff7699100f497fb74138f3df4f
cannot parse cert crash-1b8fb1be45db3aff7699100f497fb74138f3df4f
cannot parse cert crash-333757b203a44751d3535f24b05f467183a96d09
cannot parse cert crash-333757b203a44751d3535f24b05f467183a96d09
cannot parse cert crash-4c4d2c3c9ade5da9347534e290305c3b9760f627
cannot parse cert crash-4c4d2c3c9ade5da9347534e290305c3b9760f627
cannot parse cert crash-5e9937b197c88bf4e7b7ee2612456cad4cb83f5b
cannot parse cert crash-5e9937b197c88bf4e7b7ee2612456cad4cb83f5b
cannot parse cert crash-75b146c4e6fac64d3e62236b27c64b50657bab2a
cannot parse cert crash-75b146c4e6fac64d3e62236b27c64b50657bab2a
cannot parse cert crash-813f3e68661da09c26d4a87dbb9d5099e92be50f
cannot parse cert crash-813f3e68661da09c26d4a87dbb9d5099e92be50f
cannot parse cert crash-98595faa58ba01d85ba4fd0b109cd3d490b45795
cannot parse cert crash-98595faa58ba01d85ba4fd0b109cd3d490b45795
cannot parse cert crash-d0f3aa7d60a094b021f635d4edb7807c055a4ea1
cannot parse cert crash-d0f3aa7d60a094b021f635d4edb7807c055a4ea1
cannot parse cert crash-df9d1243057b27bbad6211e5a23d1cb699028aa2
cannot parse cert crash-df9d1243057b27bbad6211e5a23d1cb699028aa2
cannot parse cert crash-e2fd5ecb3b37926743256f1083f47a07c39e10c2
cannot parse cert crash-e2fd5ecb3b37926743256f1083f47a07c39e10c2
cannot parse cert valid-blobmsg.bin
cannot parse cert valid-blobmsg.bin