Eric Lee / smarc-uboot

Download as
Browse Code ยป

Commit 02197fa749e21107e3330b2a244f7d5c455e456e

Authored by Ovidiu Panait
Committed by Simon Glass
1 parent dfd5321bec
Exists in smarc_8mq_lf_v2020.04 and in 9 other branches 8qm-imx_v2020.04_5.4.70_2.3.0, emb_lf_v2022.04, pitx_8mp_lf_v2020.04, smarc-8m-android-10.0.0_2.6.0, smarc-8m-android-11.0.0_2.0.0, smarc-8mp-android-11.0.0_2.0.0, smarc_8mm-imx_v2020.04_5.4.24_2.1.0, smarc_8mp_lf_v2020.04, smarc_8mq-imx_v2020.04_5.4.24_2.1.0

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>

Showing 1 changed file with 2 additions and 1 deletions Side-by-side Diff

... ... @@ -107,7 +107,8 @@
107 107 puts("Driver Compatible\n");
108 108 puts("--------------------------------\n");
109 109 for (entry = d; entry < d + n_ents; entry++) {
110   - for (match = entry->of_match; match->compatible; match++)
  110 + for (match = entry->of_match;
  111 + match && match->compatible; match++)
111 112 printf("%-20.20s %s\n",
112 113 match == entry->of_match ? entry->name : "",
113 114 match->compatible);