| Message ID | 20230424094446.105133-1-colin.i.king@gmail.com (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | rv: Fix addition on an uninitialized variable 'run' | expand |
On 24/04/2023 10:44, 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> > --- > tools/verification/rv/src/rv.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > 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[] = { > "", Oops, I forgot the V2, will re-send as a V3. My bad.
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> --- tools/verification/rv/src/rv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)