[PATCH 0/3] squashfs fixes

Thomas Petazzoni thomas.petazzoni at bootlin.com
Wed Jul 15 22:11:40 CEST 2020


Hello Joao,

As I started reviewing your patch series, I encountered a number of
build issues after applying your patches, and building on the latest
master. The following patches address the build failures I have
encountered. Could you squash them into your commits, so that those
fixes are part of your v2 ? Of course, please do some testing, as I
have only done build testing so far.

Also, even with those fixes in place, I do still get a worrying gcc
warning (from gcc 10):

fs/squashfs/sqfs_inode.c: In function ‘sqfs_find_inode’:
fs/squashfs/sqfs_inode.c:72:34: warning: array subscript l is outside array bounds of ‘struct squashfs_directory_index[0]’ [-Warray-bounds]
   72 |     index_list_size += ldir.index[l].size + 1;
      |                        ~~~~~~~~~~^~~
In file included from fs/squashfs/sqfs_inode.c:16:
fs/squashfs/sqfs_filesystem.h:207:34: note: while referencing ‘index’
  207 |  struct squashfs_directory_index index[0];
      |                                  ^~~~~
fs/squashfs/sqfs_inode.c:31:29: note: defined here ‘ldir’
   31 |  struct squashfs_ldir_inode ldir;
      |                             ^~~~

Could you investigate this ?

Thanks,

Thomas

Thomas Petazzoni (3):
  cmd/sqfs.c, include/fs.h: fix build failures, don't expose do_sqfs_*()
    functions globally
  fs/squashfs: use "struct disk_partition" instead of "disk_partition_t"
  fs/squashfs: do not use CMD_RET_* defines in the filesystem code

 cmd/sqfs.c         |  5 ++--
 fs/squashfs/sqfs.c | 66 ++++++++++++++++++++--------------------------
 include/fs.h       | 12 ---------
 include/squashfs.h |  4 ++-
 4 files changed, 34 insertions(+), 53 deletions(-)

-- 
2.26.2



More information about the U-Boot mailing list