squashfs-tools-ng.git/lib, branch v1.0.6

Fix libsquashfs directory writer size accounting

2021-07-21T08:01:39+00:00

The squashfs readdir() implementation in the Linux kernel returns non-existing "." and ".." entries for offsets 0 and 1, and after that reads from disk. For convenience, it was decided to store an off-by-3 value on disk instead of doing complex primary school math to adjust for this. This didn't show up until now, because the kernel implementation trusts the value from the directory header more than the actual size in the inode and happily reads 3 more than the inode would allow it to. This only showed up with 7-zip which subtracts 3 from the size and expects the result to be exact and bails if the directory headers suggest otherwise. And yes, I did consider making a "Holy Hand Granade of Antioch" reference, but consciously decided not to. Signed-off-by: David Oberhollenzer

Fix printf format specifiers used for generating tarballs

2021-07-09T18:00:36+00:00

When processing files > 4G, using "%o" truncates the result and the tarball is not readable. This should have been discovered when auto-patching the printf format specifiers, but a cast was added instead and the issue was overlooked. This commit replaces the down-cast and printf format specifiers. Signed-off-by: David Oberhollenzer

Add default cases for every switch block

2021-06-25T15:49:11+00:00

Signed-off-by: David Oberhollenzer

Remove casual un-const casting in various places

2021-06-25T15:48:49+00:00

Signed-off-by: David Oberhollenzer

libsquashfs: get rid of potentially unaligned access and VLAs

2021-06-25T15:37:56+00:00

The same problem with the meta data header again, 16 bit read from a buffer: copy the buffer data into a 16 bit variable instead of casting to something potentially unaligned. Signed-off-by: David Oberhollenzer

libcommon: remove potentially un-aligned access in LZO compressor

2021-06-25T15:37:56+00:00

When accessing the 16 bit header, don't cast the buffer pointer to an uint16_t pointer, the result might not be aligned propperly. Instead memcpy to and from an uint16_t. Signed-off-by: David Oberhollenzer

libfstree: guard against link count and inode number overflow

2021-06-25T15:37:56+00:00

If the hard link counter or the inode number counter overflow the maximum representable value (for SquashFS 16 bit and 32 bit respecitively), abort with an error message. Signed-off-by: David Oberhollenzer

Fix: libsquashfs: add sqfs_free() function

2021-06-07T12:35:40+00:00

On systems like Windows, the dynamic library and applications can easily end up being linked against different runtime libraries, so applications cannot be expected to be able to free() any malloc'd pointer that the library returns. This commit adds an sqfs_free function so the application can pass pointers back to the library to call the correct free() implementation. Signed-off-by: David Oberhollenzer

libsquashfs: fix: also preserve alignment flag in block processor

2021-06-07T12:32:23+00:00

Currently, when the block processor aggreagtes fragments into a fragment block, it applies the "don't compress" flag if any of the original framgnets has it set, but the "align to device block" flag is lost. This commit ensures that both flags get applied to the fragment block if set. Signed-off-by: David Oberhollenzer

libsquashfs: fix block alignment if requested

2021-06-07T12:30:29+00:00

1) If the block alignment flag is set, the padding bytes must be inserted _before_ recording the start position, otherwise the resulting image is not readable. 2) Also perform alignment if the flag is set on a fragment block. Signed-off-by: David Oberhollenzer