Btrfs: avoid buffer overrun in btrfs_printk
authorJim Meyering <jim@meyering.net>
Thu, 26 Apr 2012 16:35:12 +0000 (18:35 +0200)
committerJosef Bacik <josef@redhat.com>
Wed, 30 May 2012 14:23:31 +0000 (10:23 -0400)
The buffer read-overrun would be triggered by a printk format
starting with <N>, where N is a single digit.  NUL-terminate
after strncpy.  Use memcpy, not strncpy, since we know the
string we're copying fits in the destination buffer and
contains no NUL byte.

Signed-off-by: Jim Meyering <meyering@redhat.com>
fs/btrfs/super.c

index 2cd32175753d8217324f6263ea8d153b3adaffe4..46b26650415f1fb8e12a13283c2b475069bdf888 100644 (file)
@@ -188,7 +188,8 @@ void btrfs_printk(struct btrfs_fs_info *fs_info, const char *fmt, ...)
        va_start(args, fmt);
 
        if (fmt[0] == '<' && isdigit(fmt[1]) && fmt[2] == '>') {
-               strncpy(lvl, fmt, 3);
+               memcpy(lvl, fmt, 3);
+               lvl[3] = '\0';
                fmt += 3;
                type = logtypes[fmt[1] - '0'];
        } else