u-boot.git/fs, branch v2021.01

fs/squashfs: sqfs_close/sqfs_read_sblk: set ctxt.sblk to NULL after free

2020-12-02T21:21:58+00:00

This will prevent a double free error if sqfs_close() is called twice.

Signed-off-by: Richard Genoud

fs: fat: use ATTR_ARCH instead of anonymous 0x20

2020-11-29T04:18:16+00:00

Using constants instead of anonymous numbers increases code readability.

Fixes: 704df6aa0a28 ("fs: fat: refactor write interface for a file offset")
Signed-off-by: Heinrich Schuchardt

fs: fat: directory entries starting with 0x05

2020-11-29T04:18:16+00:00

0x05 is used as replacement letter for 0xe5 at the first position of short
file names. We must not skip over directory entries starting with 0x05.

Cf. Microsoft FAT Specification, August 30 2005

Fixes: 39606d462c97 ("fs: fat: handle deleted directory entries correctly")
Signed-off-by: Heinrich Schuchardt 
Reviewed-by: Christian Gmeiner 
Reviewed-by: Simon Glass

fs: fat: avoid NULL dereference when root dir is full

2020-11-29T04:18:16+00:00

When trying to create a file in the full root directory of a FAT32
filesystem a NULL dereference can be observed.

When the root directory of a FAT16 filesystem is full fill_dir_slot() must
return -1 to signal that a new directory entry could not be allocated.

Fixes: cd2d727fff7e ("fs: fat: allocate a new cluster for root directory of fat32")
Signed-off-by: Heinrich Schuchardt

fs/squashfs: implement exists() function

2020-11-19T14:45:49+00:00

This permits to find a file and use the distro_bootcmd

Reviewed-by: Joao Marcos Costa 
Signed-off-by: Richard Genoud

fs/squashfs: sqfs_read: remove buggy offset functionality

2020-11-19T14:45:49+00:00

offset is the offset in the file read, not the offset in the destination
buffer.
If the offset is not null, this will lead to a memory corruption.
So, for now, we are returning an error if the offset is used.

Signed-off-by: Richard Genoud

fs/squashfs: sqfs_read: don't write beyond buffer size

2020-11-19T14:45:49+00:00

The length of the buffer wasn't taken into account when writing to the
given buffer.

Signed-off-by: Richard Genoud

fs/squashfs: sqfs_probe: use sqfs_decompressor_init() return value

2020-11-19T14:45:49+00:00

sqfs_decompressor_init() returns a value, so it's better to use it than
to force the return value to EINVAL (it could be ENOMEM)

Reviewed-by: Joao Marcos Costa 
Signed-off-by: Richard Genoud

fs/squashfs: sqfs_probe: reset cur_dev/cur_part_info to NULL on error

2020-11-19T14:45:49+00:00

Resetting the context on error will prevent some checks like:
if (!ctx.cur_dev)
To pass when the probe method has failed

Signed-off-by: Richard Genoud

fs/squashfs: sqfs_probe: fix possible memory leak on error

2020-11-19T14:45:49+00:00

If SquashFS magic number is invalid, there's a memory leak.

Reviewed-by: Joao Marcos Costa 
Signed-off-by: Richard Genoud