[PATCH] EDAC/synopsys: Fix wrong return value of mc_probe()
Zhang Xiaoxu
zhangxiaoxu5 at huawei.com
Mon Nov 16 08:58:10 EST 2020
If create the inject sysfs file failed, we should return
the error, rather than 0. Otherwise, there maybe error
pointer access.
Signed-off-by: Zhang Xiaoxu <zhangxiaoxu5 at huawei.com>
---
drivers/edac/synopsys_edac.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/edac/synopsys_edac.c b/drivers/edac/synopsys_edac.c
index 12211dc040e8..7e7146b22c16 100644
--- a/drivers/edac/synopsys_edac.c
+++ b/drivers/edac/synopsys_edac.c
@@ -1344,7 +1344,8 @@ static int mc_probe(struct platform_device *pdev)
#ifdef CONFIG_EDAC_DEBUG
if (priv->p_data->quirks & DDR_ECC_DATA_POISON_SUPPORT) {
- if (edac_create_sysfs_attributes(mci)) {
+ rc = edac_create_sysfs_attributes(mci);
+ if (rc) {
edac_printk(KERN_ERR, EDAC_MC,
"Failed to create sysfs entries\n");
goto free_edac_mc;
--
2.25.4
More information about the linux-arm-kernel
mailing list