diff options
author | Chao Yu <[email protected]> | 2020-05-08 09:16:03 +0800 |
---|---|---|
committer | Jaegeuk Kim <[email protected]> | 2020-05-11 20:37:13 -0700 |
commit | 1454c978efbb57b052670d50023f48c759d704ce (patch) | |
tree | 27e5443f67eb0dec0eb93c238165193333b93d8d | |
parent | 9c1223845a37ce09fd498b8c8ed061decff20eda (diff) |
f2fs: compress: fix zstd data corruption
During zstd compression, ZSTD_endStream() may return non-zero value
because distination buffer is full, but there is still compressed data
remained in intermediate buffer, it means that zstd algorithm can not
save at last one block space, let's just writeback raw data instead of
compressed one, this can fix data corruption when decompressing
incomplete stored compression data.
Fixes: 50cfa66f0de0 ("f2fs: compress: support zstd compress algorithm")
Signed-off-by: Daeho Jeong <[email protected]>
Signed-off-by: Chao Yu <[email protected]>
Signed-off-by: Jaegeuk Kim <[email protected]>
-rw-r--r-- | fs/f2fs/compress.c | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/fs/f2fs/compress.c b/fs/f2fs/compress.c index dcedf008f359..bf152c0d79fe 100644 --- a/fs/f2fs/compress.c +++ b/fs/f2fs/compress.c @@ -360,6 +360,13 @@ static int zstd_compress_pages(struct compress_ctx *cc) return -EIO; } + /* + * there is compressed data remained in intermediate buffer due to + * no more space in cbuf.cdata + */ + if (ret) + return -EAGAIN; + cc->clen = outbuf.pos; return 0; } |