diff options
author | Aaron Durbin <adurbin@chromium.org> | 2015-10-23 17:38:40 -0500 |
---|---|---|
committer | Patrick Georgi <pgeorgi@google.com> | 2015-10-25 13:23:15 +0100 |
commit | 5213c53f6f126bf79ca2849d7d04a0e5141811ea (patch) | |
tree | aaf45f052b52a6c8c33a80d8a1ce51093c28ab54 | |
parent | cd9ba8ac034d9a693066ac3226332053f744299e (diff) |
cbfstool: have decompress functions provide ouput data size
Currently cbfs stage files that are compressed do not have
the decompressed size readily available. Therefore there's
no good way to know actual size of data after it is
decompressed. Optionally return the decompressed data size
if requested.
Change-Id: If371753d28d0ff512118d8bc06fdd48f4a0aeae7
Signed-off-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-on: http://review.coreboot.org/12173
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
-rw-r--r-- | util/cbfstool/cbfs_image.c | 2 | ||||
-rw-r--r-- | util/cbfstool/common.h | 6 | ||||
-rw-r--r-- | util/cbfstool/compress.c | 10 | ||||
-rw-r--r-- | util/cbfstool/lzma/lzma.c | 6 |
4 files changed, 17 insertions, 7 deletions
diff --git a/util/cbfstool/cbfs_image.c b/util/cbfstool/cbfs_image.c index bc5c5d7dec..bf01c9676d 100644 --- a/util/cbfstool/cbfs_image.c +++ b/util/cbfstool/cbfs_image.c @@ -692,7 +692,7 @@ int cbfs_export_entry(struct cbfs_image *image, const char *entry_name, buffer.data = malloc(decompressed_size); buffer.size = decompressed_size; if (decompress(CBFS_SUBHEADER(entry), ntohl(entry->len), - buffer.data, buffer.size)) { + buffer.data, buffer.size, NULL)) { ERROR("decompression failed for %s\n", entry_name); return -1; } diff --git a/util/cbfstool/common.h b/util/cbfstool/common.h index e889e52704..4920d76f59 100644 --- a/util/cbfstool/common.h +++ b/util/cbfstool/common.h @@ -153,7 +153,8 @@ typedef int (*comp_func_ptr) (char *in, int in_len, char *out, int *out_len); * Returns 0 on error, * != 0 otherwise, depending on the decompressing function. */ -typedef int (*decomp_func_ptr) (char *in, int in_len, char *out, int out_len); +typedef int (*decomp_func_ptr) (char *in, int in_len, char *out, int out_len, + size_t *actual_size); enum comp_algo { CBFS_COMPRESS_NONE = 0, CBFS_COMPRESS_LZMA = 1 }; @@ -187,7 +188,8 @@ void print_supported_filetypes(void); /* lzma/lzma.c */ int do_lzma_compress(char *in, int in_len, char *out, int *out_len); -int do_lzma_uncompress(char *dst, int dst_len, char *src, int src_len); +int do_lzma_uncompress(char *dst, int dst_len, char *src, int src_len, + size_t *actual_size); /* xdr.c */ struct xdr { diff --git a/util/cbfstool/compress.c b/util/cbfstool/compress.c index 2bde7df96a..0313b96898 100644 --- a/util/cbfstool/compress.c +++ b/util/cbfstool/compress.c @@ -31,9 +31,10 @@ static int lzma_compress(char *in, int in_len, char *out, int *out_len) return do_lzma_compress(in, in_len, out, out_len); } -static int lzma_decompress(char *in, int in_len, char *out, unused int out_len) +static int lzma_decompress(char *in, int in_len, char *out, unused int out_len, + size_t *actual_size) { - return do_lzma_uncompress(out, out_len, in, in_len); + return do_lzma_uncompress(out, out_len, in, in_len, actual_size); } static int none_compress(char *in, int in_len, char *out, int *out_len) { @@ -42,9 +43,12 @@ static int none_compress(char *in, int in_len, char *out, int *out_len) return 0; } -static int none_decompress(char *in, int in_len, char *out, unused int out_len) +static int none_decompress(char *in, int in_len, char *out, unused int out_len, + size_t *actual_size) { memcpy(out, in, in_len); + if (actual_size != NULL) + *actual_size = in_len; return 0; } diff --git a/util/cbfstool/lzma/lzma.c b/util/cbfstool/lzma/lzma.c index 856932d22e..986ebfaa22 100644 --- a/util/cbfstool/lzma/lzma.c +++ b/util/cbfstool/lzma/lzma.c @@ -152,7 +152,8 @@ int do_lzma_compress(char *in, int in_len, char *out, int *out_len) return 0; } -int do_lzma_uncompress(char *dst, int dst_len, char *src, int src_len) +int do_lzma_uncompress(char *dst, int dst_len, char *src, int src_len, + size_t *actual_size) { if (src_len <= LZMA_PROPS_SIZE + 8) { ERROR("LZMA: Input length is too small.\n"); @@ -184,5 +185,8 @@ int do_lzma_uncompress(char *dst, int dst_len, char *src, int src_len) return -1; } + if (actual_size != NULL) + *actual_size = destlen; + return 0; } |