]> git.baikalelectronics.ru Git - kernel.git/commit
drm/vmwgfx: Fix two list_for_each loop exit tests
authorDan Carpenter <dan.carpenter@oracle.com>
Fri, 26 Jun 2020 10:39:59 +0000 (13:39 +0300)
committerRoland Scheidegger <sroland@vmware.com>
Tue, 11 Aug 2020 01:16:02 +0000 (03:16 +0200)
commit0542c15cb81b05c7908c6e0fcc541b5db4efe27f
treefb42b630996b00470a37d568b0726cef8c3c1bdd
parentefc88064ddca90b091c80c67f65be88d205a6307
drm/vmwgfx: Fix two list_for_each loop exit tests

These if statements are supposed to be true if we ended the
list_for_each_entry() loops without hitting a break statement but they
don't work.

In the first loop, we increment "i" after the "if (i == unit)" condition
so we don't necessarily know that "i" is not equal to unit at the end of
the loop.

In the second loop we exit when mode is not pointing to a valid
drm_display_mode struct so it doesn't make sense to check "mode->type".

Fixes: 4911095d5b17 ("drm/vmwgfx: Implement fbdev on kms v2")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Reviewed-by: Roland Scheidegger <sroland@vmware.com>
Signed-off-by: Roland Scheidegger <sroland@vmware.com>
drivers/gpu/drm/vmwgfx/vmwgfx_kms.c