diff options
| author | Steve Magnani | 2017-10-12 15:48:41 +0200 |
|---|---|---|
| committer | Jan Kara | 2017-10-17 12:00:58 +0200 |
| commit | fcbf7637e6647e00de04d4b2e05ece2484bb3062 (patch) | |
| tree | 438535f802620428f7f42c6f44153b5fc5bfe29b /fs/udf/namei.c | |
| parent | udf: Fix 64-bit sign extension issues affecting blocks > 0x7FFFFFFF (diff) | |
| download | kernel-qcow2-linux-fcbf7637e6647e00de04d4b2e05ece2484bb3062.tar.gz kernel-qcow2-linux-fcbf7637e6647e00de04d4b2e05ece2484bb3062.tar.xz kernel-qcow2-linux-fcbf7637e6647e00de04d4b2e05ece2484bb3062.zip | |
udf: Fix signed/unsigned format specifiers
Fix problems noted in compilion with -Wformat=2 -Wformat-signedness.
In particular, a mismatch between the signedness of a value and the
signedness of its format specifier can result in unsigned values being
printed as negative numbers, e.g.:
Partition (0 type 1511) starts at physical 460, block length -1779968542
...which occurs when mounting a large (> 1 TiB) UDF partition.
Changes since V1:
* Fixed additional issues noted in udf_bitmap_free_blocks(),
udf_get_fileident(), udf_show_options()
Signed-off-by: Steven J. Magnani <steve@digidescorp.com>
Signed-off-by: Jan Kara <jack@suse.cz>
Diffstat (limited to 'fs/udf/namei.c')
| -rw-r--r-- | fs/udf/namei.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/udf/namei.c b/fs/udf/namei.c index d3f18a6fbe40..0458dd47e105 100644 --- a/fs/udf/namei.c +++ b/fs/udf/namei.c @@ -840,7 +840,7 @@ static int udf_rmdir(struct inode *dir, struct dentry *dentry) if (retval) goto end_rmdir; if (inode->i_nlink != 2) - udf_warn(inode->i_sb, "empty directory has nlink != 2 (%d)\n", + udf_warn(inode->i_sb, "empty directory has nlink != 2 (%u)\n", inode->i_nlink); clear_nlink(inode); inode->i_size = 0; @@ -882,7 +882,7 @@ static int udf_unlink(struct inode *dir, struct dentry *dentry) goto end_unlink; if (!inode->i_nlink) { - udf_debug("Deleting nonexistent file (%lu), %d\n", + udf_debug("Deleting nonexistent file (%lu), %u\n", inode->i_ino, inode->i_nlink); set_nlink(inode, 1); } |