diff options
| author |   Imre Deak <imre.deak@intel.com> | 2017-07-19 16:46:32 +0300 |
|---|---|---|
| committer |   Daniel Vetter <daniel.vetter@ffwll.ch> | 2017-07-20 10:20:31 +0200 |
| commit | 636c4c3e762b62aa93632c645ca65879285b16e3 (patch) | |
| tree | 93c19d2df988aa981ce9dc99f73bed77dd37fbd9 /include | |
| parent | drm/mst: Avoid dereferencing a NULL mstb in drm_dp_mst_handle_up_req() (diff) | |
| download | linux-dev-636c4c3e762b62aa93632c645ca65879285b16e3.tar.xz linux-dev-636c4c3e762b62aa93632c645ca65879285b16e3.zip | |
drm/mst: Avoid processing partially received up/down message transactions
Currently we may process up/down message transactions containing
uninitialized data. This can happen if there was an error during the
reception of any message in the transaction, but we happened to receive
the last message correctly with the end-of-message flag set.
To avoid this abort the reception of the transaction when the first
error is detected, rejecting any messages until a message with the
start-of-message flag is received (which will start a new transaction).
This is also what the DP 1.4 spec 2.11.8.2 calls for in this case.
In addtion this also prevents receiving bogus transactions without the
first message with the the start-of-message flag set.
v2:
- unchanged
v3:
- git add the part that actually skips messages after an error in
drm_dp_sideband_msg_build()
Cc: Dave Airlie <airlied@redhat.com>
Cc: Lyude <lyude@redhat.com>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Imre Deak <imre.deak@intel.com>
Reviewed-by: Lyude <lyude@redhat.com>
Cc: stable@vger.kernel.org
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Link: https://patchwork.freedesktop.org/patch/msgid/20170719134632.13366-1-imre.deak@intel.com
Diffstat (limited to 'include')
0 files changed, 0 insertions, 0 deletions