diff options
author | Sebastian Andrzej Siewior <bigeasy@linutronix.de> | 2012-01-11 21:43:38 +0100 |
---|---|---|
committer | Nicholas Bellinger <nab@linux-iscsi.org> | 2012-01-18 09:33:44 +0100 |
commit | 8d9efe539cf78f6a90947d47100e4a86d907750f (patch) | |
tree | 82c9981047853687f83117d65de94ba18a25f144 /kernel/pid_namespace.c | |
parent | target: use save/restore lock primitive in core_dec_lacl_count() (diff) | |
download | linux-8d9efe539cf78f6a90947d47100e4a86d907750f.tar.xz linux-8d9efe539cf78f6a90947d47100e4a86d907750f.zip |
target: fix return code of core_tpg_.*_lun
- core_tpg_pre_addlun()
returns always ERR_PTR() or the pointer, never NULL. The additional
check for NULL in core_dev_add_lun() is not required.
- core_tpg_pre_dellun()
returns always ERR_PTR() or the pointer, never NULL. The check for NULL
in core_dev_del_lun() is wrong. The third argument (int *) is never
used, remove it.
- core_dev_add_lun()
returns always NULL or the pointer, never ERR_PTR. The check for
IS_ERR() is not required.
(nab: Convert core_dev_add_lun() use err.h macros for failure
handling to be consistent with the rest of target_core_fabric_configfs.c
callers)
Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Cc: <stable@vger.kernel.org>
Signed-off-by: Nicholas Bellinger <nab@linux-iscsi.org>
Diffstat (limited to 'kernel/pid_namespace.c')
0 files changed, 0 insertions, 0 deletions