diff mbox

btrfs: fix static checker warnings

Message ID 528D2E9F.6070101@suse.com
State Accepted, archived
Headers show

Commit Message

Jeff Mahoney Nov. 20, 2013, 9:50 p.m. UTC
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>
---
 fs/btrfs/extent-tree.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Kees Cook Nov. 20, 2013, 10:16 p.m. UTC | #1
On Wed, Nov 20, 2013 at 1:50 PM, Jeff Mahoney <jeffm@suse.com> wrote:
> 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>

Thanks!

-Kees
diff mbox

Patch

--- a/fs/btrfs/extent-tree.c	2013-11-20 12:21:23.969683074 -0500
+++ b/fs/btrfs/extent-tree.c	2013-11-20 14:43:34.494306159 -0500
@@ -6119,7 +6119,7 @@  static const char *btrfs_raid_type_names
 	[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;
@@ -8328,7 +8328,7 @@  static void __link_block_group(struct bt
 
 		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");