diff options
| author | Ovidiu Panait <ovpanait@gmail.com> | 2020-04-05 19:47:40 +0300 |
|---|---|---|
| committer | Simon Glass <sjg@chromium.org> | 2020-04-09 15:12:34 -0600 |
| commit | 02197fa749e21107e3330b2a244f7d5c455e456e (patch) | |
| tree | 7cb4f7bf253b1e89a8eee4962838c37ae6fd68f6 | |
| parent | dfd5321becc54d7ce9fd564aaaba70a2132c058e (diff) | |
| download | u-boot-02197fa749e21107e3330b2a244f7d5c455e456e.tar.gz | |
dm: dump.c: Fix segfault when entry->of_match is NULL
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.
Signed-off-by: Ovidiu Panait <ovpanait@gmail.com>
Cc: Sean Anderson <seanga2@gmail.com>
Cc: Simon Glass <sjg@chromium.org>
| -rw-r--r-- | drivers/core/dump.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/core/dump.c b/drivers/core/dump.c index e73ebeabcc..b5046398d4 100644 --- a/drivers/core/dump.c +++ b/drivers/core/dump.c @@ -107,7 +107,8 @@ void dm_dump_drivers(void) puts("Driver Compatible\n"); puts("--------------------------------\n"); for (entry = d; entry < d + n_ents; entry++) { - for (match = entry->of_match; match->compatible; match++) + for (match = entry->of_match; + match && match->compatible; match++) printf("%-20.20s %s\n", match == entry->of_match ? entry->name : "", match->compatible); |