drm/mediatek: Add cnt checking for coverity issue
authorJason-JH.Lin <jason-jh.lin@mediatek.com>
Fri, 14 Jul 2023 09:49:06 +0000 (17:49 +0800)
committerChun-Kuang Hu <chunkuang.hu@kernel.org>
Thu, 10 Aug 2023 23:47:49 +0000 (23:47 +0000)
commitd761b9450e31e5abd212f0085d424ed32760de5a
tree7a4356714bc97fe112df1cc453463be58b8dd31f
parent27b9e2ea3f2757da26bb8280e46f7fdbb1acb219
drm/mediatek: Add cnt checking for coverity issue

CERT-C Characters and Strings (CERT STR31-C)
all_drm_priv[cnt] evaluates to an address that could be at negative
offset of an array.

In mtk_drm_get_all_drm_priv():
Guarantee that storage for strings has sufficient space for character
data and the null terminator.

So change cnt to unsigned int and check its max value.

Fixes: 1ef7ed48356c ("drm/mediatek: Modify mediatek-drm for mt8195 multi mmsys support")
Signed-off-by: Jason-JH.Lin <jason-jh.lin@mediatek.com>
Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
Reviewed-by: CK Hu <ck.hu@mediatek.com>
Reviewed-by: Alexandre Mergnat <amergnat@baylibre.com>
Link: https://patchwork.kernel.org/project/dri-devel/patch/20230714094908.13087-3-jason-jh.lin@mediatek.com/
Signed-off-by: Chun-Kuang Hu <chunkuang.hu@kernel.org>
drivers/gpu/drm/mediatek/mtk_drm_drv.c