[PATCH 05/12] staging: lustre: Optimize error handling in class_register_type()

From: SF Markus Elfring
Date: Tue Jul 26 2016 - 15:06:07 EST


From: Markus Elfring <elfring@xxxxxxxxxxxxxxxxxxxxx>
Date: Tue, 26 Jul 2016 14:23:23 +0200

Return a constant error code without storing it in the local variable "rc"
after a failed memory allocation at the beginning of this function.

Signed-off-by: Markus Elfring <elfring@xxxxxxxxxxxxxxxxxxxxx>
---
drivers/staging/lustre/lustre/obdclass/genops.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/staging/lustre/lustre/obdclass/genops.c b/drivers/staging/lustre/lustre/obdclass/genops.c
index fd5e61f..4752091 100644
--- a/drivers/staging/lustre/lustre/obdclass/genops.c
+++ b/drivers/staging/lustre/lustre/obdclass/genops.c
@@ -152,10 +152,9 @@ int class_register_type(struct obd_ops *dt_ops, struct md_ops *md_ops,
return -EEXIST;
}

- rc = -ENOMEM;
type = kzalloc(sizeof(*type), GFP_NOFS);
if (!type)
- return rc;
+ return -ENOMEM;

type->typ_dt_ops = kzalloc(sizeof(*type->typ_dt_ops), GFP_NOFS);
if (!type->typ_dt_ops) {
--
2.9.2