[PATCH] rtems-fdt/rtems-fdt.c: Fix bug in loop termination

Gedare Bloom gedare at rtems.org
Sat Mar 20 06:09:17 UTC 2021


This looks good to me. Is the bug also on 5? If so, a ticket should be
opened and a patch filed for 5.2.

If someone cares to push this please go ahead. I wouldn't get to it
until next week sometime. Niteesh, ping it on Tuesday if no one has
pushed.

On Wed, Mar 17, 2021 at 8:51 PM G S Niteesh Babu <niteesh.gs at gmail.com> wrote:
>
> The while loop, loops infinitely in case of raw FDT data.
> The loop condition (size) is not modified during iterations.
> ---
>  cpukit/libmisc/rtems-fdt/rtems-fdt.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/cpukit/libmisc/rtems-fdt/rtems-fdt.c b/cpukit/libmisc/rtems-fdt/rtems-fdt.c
> index 0ea365314f..7747ba9bf8 100644
> --- a/cpukit/libmisc/rtems-fdt/rtems-fdt.c
> +++ b/cpukit/libmisc/rtems-fdt/rtems-fdt.c
> @@ -580,7 +580,7 @@ rtems_fdt_load (const char* filename, rtems_fdt_handle* handle)
>          close (bf);
>          return -RTEMS_FDT_ERR_READ_FAIL;
>        }
> -      r -= size;
> +      size -= r;
>        buf += r;
>      }
>    }
> --
> 2.17.1
>
> _______________________________________________
> devel mailing list
> devel at rtems.org
> http://lists.rtems.org/mailman/listinfo/devel


More information about the devel mailing list