diff options
author | Kevin Cernekee <cernekee@gmail.com> | 2014-11-09 11:55:47 +0300 |
---|---|---|
committer | Rob Herring <robh@kernel.org> | 2014-11-18 20:35:02 +0300 |
commit | ab74d00a39f70e1bc34a01322bb59f3750ca7a8c (patch) | |
tree | 22e4264309f563decae0165926e1171fd98be14c /Documentation/devicetree/bindings/arm/brcm-brcmstb.txt | |
parent | 66865de4314caca30598244b86817e774c188afa (diff) | |
download | linux-ab74d00a39f70e1bc34a01322bb59f3750ca7a8c.tar.xz |
of: Fix crash if an earlycon driver is not found
__earlycon_of_table_sentinel.compatible is a char[128], not a pointer, so
it will never be NULL. Checking it against NULL causes the match loop to
run past the end of the array, and eventually match a bogus entry, under
the following conditions:
- Kernel command line specifies "earlycon" with no parameters
- DT has a stdout-path pointing to a UART node
- The UART driver doesn't use OF_EARLYCON_DECLARE (or maybe the console
driver is compiled out)
Fix this by checking to see if match->compatible is a non-empty string.
Signed-off-by: Kevin Cernekee <cernekee@gmail.com>
Cc: <stable@vger.kernel.org> # 3.16+
Signed-off-by: Rob Herring <robh@kernel.org>
Diffstat (limited to 'Documentation/devicetree/bindings/arm/brcm-brcmstb.txt')
0 files changed, 0 insertions, 0 deletions