diff options
author | Simon Goldschmidt <simon.k.r.goldschmidt@gmail.com> | 2019-07-12 20:58:18 +0200 |
---|---|---|
committer | Joe Hershberger <joe.hershberger@ni.com> | 2019-07-18 16:37:13 -0500 |
commit | cb58d18beb3cd65b66c20f913cb55c38c322457b (patch) | |
tree | 8afb0fdd3ae4c87d9cd880e1e137eff2fb8f9efd /cmd | |
parent | c3d9f3f899038e21a66e1e4a685abc0a5a1d5d9c (diff) | |
download | u-boot-cb58d18beb3cd65b66c20f913cb55c38c322457b.zip u-boot-cb58d18beb3cd65b66c20f913cb55c38c322457b.tar.gz u-boot-cb58d18beb3cd65b66c20f913cb55c38c322457b.tar.bz2 |
cmd: mdio: prevent data abort when no mdio bus is found
Calling 'mdio read ...' currently leads to a data abort when no mdio
bus is found.
To fix this, check if 'bus' is a valid pointer before accessing it.
Signed-off-by: Simon Goldschmidt <simon.k.r.goldschmidt@gmail.com>
Tested-by: Vladimir Oltean <olteanv@gmail.com>
Reviewed-by: Vladimir Oltean <olteanv@gmail.com>
Acked-by: Joe Hershberger <joe.hershberger@ni.com>
Diffstat (limited to 'cmd')
-rw-r--r-- | cmd/mdio.c | 5 |
1 files changed, 5 insertions, 0 deletions
@@ -268,6 +268,11 @@ static int do_mdio(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) break; } + if (!bus) { + puts("No MDIO bus found\n"); + return CMD_RET_FAILURE; + } + if (op[0] == 'l') { mdio_list_devices(); |