From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ovidiu Panait <ovpanait@gmail.com>
Date: Sun, 5 Apr 2020 20:08:01 +0300
Subject: [PATCH 1/2] dm: dump.c: Fix segfault when entry->of_match is NULL
In-Reply-To: <782e5239-06eb-6eff-243e-8c08a0b914e3@gmail.com>
References: <20200405164741.11243-1-ovpanait@gmail.com>
 <782e5239-06eb-6eff-243e-8c08a0b914e3@gmail.com>
Message-ID: <72eea9f2-2a72-d731-7eb6-18b656bfbba1@gmail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On 05.04.2020 19:53, Sean Anderson wrote:

> On 4/5/20 12:47 PM, Ovidiu Panait wrote:
>> Currently, dm drivers command produces a segfault:
>> => dm drivers
>> Driver                Compatible
>> --------------------------------
>> Segmentation fault (core dumped)
>>
>> This is caused by a NULL pointer dereference of entry->of_match.
>> Add a check to prevent this.
> This should have been fixed in version 3 of the original patch [1]. Did
> it not get merged properly? This is the second time I've been CC'd by
> someone who wants to fix this.
>
> [1] https://patchwork.ozlabs.org/patch/1234460/

Yes, it seems that an older version of the patch was merged:

https://github.com/u-boot/u-boot/commit/7b9d60fc1ff67b3959a7db394084b27268a7686d


Ovidiu

> --Sean