Re: [PATCH 1/2] misc: fix all the compiler warnings
From: Andreas Dilger <hidden>
Date: 2017-03-03 20:52:05
On Mar 2, 2017, at 3:20 PM, Darrick J. Wong [off-list ref] wrote:
From: Darrick J. Wong <redacted> Fix the various compiler warnings that have crept in, and only define __bitwise if the system headers haven't already done so. Linux 4.10 changes the __bitwise definition so that our redefinition here is just different enough that gcc complains. Signed-off-by: Darrick J. Wong <redacted>
Reviewed-by: Andreas Dilger <redacted>
quoted hunk ↗ jump to hunk
--- debugfs/debugfs.c | 2 -- debugfs/xattrs.c | 5 ++--- e2fsck/pass2.c | 2 +- lib/ext2fs/ext2_types.h.in | 8 ++++++-- lib/ext2fs/ext_attr.c | 2 -- lib/support/cstring.c | 6 +++--- misc/dumpe2fs.c | 15 --------------- 7 files changed, 12 insertions(+), 28 deletions(-)diff --git a/debugfs/debugfs.c b/debugfs/debugfs.c index 1a84cfc..059ddc3 100644 --- a/debugfs/debugfs.c +++ b/debugfs/debugfs.c@@ -1605,9 +1605,7 @@ void do_unlink(int argc, char *argv[])void do_copy_inode(int argc, char *argv[]) { ext2_ino_t src_ino, dest_ino; - struct ext2_inode inode; unsigned char buf[4096]; - int retval; if (common_args_process(argc, argv, 3, 3, "copy_inode", "<source file> <dest_name>", CHECK_FS_RW))diff --git a/debugfs/xattrs.c b/debugfs/xattrs.c index 2443518..9b87d14 100644 --- a/debugfs/xattrs.c +++ b/debugfs/xattrs.c@@ -146,7 +146,7 @@ void do_get_xattr(int argc, char **argv)size_t buflen; int i; int print_flags = 0; - int handle_flags = 0; + unsigned int handle_flags = 0; errcode_t err; reset_getopt();@@ -241,8 +241,7 @@ void do_set_xattr(int argc, char **argv)FILE *fp = NULL; char *buf = NULL; size_t buflen; - int print_flags = 0; - int handle_flags = 0; + unsigned int handle_flags = 0; int i; errcode_t err;diff --git a/e2fsck/pass2.c b/e2fsck/pass2.c index 11c19e8..170878c 100644 --- a/e2fsck/pass2.c +++ b/e2fsck/pass2.c@@ -90,7 +90,7 @@ void e2fsck_pass2(e2fsck_t ctx)struct ext2_super_block *sb = ctx->fs->super; struct problem_context pctx; ext2_filsys fs = ctx->fs; - char *buf; + char *buf = NULL; #ifdef RESOURCE_TRACK struct resource_track rtrack; #endifdiff --git a/lib/ext2fs/ext2_types.h.in b/lib/ext2fs/ext2_types.h.in index 07adc64..fb36a5d 100644 --- a/lib/ext2fs/ext2_types.h.in +++ b/lib/ext2fs/ext2_types.h.in@@ -173,10 +173,14 @@ typedef long __s64;#define EXT2_ENDIAN_H_ #ifdef __CHECKER__ -#define __bitwise __attribute__((bitwise)) +# ifndef __bitwise +# define __bitwise __attribute__((bitwise)) +# endif #define __force __attribute__((force)) #else -#define __bitwise +# ifndef __bitwise +# define __bitwise +# endif #define __force #endifdiff --git a/lib/ext2fs/ext_attr.c b/lib/ext2fs/ext_attr.c index ce2f48f..7a9a2d5 100644 --- a/lib/ext2fs/ext_attr.c +++ b/lib/ext2fs/ext_attr.c@@ -478,7 +478,6 @@ static errcode_t convert_posix_acl_to_disk_buffer(const void *value, size_t sizeext4_acl_header *ext_acl; size_t s; void *e; - int err; int count;@@ -532,7 +531,6 @@ static errcode_t convert_disk_buffer_to_posix_acl(const void *value, size_t sizeerrcode_t err; const char *cp; char *out; - int count; if ((!value) || (size < sizeof(ext4_acl_header)) ||diff --git a/lib/support/cstring.c b/lib/support/cstring.c index 7965e95..57f4522 100644 --- a/lib/support/cstring.c +++ b/lib/support/cstring.c@@ -15,12 +15,12 @@int parse_c_string(char *str) { - unsigned char *to, *from, ch; + char *to, *from, ch; int v; to = from = str; - for (to = from = (unsigned char *) str; + for (to = from = (char *) str; *from && *from != '"'; to++, from++) { if (*from == '\\') { ch = *(++from);@@ -93,7 +93,7 @@ int parse_c_string(char *str)*to = *from; } *to = '\0'; - return to - (unsigned char *) str; + return to - (char *) str; } void print_c_string(FILE *f, const char *cp, int len)diff --git a/misc/dumpe2fs.c b/misc/dumpe2fs.c index 621840a..395ea9e 100644 --- a/misc/dumpe2fs.c +++ b/misc/dumpe2fs.c@@ -356,16 +356,6 @@ static void list_bad_blocks(ext2_filsys fs, int dump)ext2fs_badblocks_list_free(bb_list); } -static const char *journal_checksum_type_str(__u8 type) -{ - switch (type) { - case JBD2_CRC32C_CHKSUM: - return "crc32c"; - default: - return "unknown"; - } -}
I'd assume this was meant to be used somewhere?
quoted hunk ↗ jump to hunk
- static void print_inline_journal_information(ext2_filsys fs) { journal_superblock_t *jsb;@@ -374,8 +364,6 @@ static void print_inline_journal_information(ext2_filsys fs)errcode_t retval; ino_t ino = fs->super->s_journal_inum; char buf[1024]; - __u32 *mask_ptr, mask, m; - int i, j, size, printed = 0; if (fs->flags & EXT2_FLAG_IMAGE_FILE) return;@@ -411,10 +399,7 @@ static void print_journal_information(ext2_filsys fs){ errcode_t retval; char buf[1024]; - char str[80]; - unsigned int i, j, printed = 0; journal_superblock_t *jsb; - __u32 *mask_ptr, mask, m; /* Get the journal superblock */ if ((retval = io_channel_read_blk64(fs->io,
Cheers, Andreas
Attachments
- signature.asc [application/pgp-signature] 195 bytes