Skip to content
Snippets Groups Projects
Commit c169c935 authored by Danny Auble's avatar Danny Auble
Browse files

BGQ - Close very small window where a step could of been removed before the

runjob happened, and the step was part of an array.  This is an addition to
commit 49e0f5f2
parent 49e0f5f2
No related branches found
No related tags found
No related merge requests found
...@@ -258,6 +258,18 @@ void Plugin::execute(bgsched::runjob::Verify& verify) ...@@ -258,6 +258,18 @@ void Plugin::execute(bgsched::runjob::Verify& verify)
} else } else
step_ptr = &step_resp->job_steps[0]; step_ptr = &step_resp->job_steps[0];
if ((uint32_t)runjob_job->job_id != step_ptr->job_id) {
message = "Step returned is for a different job "
+ boost::lexical_cast<std::string>(step_ptr->job_id)
+ "."
+ boost::lexical_cast<std::string>(step_ptr->step_id)
+ " != "
+ boost::lexical_cast<std::string>(runjob_job->job_id)
+ "."
+ boost::lexical_cast<std::string>(runjob_job->step_id);
goto deny_job;
}
/* A bit of verification to make sure this is the correct user /* A bit of verification to make sure this is the correct user
supposed to be running. supposed to be running.
*/ */
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment