lkml.org 
[lkml]   [2006]   [Jun]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
Subject[PATCH] affs_fill_super() %s abuses
%s is valid only on NUL-terminated arrays, damnit!

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>

---

fs/affs/super.c | 20 ++++++++++++++------
1 files changed, 14 insertions(+), 6 deletions(-)

02cc7ba4e655dc01773f43e5324986188d42653d
diff --git a/fs/affs/super.c b/fs/affs/super.c
index 4d7e5b1..02aeb22 100644
--- a/fs/affs/super.c
+++ b/fs/affs/super.c
@@ -271,6 +271,7 @@ static int affs_fill_super(struct super_
int reserved;
unsigned long mount_flags;
int tmp_flags; /* fix remount prototype... */
+ u8 sig[4];

pr_debug("AFFS: read_super(%s)\n",data ? (const char *)data : "no options");

@@ -370,8 +371,9 @@ got_root:
printk(KERN_ERR "AFFS: Cannot read boot block\n");
goto out_error;
}
- chksum = be32_to_cpu(*(__be32 *)boot_bh->b_data);
+ memcpy(sig, boot_bh->b_data, 4);
brelse(boot_bh);
+ chksum = be32_to_cpu(*(__be32 *)sig);

/* Dircache filesystems are compatible with non-dircache ones
* when reading. As long as they aren't supported, writing is
@@ -420,11 +422,17 @@ got_root:
}

if (mount_flags & SF_VERBOSE) {
- chksum = cpu_to_be32(chksum);
- printk(KERN_NOTICE "AFFS: Mounting volume \"%*s\": Type=%.3s\\%c, Blocksize=%d\n",
- AFFS_ROOT_TAIL(sb, root_bh)->disk_name[0],
- AFFS_ROOT_TAIL(sb, root_bh)->disk_name + 1,
- (char *)&chksum,((char *)&chksum)[3] + '0',blocksize);
+ int len = AFFS_ROOT_TAIL(sb, root_bh)->disk_name[0];
+ char name[32];
+
+ if (len > 31)
+ len = 31;
+ memcpy(name, AFFS_ROOT_TAIL(sb, root_bh)->disk_name + 1, len);
+ name[len] = '\0';
+
+ printk(KERN_NOTICE "AFFS: Mounting volume \"%*s\": "
+ "Type=%c%c%c\\%c, Blocksize=%d\n",
+ len, name, sig[0], sig[1], sig[2], sig[3], blocksize);
}

sb->s_flags |= MS_NODEV | MS_NOSUID;
--
1.3.GIT
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2006-06-15 13:06    [W:0.077 / U:0.184 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site