sacctmgr: load dump prints incorrect default QOS when no QOS is set.

It is not mandatory to explicitly set a default QOS so print it as an empty
field on load as is done for show.

Bug 13311

NEWS

@@ -81,6 +81,7 @@ documents those changes that are of interest to users and administrators.
  -- Move the KeepAliveTime option into CommunicationParameters.
  -- slurm.spec - stop explicitly packaging pkgconfig directory to avoid a
     conflict with the pkgconfig package.
+ -- Fix sacctmgr load dump printing incorrect default QOS when no value is set.
 * Changes in Slurm 21.08.6
 ==========================

src/sacctmgr/association_functions.c

@@ -654,10 +654,15 @@ extern void sacctmgr_print_assoc_rec(slurmdb_assoc_rec_t *assoc,
 		if (!g_qos_list)
 			g_qos_list = slurmdb_qos_get(
 				db_conn, NULL);
-		tmp_char = slurmdb_qos_str(g_qos_list, assoc->def_qos_id);
-		if (!tmp_char)
-			tmp_char = print_acct =
-				xstrdup_printf("UNKN-%u", assoc->def_qos_id);
+		if (assoc->def_qos_id != NO_VAL) {
+			tmp_char = slurmdb_qos_str(g_qos_list,
+						   assoc->def_qos_id);
+			if (!tmp_char)
+				tmp_char = print_acct =
+					xstrdup_printf("UNKN-%u",
+						       assoc->def_qos_id);
+		} else
+			tmp_char = print_acct = xstrdup("");
 		field->print_routine(field, tmp_char, last);
 		xfree(print_acct);
 		break;