bugfix: only abort test if in status 'preparing' or 'running'

server/webserver/user/include/libflocklab.php

@@ -1110,14 +1110,16 @@ function update_add_test($xml_config, &$errors, $existing_test_id = NULL, $abort
 					mysqli_query($db, $sql) or flocklab_die('Cannot abort test: ' . mysqli_error($db));
 					// update test entry
 					$end = $r['end_time'];
-					$sql =  'UPDATE `tbl_serv_tests` SET
-					`time_end_wish`="'.mysqli_real_escape_string($db, $end->format(DATE_ISO8601)).'",
-					`test_status`="aborting"
-					WHERE `serv_tests_key`='.$existing_test_id;
-					mysqli_query($db, $sql) or flocklab_die('Cannot store test configuration in database because: ' . mysqli_error($db));
-					$testId = $existing_test_id;
-					mysqli_close($db);
-					add_resource_allocation($testId, $resources, $r['start_time']);
+					// only schedule abort procedure if test has been started and not yet finished
+					$sql = "SELECT `test_status` FROM tbl_serv_tests WHERE `serv_tests_key`=".$existing_test_id." AND `test_status` IN ('running', 'preparing')";
+					$res = mysqli_query($db, $sql);
+					if (mysqli_num_rows($res)) {
+						$sql = 'UPDATE `tbl_serv_tests` SET `time_end_wish`="'.mysqli_real_escape_string($db, $end->format(DATE_ISO8601)).'", `test_status`="aborting" WHERE `serv_tests_key`='.$existing_test_id;
+						mysqli_query($db, $sql) or flocklab_die('Cannot store test configuration in database because: ' . mysqli_error($db));
+						$testId = $existing_test_id;
+						mysqli_close($db);
+						add_resource_allocation($testId, $resources, $r['start_time']);
+					}
 				}
 				else {
 					if (!$r['feasible'])