slurmsmwd - handle parsing issue when the inst or svc_id fields are set.

The format for this field is "<inst>:<svc_id>:<comp_id>"; if inst or svc_id
are unset then they're blank, and the parsing code had assumed they would
always be blank as the systems slurmsmwd were developed on never used them.

Bug 5411.

NEWS

@@ -44,6 +44,8 @@ documents those changes that are of interest to users and administrators.
  -- Fix invalid read when not using backup slurmctlds.
  -- Prevent acct coordinators from changing default acct on add user.
  -- Don't allow scontrol top do modify job priorities when priority == 1.
+ -- slurmsmwd - change parsing code to handle systems with the svid or inst
+    fields set in xtconsumer output.
 * Changes in Slurm 18.08.0
 ==========================

contribs/cray/slurmsmwd/main.c

@@ -236,8 +236,10 @@ static void _send_failed_nodes(char *nodelist)
 	while ((ptr = strtok_r(search, " ", &svptr))) {
 		search = NULL;
 
-		while (*ptr == ':')
-			ptr++;
+		ptr = strrchr(ptr, ':');
+		if (!ptr)
+			continue;
+		ptr++;
 
 		ptr = _trim(ptr);