Maximilian Brune has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/83618?usp=email )
Change subject: [WIP] util/cbfstool/cbfs-payload-linux.c: Remove TODO ......................................................................
[WIP] util/cbfstool/cbfs-payload-linux.c: Remove TODO
LZMA checks at util/cbfstool/lzma/lzma.c:Write() for the output buffer/stream size and does not write beyond it.
LZ4 TODO check
Change-Id: I41298b509b3f5e775bb4000c82c539eefa80c885 Signed-off-by: Maximilian Brune maximilian.brune@9elements.com --- M util/cbfstool/cbfs-payload-linux.c 1 file changed, 0 insertions(+), 5 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/18/83618/1
diff --git a/util/cbfstool/cbfs-payload-linux.c b/util/cbfstool/cbfs-payload-linux.c index 94f6161..9ea475e 100644 --- a/util/cbfstool/cbfs-payload-linux.c +++ b/util/cbfstool/cbfs-payload-linux.c @@ -193,11 +193,6 @@ * add support for more parameters to trampoline: * alt_mem_k, ext_mem_k (not strictly necessary since e820 takes precedence) * framebuffer/console values - * - * larger work: - * is compress() safe to use in a size constrained buffer? ie. do(es) the - * compression algorithm(s) stop once the compression result reaches input - * size (ie. incompressible data)? */ int parse_bzImage_to_payload(const struct buffer *input, struct buffer *output, const char *initrd_name,