Return error when bad separator is given for scontrol update job licenses.

NEWS

@@ -64,6 +64,7 @@ documents those changes that are of interest to users and administrators.
     destroying a mutex.
  -- Fix updating job priority on multiple partitions to be correct.
  -- Don't remove admin comment when updating a job.
+ -- Return error when bad separator is given for scontrol update job licenses.
 
 * Changes in Slurm 17.02.2
 ==========================

src/slurmctld/licenses.c

@@ -148,9 +148,12 @@ static List _build_license_list(char *licenses, bool *valid)
 			if ((token[i] == ':') || (token[i] == '*')) {
 				token[i++] = '\0';
 				num = (int32_t)strtol(&token[i], &end_num, 10);
+				if (*end_num != '\0')
+					 *valid = false;
+				break;
 			}
 		}
-		if (num < 0) {
+		if (num < 0 || !(*valid)) {
 			*valid = false;
 			break;
 		}