[PATCH v2 4/4] drm/mediatek: Fix dereference before null check

Jason-JH Lin (林睿祥) Jason-JH.Lin at mediatek.com
Sun Jun 18 08:21:40 UTC 2023


Hi Angelo,


On Wed, 2023-06-14 at 10:43 +0200, AngeloGioacchino Del Regno wrote:
External email : Please do not click links or open attachments until you have verified the sender or the content.

Il 13/06/23 13:32, Jason-JH.Lin ha scritto:

> Null-checking state suggests that it may be null, but it has already

> been dereferenced on drm_atomic_get_new_plane_state(state, plane).

>

> The parameter state will never be NULL currently, so just remove the

> state is NULL flow in this function.

>

> Signed-off-by: Jason-JH.Lin <jason-jh.lin at mediatek.com>

> Fixes: 5ddb0bd4ddc3 ("drm/atomic: Pass the full state to planes async atomic check and update")


Fixes before S-o-b...


Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno at collabora.com>


Thanks, I'll swap it at the next version.

Regards,
Jason-JH.Lin

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20230618/cecd2d9c/attachment-0001.htm>


More information about the dri-devel mailing list