diff options
| author | Daniel Vetter | 2017-03-30 15:32:53 +0200 |
|---|---|---|
| committer | Daniel Vetter | 2017-03-30 20:23:51 +0200 |
| commit | 031e5896dfdc24839d60f168f6e8560e294674d3 (patch) | |
| tree | e6936f46b2930d4f1372b72d0f9d2fe460fd61e0 /include/drm | |
| parent | drm: Convert cmpxchg(bool) back to a two step operation (diff) | |
| download | kernel-qcow2-linux-031e5896dfdc24839d60f168f6e8560e294674d3.tar.gz kernel-qcow2-linux-031e5896dfdc24839d60f168f6e8560e294674d3.tar.xz kernel-qcow2-linux-031e5896dfdc24839d60f168f6e8560e294674d3.zip | |
drm: Clear e after kfree in drm_mode_page_flip_ioctl
With the explicit retry loop static analyzers get confused by the
control flow and believe that e could be accessed after kfree. That's
not possible, but it's non-obvious, so let's clear it to NULL.
We already cleared e = NULL at the top of the function, so this is all
in line.
Cc: Julia Lawall <julia.lawall@lip6.fr>
Reported-by: Julia Lawall <julia.lawall@lip6.fr>
Fixes: 29dc0d1de182 ("drm: Roll out acquire context for the page_flip ioctl")
Cc: Harry Wentland <harry.wentland@amd.com>
Cc: Daniel Vetter <daniel.vetter@intel.com>
Cc: Jani Nikula <jani.nikula@linux.intel.com>
Cc: Sean Paul <seanpaul@chromium.org>
Reviewed-by: Harry Wentland <harry.wentland@amd.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/20170330133253.29500-1-daniel.vetter@ffwll.ch
Diffstat (limited to 'include/drm')
0 files changed, 0 insertions, 0 deletions