This looks ok. I'm wondering however if these rtems_rtl_lock () ever
should fail in practice, should it be an assert? Just wondering.
On Tue, Mar 30, 2021 at 10:58 AM Ryan Long <ryan.l...@oarcorp.com> wrote:
>
> ping
>
> -----Original Message-----
> From: Ryan Long <ryan.l...@oarcorp.com>
> Sent: Monday, March 22, 2021 12:08 PM
> To: devel@rtems.org
> Cc: Ryan Long <ryan.l...@oarcorp.com>
> Subject: [PATCH 1/2] rtl-obj.c: Added an early return if rtl lock fails
>
> CID 1444138: Dereference null return value in rtems_rtl_obj_find_file().
>
> Closes #4332
> ---
> cpukit/libdl/rtl-obj.c | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/cpukit/libdl/rtl-obj.c b/cpukit/libdl/rtl-obj.c index
> a7dd740..d5a867e 100644
> --- a/cpukit/libdl/rtl-obj.c
> +++ b/cpukit/libdl/rtl-obj.c
> @@ -409,6 +409,10 @@ rtems_rtl_obj_find_file (rtems_rtl_obj* obj, const char*
> name)
>
> rtl = rtems_rtl_lock ();
>
> + if (rtl == NULL) {
> + return false;
> + }
> +
> if (!rtems_rtl_find_file (pname, rtl->paths, &obj->fname, &obj->fsize))
> {
> rtems_rtl_set_error (ENOENT, "file not found");
> --
> 1.8.3.1
>
> _______________________________________________
> devel mailing list
> devel@rtems.org
> http://lists.rtems.org/mailman/listinfo/devel
_______________________________________________
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel
