btrfs: fix static checker warnings
authorJeff Mahoney <jeffm@suse.com>
Wed, 20 Nov 2013 21:50:23 +0000 (16:50 -0500)
committerChris Mason <clm@fb.com>
Tue, 28 Jan 2014 21:19:44 +0000 (13:19 -0800)
This patch fixes the following warnings:
fs/btrfs/extent-tree.c:6201:12: sparse: symbol 'get_raid_name' was not declared. Should it be static?
fs/btrfs/extent-tree.c:8430:9: error: format not a string literal and no format arguments [-Werror=format-security] get_raid_name(index));

Signed-off-by: Jeff Mahoney <jeffm@suse.com>
Reviewed-by: Kees Cook <keescook@chromium.org>
Signed-off-by: Josef Bacik <jbacik@fb.com>
Signed-off-by: Chris Mason <clm@fb.com>
fs/btrfs/extent-tree.c

index fe651f4d1b93d9f07f59068ee525b2874eb75a41..f08f6dda949f0ee16569edd0b562d154cdd4b48f 100644 (file)
@@ -6195,7 +6195,7 @@ static const char *btrfs_raid_type_names[BTRFS_NR_RAID_TYPES] = {
        [BTRFS_RAID_RAID6]      = "raid6",
 };
 
-const char *get_raid_name(enum btrfs_raid_types type)
+static const char *get_raid_name(enum btrfs_raid_types type)
 {
        if (type >= BTRFS_NR_RAID_TYPES)
                return NULL;
@@ -8423,7 +8423,7 @@ static void __link_block_group(struct btrfs_space_info *space_info,
 
                kobject_get(&space_info->kobj); /* put in release */
                ret = kobject_init_and_add(kobj, &btrfs_raid_ktype,
-                                          &space_info->kobj,
+                                          &space_info->kobj, "%s",
                                           get_raid_name(index));
                if (ret) {
                        pr_warn("btrfs: failed to add kobject for block cache. ignoring.\n");