tools/power/x86/intel-speed-select: Fix off by one check
authorDan Carpenter <dan.carpenter@oracle.com>
Mon, 18 Jul 2022 13:17:17 +0000 (16:17 +0300)
committerHans de Goede <hdegoede@redhat.com>
Thu, 28 Jul 2022 17:41:07 +0000 (19:41 +0200)
Change > MAX_DIE_PER_PACKAGE to >= MAX_DIE_PER_PACKAGE to prevent
accessing one element beyond the end of the array.

Fixes: 7fd786dfbd2c ("tools/power/x86/intel-speed-select: OOB daemon mode")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Reviewed-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
tools/power/x86/intel-speed-select/isst-daemon.c

index dd37292..d0400c6 100644 (file)
@@ -41,7 +41,7 @@ void process_level_change(int cpu)
        time_t tm;
        int ret;
 
-       if (pkg_id >= MAX_PACKAGE_COUNT || die_id > MAX_DIE_PER_PACKAGE) {
+       if (pkg_id >= MAX_PACKAGE_COUNT || die_id >= MAX_DIE_PER_PACKAGE) {
                debug_printf("Invalid package/die info for cpu:%d\n", cpu);
                return;
        }