diff options
| author | Dan Carpenter <[email protected]> | 2017-04-12 11:31:53 +0300 |
|---|---|---|
| committer | Greg Kroah-Hartman <[email protected]> | 2017-04-14 10:22:03 +0200 |
| commit | 73bd2eaa8785df85a90212c0841541231cc8e723 (patch) | |
| tree | 2e19d94b2beed009b3a5ac6c196f8340a1222026 | |
| parent | 33a2e390c4405f003fae5494c87dda45354b43cc (diff) | |
staging: lustre: checking for NULL instead if IS_ERR
lustre_cfg_new() returns error pointers and never NULL.
Signed-off-by: Dan Carpenter <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
| -rw-r--r-- | drivers/staging/lustre/lustre/obdclass/obd_config.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/staging/lustre/lustre/obdclass/obd_config.c b/drivers/staging/lustre/lustre/obdclass/obd_config.c index 8fce88f69039..6a7e7a7d2af1 100644 --- a/drivers/staging/lustre/lustre/obdclass/obd_config.c +++ b/drivers/staging/lustre/lustre/obdclass/obd_config.c @@ -1427,8 +1427,8 @@ int class_manual_cleanup(struct obd_device *obd) lustre_cfg_bufs_reset(&bufs, obd->obd_name); lustre_cfg_bufs_set_string(&bufs, 1, flags); lcfg = lustre_cfg_new(LCFG_CLEANUP, &bufs); - if (!lcfg) - return -ENOMEM; + if (IS_ERR(lcfg)) + return PTR_ERR(lcfg); rc = class_process_config(lcfg); if (rc) { |