Messages in this thread Patch in this message | | | Date | Thu, 12 Jun 2008 17:26:46 +0200 | From | Louis Rilling <> | Subject | [RFC][PATCH 1/3] configfs: Report errors in config_*_init_type_name() |
| |
config_item_set_name() may fail but its error code is not checked in config_*init_type_name().
This patch adds the missing error checking and make config_*_init_type_name() report errors.
Signed-off-by: Louis Rilling <Louis.Rilling@kerlabs.com> --- fs/configfs/item.c | 18 ++++++++++++++---- include/linux/configfs.h | 8 ++++---- 2 files changed, 18 insertions(+), 8 deletions(-)
Index: b/fs/configfs/item.c =================================================================== --- a/fs/configfs/item.c 2008-06-12 17:13:32.000000000 +0200 +++ b/fs/configfs/item.c 2008-06-12 17:13:35.000000000 +0200 @@ -112,22 +112,32 @@ int config_item_set_name(struct config_i EXPORT_SYMBOL(config_item_set_name); -void config_item_init_type_name(struct config_item *item, +int config_item_init_type_name(struct config_item *item, const char *name, struct config_item_type *type) { - config_item_set_name(item, name); + int error; + + error = config_item_set_name(item, name); + if (error) + return error; item->ci_type = type; config_item_init(item); + return 0; } EXPORT_SYMBOL(config_item_init_type_name); -void config_group_init_type_name(struct config_group *group, const char *name, +int config_group_init_type_name(struct config_group *group, const char *name, struct config_item_type *type) { - config_item_set_name(&group->cg_item, name); + int error; + + error = config_item_set_name(&group->cg_item, name); + if (error) + return error; group->cg_item.ci_type = type; config_group_init(group); + return 0; } EXPORT_SYMBOL(config_group_init_type_name); Index: b/include/linux/configfs.h =================================================================== --- a/include/linux/configfs.h 2008-06-12 17:13:32.000000000 +0200 +++ b/include/linux/configfs.h 2008-06-12 17:13:35.000000000 +0200 @@ -71,9 +71,9 @@ static inline char *config_item_name(str } extern void config_item_init(struct config_item *); -extern void config_item_init_type_name(struct config_item *item, - const char *name, - struct config_item_type *type); +extern int config_item_init_type_name(struct config_item *item, + const char *name, + struct config_item_type *type); extern struct config_item * config_item_get(struct config_item *); extern void config_item_put(struct config_item *); @@ -97,7 +97,7 @@ struct config_group { }; extern void config_group_init(struct config_group *group); -extern void config_group_init_type_name(struct config_group *group, +extern int config_group_init_type_name(struct config_group *group, const char *name, struct config_item_type *type); -- Dr Louis Rilling Kerlabs Skype: louis.rilling Batiment Germanium Phone: (+33|0) 6 80 89 08 23 80 avenue des Buttes de Coesmes http://www.kerlabs.com/ 35700 Rennes
| |