On Fri, Jul 31, 2015 at 11:31 PM, Simon Glass <s...@chromium.org> wrote:
> This function needs to check the list has entries before traversing it.
> Fix this bug.
>
> Signed-off-by: Simon Glass <s...@chromium.org>
> ---
>
> Changes in v2:
> - Correct the return code to avoid a test failure
>
> drivers/core/uclass.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/core/uclass.c b/drivers/core/uclass.c
> index aba9880..ffe6995 100644
> --- a/drivers/core/uclass.c
> +++ b/drivers/core/uclass.c
> @@ -153,6 +153,8 @@ int uclass_find_device(enum uclass_id id, int index,
> struct udevice **devp)
> ret = uclass_get(id, &uc);
> if (ret)
> return ret;
> + if (list_empty(&uc->dev_head))
> + return -ENODEV;
>
> list_for_each_entry(dev, &uc->dev_head, uclass_node) {
> if (!index--) {
> --
Reviewed-by: Bin Meng <bmeng...@gmail.com>
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
