From a25b5d257dbfbff808b19bf8c48565435e6bef9d Mon Sep 17 00:00:00 2001 From: Julius Werner Date: Mon, 8 Feb 2016 11:46:22 -0800 Subject: lzma: Port size-checking ulzman() version to coreboot We've had a second version of ulzma() that would check the input and output buffer sizes in libpayload for a while now. Since it's generally never a bad idea to double-check for overruns, let's port it to coreboot and use it where applicable. (This requires a small fix in the four byte at a time read optimization we only have in coreboot, since it made the stream counter hit the end a little earlier than the algorithm liked and could trigger an assertion.) BRANCH=None BUG=None TEST=Booted Oak, Jerry and Falco. Change-Id: Id566b31dfa896ea1b991badf5a6ad9d075aef987 Signed-off-by: Julius Werner Reviewed-on: https://review.coreboot.org/13637 Tested-by: build bot (Jenkins) Reviewed-by: Aaron Durbin --- src/include/lib.h | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) (limited to 'src/include/lib.h') diff --git a/src/include/lib.h b/src/include/lib.h index ab1f067ee3..737ab36b83 100644 --- a/src/include/lib.h +++ b/src/include/lib.h @@ -20,8 +20,9 @@ #include #include -/* Defined in src/lib/lzma.c */ -unsigned long ulzma(unsigned char *src, unsigned char *dst); +/* Defined in src/lib/lzma.c. Returns decompressed size or 0 on error. */ +size_t ulzma(const void *src, void *dst); +size_t ulzman(const void *src, size_t srcn, void *dst, size_t dstn); /* Defined in src/lib/ramtest.c */ void ram_check(unsigned long start, unsigned long stop); -- cgit v1.2.3