flywithu/ntel-gpu-tools
1
0
Fork 0
mirror of https://github.com/tiagovignatti/intel-gpu-tools.git synced 2025-06-12 02:16:17 +00:00
Code Issues Releases Wiki Activity

intel_gpu_frequency: fix array bounds warning

Browse Source 
Add a parameter for the size of the act_upon array in the parse function
since its size cannot be calculated with ARRAY_SIZE from just the
pointer.

Cc: Ben Widawsky <benjamin.widawsky@intel.com>
Signed-off-by: Thomas Wood <thomas.wood@intel.com>
This commit is contained in:
Thomas Wood 2015-01-13 11:50:02 +00:00
parent a263cc8795
commit c1d8293cc7
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
tools/intel_gpu_frequency.c
View File

@ -179,7 +179,7 @@ version(const char *prog)
/* Returns read or write operation */ /* Returns read or write operation */
static bool static bool
parse(int argc, char *argv[], bool *act_upon, int *new_freq) parse(int argc, char *argv[], bool *act_upon, size_t act_upon_n, int *new_freq)
{ {
int c, tmp; int c, tmp;
bool write = false; bool write = false;
@ -193,7 +193,7 @@ parse(int argc, char *argv[], bool *act_upon, int *new_freq)
/* No args means -g" */ /* No args means -g" */
if (argc == 1) { if (argc == 1) {
for (c = 0; c < ARRAY_SIZE(act_upon); c++) for (c = 0; c < act_upon_n; c++)
act_upon[c] = true; act_upon[c] = true;
goto done; goto done;
} }
@ -233,7 +233,7 @@ parse(int argc, char *argv[], bool *act_upon, int *new_freq)
} }
} else { } else {
int i; int i;
for (i = 0; i < ARRAY_SIZE(act_upon); i++) for (i = 0; i < act_upon_n; i++)
act_upon[i] = true; act_upon[i] = true;
} }
break; break;
@ -333,7 +333,7 @@ int main(int argc, char *argv[])
devid = intel_get_drm_devid(drm_open_any()); devid = intel_get_drm_devid(drm_open_any());
device = drm_get_card(); device = drm_get_card();
write = parse(argc, argv, targets, set_freq); write = parse(argc, argv, targets, ARRAY_SIZE(targets), set_freq);
fail = write; fail = write;
/* If we've previously locked the frequency, we need to make sure to set things /* If we've previously locked the frequency, we need to make sure to set things