diff options
| author | Yang Yingliang <yangyingliang@huawei.com> | 2022-06-28 22:02:49 +0800 |
|---|---|---|
| committer | Shawn Guo <shawnguo@kernel.org> | 2022-07-05 08:41:55 +0800 |
| commit | 63f98153075167a6c88d8ac90bfd26f1887c670c (patch) | |
| tree | 71df5ac55682cddc8d64aae37b091e598d1fec40 /drivers/soc | |
| parent | edf12b913abb69abce4cd6996398a4ce23e5b047 (diff) | |
soc: fsl: guts: check return value after calling of_iomap() in fsl_guts_get_soc_uid()
of_iomap() may return NULL, so we need check the return value.
Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: Yang Yingliang <yangyingliang@huawei.com>
Reviewed-by: Michael Walle <michael@walle.cc>
Signed-off-by: Shawn Guo <shawnguo@kernel.org>
Diffstat (limited to 'drivers/soc')
| -rw-r--r-- | drivers/soc/fsl/guts.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/drivers/soc/fsl/guts.c b/drivers/soc/fsl/guts.c index 8038c599ad83..6bf3e6a980ff 100644 --- a/drivers/soc/fsl/guts.c +++ b/drivers/soc/fsl/guts.c @@ -126,6 +126,10 @@ static u64 fsl_guts_get_soc_uid(const char *compat, unsigned int offset) return 0; sfp_base = of_iomap(np, 0); + if (!sfp_base) { + of_node_put(np); + return 0; + } uid = ioread32(sfp_base + offset); uid <<= 32; |