diff options
| author | Peng Fan <[email protected]> | 2016-06-17 14:18:11 +0800 |
|---|---|---|
| committer | Marek Vasut <[email protected]> | 2016-06-18 00:48:31 +0200 |
| commit | 7839f5f8092762f048076c6c7b010c10f2ee8a0b (patch) | |
| tree | 6bce194048f0a09be82743d43769f7528aa938ec /cmd | |
| parent | ef53b8c4ce2919d6685d2dc0c51a91a180433ff0 (diff) | |
cmd: usb: check if_type before using this device
For legacy usb storage driver, USB_MAX_STOR_DEV is defined as 7.
If we only have one usb disk on board, `usb dev 0` is ok.
But if `usb dev 1`, still ok, then `usb read xxx` will trigger
system fault and reboot.
So check if_type before using this device.
Signed-off-by: Peng Fan <[email protected]>
Cc: Simon Glass <[email protected]>
Cc: Hans de Goede <[email protected]>
Cc: Bin Meng <[email protected]>
Cc: Marek Vasut <[email protected]>
Cc: Stefan Roese <[email protected]>
Cc: Marcel Ziswiler <[email protected]>
Cc: Peng Fan <[email protected]>
Cc: Stephen Warren <[email protected]>
Diffstat (limited to 'cmd')
| -rw-r--r-- | cmd/usb.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/cmd/usb.c b/cmd/usb.c index b83d3233b78..58d9db29d73 100644 --- a/cmd/usb.c +++ b/cmd/usb.c @@ -800,7 +800,8 @@ static int do_usb(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) int dev = (int)simple_strtoul(argv[2], NULL, 10); printf("\nUSB device %d: ", dev); stor_dev = blk_get_devnum_by_type(IF_TYPE_USB, dev); - if (stor_dev == NULL) { + if ((stor_dev == NULL) || + (stor_dev->if_type == IF_TYPE_UNKNOWN)) { printf("unknown device\n"); return 1; } |