| Message ID | 20180606072717.28854-2-wqu@suse.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
diff --git a/cmds-restore.c b/cmds-restore.c index 342f5cc7eabb..29a329a397c2 100644 --- a/cmds-restore.c +++ b/cmds-restore.c @@ -324,7 +324,8 @@ static int copy_one_inline(struct btrfs_root *root, int fd, return -ENOMEM; } - ret = decompress(root, buf, outbuf, len, &ram_size, compress); + ret = decompress(root, buf, outbuf, inline_item_len, &ram_size, + compress); if (ret) { free(outbuf); return ret;
When using decompress() in copy_one_inline(), we're passing the decompressed extent size (ram_bytes) into decompress(). However we only has @inline_item_len read out, the pending data will be uninitialized data. Thankfully, all compression methods supported have some extra data in its header, thus it won't cause real problem. Whatever fixing it is never a bad idea. Signed-off-by: Qu Wenruo <wqu@suse.com> --- cmds-restore.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)