| Message ID | 20230424094730.105313-1-colin.i.king@gmail.com (mailing list archive) |
|---|---|
| State | Accepted |
| Headers | show |
| Series | [next,V3] rv: Fix addition on an uninitialized variable 'run' | expand |
On 4/24/23 11:47, Colin Ian King wrote: > The variable run is not initialized however it is being accumulated > by the return value from the call to ikm_run_monitor. Fix this by > initializing run to zero at the start of the function. > > Fixes: 4bc4b131d44c ("rv: Add rv tool") > > Signed-off-by: Colin Ian King <colin.i.king@gmail.com> Acked-by: Daniel Bristot de Oliveira <bristot@kernel.org> Thanks! -- Daniel
diff --git a/tools/verification/rv/src/rv.c b/tools/verification/rv/src/rv.c index e601cd9c411e..1ddb85532816 100644 --- a/tools/verification/rv/src/rv.c +++ b/tools/verification/rv/src/rv.c @@ -74,7 +74,7 @@ static void rv_list(int argc, char **argv) static void rv_mon(int argc, char **argv) { char *monitor_name; - int i, run; + int i, run = 0; static const char *const usage[] = { "",
The variable run is not initialized however it is being accumulated by the return value from the call to ikm_run_monitor. Fix this by initializing run to zero at the start of the function. Fixes: 4bc4b131d44c ("rv: Add rv tool") Signed-off-by: Colin Ian King <colin.i.king@gmail.com> --- V2: Initialize run rather than assign it from return from ikm_run_monitor V3: Add missing V3 and next in patch subject --- tools/verification/rv/src/rv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)