From 6ec9b69871445bd223f58871be31df0dd85c6a7a Mon Sep 17 00:00:00 2001
From: Morris Jette <jette@schedmd.com>
Date: Thu, 29 Sep 2016 19:19:10 -0600
Subject: [PATCH] Don't re-use a variable
Counter became invalid due to re-use of a variable within a loop.
---
src/srun/libsrun/launch.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/srun/libsrun/launch.c b/src/srun/libsrun/launch.c
index dffb6ac4070..7acb7134dce 100644
--- a/src/srun/libsrun/launch.c
+++ b/src/srun/libsrun/launch.c
@@ -161,7 +161,7 @@ extern int launch_common_create_job_step(srun_job_t *job, bool use_all_cpus,
void (*signal_function)(int),
sig_atomic_t *destroy_job)
{
- int i, rc;
+ int i, j, rc;
unsigned long step_wait = 0, my_sleep = 0;
uint16_t base_dist;
@@ -352,8 +352,8 @@ extern int launch_common_create_job_step(srun_job_t *job, bool use_all_cpus,
"retrying");
}
xsignal_unblock(sig_array);
- for (i = 0; sig_array[i]; i++)
- xsignal(sig_array[i], signal_function);
+ for (j = 0; sig_array[j]; j++)
+ xsignal(sig_array[j], signal_function);
my_sleep = (getpid() % 1000) * 100 + 100000;
} else {
verbose("Job step creation still disabled, retrying");
--
GitLab