diff options
author | David Gibson <david@gibson.dropbear.id.au> | 2019-05-06 17:00:07 +1000 |
---|---|---|
committer | David Gibson <david@gibson.dropbear.id.au> | 2019-05-06 17:00:07 +1000 |
commit | 8f695676227be42074711f3464d8c9f82bd80999 (patch) | |
tree | f3bcc35b8275d5ba1343c1e47b8d35bc3c124699 /tests | |
parent | 5c3513f68921cc15231af7ae6806c1f61c517c7c (diff) | |
download | dtc-8f695676227be42074711f3464d8c9f82bd80999.zip dtc-8f695676227be42074711f3464d8c9f82bd80999.tar.gz dtc-8f695676227be42074711f3464d8c9f82bd80999.tar.bz2 |
Avoid assertion in check_interrupts_property()
If running on a tree with an 'interrupt-parent' property which contains
an invalid phandle (0 or -1, not merely for a node which doesn't exist),
then check_interrupts_property() will trip the assertion in
get_node_by_phandle().
There's logic that almost detects this, but it only handles the overlay
case, where we can't fully check because the links will be fixed up later.
For the non-overlay case, this is definitely a bad property, but we
shouldn't crash. Fix it by failing the check early.
Fixes: c1e55a5513e9 ("checks: fix handling of unresolved phandles for dts plugins")
Fixes: ee3d26f6960b ("checks: add interrupts property check")
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Diffstat (limited to 'tests')
0 files changed, 0 insertions, 0 deletions