Archive-Team
/
zfs
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases 103 Wiki Activity

make zdb_decompress_block check decompression reliably 

This function decompresses to two buffers and then compares them to
check whether the (opaque) decompression process filled the whole
buffer. Previously it began with lbuf uninitialized and lbuf2 filled
with pseudorandom data. This neither guarantees that any bytes not
written by the compressor would be different, nor seems incredibly
sound otherwise!

After these changes, instead of filling one buffer with generated
pseudorandom data we overwrite each buffer with completely different
data. This should remove the possibility of low-probability failures,
as well as make the process simpler and cheaper.

Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Reviewed-by: Rich Ercolani <rincebrain@gmail.com>
Signed-off-by: Kent Ross <k@mad.cash>
Closes #15733
This commit is contained in:
Kent Ross 2024-01-09 09:13:52 -08:00 committed by GitHub
parent bd3f90c0c1
commit 7ecaa07580
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 7 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
cmd/zdb/zdb.c
View File

@ -8533,11 +8533,14 @@ zdb_decompress_block(abd_t *pabd, void *buf, void *lbuf, uint64_t lsize,
} }
/* /*
* We randomize lbuf2, and decompress to both * We set lbuf to all zeros and lbuf2 to all
* lbuf and lbuf2. This way, we will know if * ones, then decompress to both buffers and
* decompression fill exactly to lsize. * compare their contents. This way we can
* know if decompression filled exactly to
* lsize or if it left some bytes unwritten.
*/ */
VERIFY0(random_get_pseudo_bytes(lbuf2, lsize)); memset(lbuf, 0x00, lsize);
memset(lbuf2, 0xff, lsize);
if (zio_decompress_data(*cfuncp, pabd, if (zio_decompress_data(*cfuncp, pabd,
lbuf, psize, lsize, NULL) == 0 && lbuf, psize, lsize, NULL) == 0 &&