From d8c64a2ad97d050f7011aab1ef153316ccd6a3b8 Mon Sep 17 00:00:00 2001 From: Olev Kartau Date: Fri, 16 Mar 2012 09:05:20 +0200 Subject: [PATCH] gfx: pvr: fw_version should compare 4 integers instead of 4 bytes sizeof() was misused in fw_version_supported; If array is given as argument, the size of it is size of pointer, so comparison used only first 4 bytes instead of 4 integers, which made all entries to match. Signed-off-by: Olev Kartau Reviewed-by: Jani Nikula Signed-off-by: Kirill A. Shutemov --- drivers/staging/mrst/pvr/services4/srvkm/devices/sgx/sgxinit.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/mrst/pvr/services4/srvkm/devices/sgx/sgxinit.c b/drivers/staging/mrst/pvr/services4/srvkm/devices/sgx/sgxinit.c index dccb448..2d8e7a9 100644 --- a/drivers/staging/mrst/pvr/services4/srvkm/devices/sgx/sgxinit.c +++ b/drivers/staging/mrst/pvr/services4/srvkm/devices/sgx/sgxinit.c @@ -1987,7 +1987,7 @@ static int fw_version_supported(const int fw_ver[4]) int i; for (i = 0; i < ARRAY_SIZE(sup_versions); i++) - if (!memcmp(fw_ver, sup_versions[i], sizeof(fw_ver))) + if (!memcmp(fw_ver, sup_versions[i], 4 * sizeof(int))) return 1; return 0; -- 2.7.4