diff options
author | Wei Yongjun <yongjun_wei@trendmicro.com.cn> | 2013-06-05 19:18:12 -0400 |
---|---|---|
committer | Tony Luck <tony.luck@intel.com> | 2013-06-05 19:18:12 -0400 |
commit | b8edb64119b4e8158f268e250dcb9919a3b7ccea (patch) | |
tree | 36947b668f06c81db8e68048517c2608a6d4b62d | |
parent | d683b96b072dc4680fc74964eca77e6a23d1fa6e (diff) |
ACPI, APEI, EINJ: Fix error return code in einj_init()
Fix to return -ENOMEM in the debugfs_create_xxx() error handling
case instead of 0, as done elsewhere in this function.
Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
Reviewed-by: Chen Gong <gong.chen@linux.intel.com>
Signed-off-by: Tony Luck <tony.luck@intel.com>
-rw-r--r-- | drivers/acpi/apei/einj.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/acpi/apei/einj.c b/drivers/acpi/apei/einj.c index 8d457b55c55a..2cc8e034a3c0 100644 --- a/drivers/acpi/apei/einj.c +++ b/drivers/acpi/apei/einj.c | |||
@@ -694,6 +694,7 @@ static int __init einj_init(void) | |||
694 | if (rc) | 694 | if (rc) |
695 | goto err_release; | 695 | goto err_release; |
696 | 696 | ||
697 | rc = -ENOMEM; | ||
697 | einj_param = einj_get_parameter_address(); | 698 | einj_param = einj_get_parameter_address(); |
698 | if ((param_extension || acpi5) && einj_param) { | 699 | if ((param_extension || acpi5) && einj_param) { |
699 | fentry = debugfs_create_x64("param1", S_IRUSR | S_IWUSR, | 700 | fentry = debugfs_create_x64("param1", S_IRUSR | S_IWUSR, |