Call swaybg without invoking a shell

This makes escaping the arguments obsolete.

Also avoid dynamic memory allocation for the output id. It only supported ids up
to 99. Now we support up to 999, and take 4 bytes off the stack instead.

sway/config.c

@@ -334,17 +334,22 @@ void apply_output_config(struct output_config *oc, swayc_t *output) {
 		}
 
 		sway_log(L_DEBUG, "Setting background for output %d to %s", i, oc->background);
-		char *cmd = malloc(
+
-				strlen("swaybg ") +
+		size_t bufsize = 4;
-				(i >= 10 ? 2 : 1) +
+		char output_id[bufsize];
-				strlen(oc->background) + 3 +
+		snprintf(output_id, bufsize, "%d", i);
-				strlen(oc->background_option) + 3 +
+		output_id[bufsize-1] = 0;
-				1);
+
-		sprintf(cmd, "swaybg %d '%s' '%s'", i, oc->background, oc->background_option);
+		char *const cmd[] = {
+			"swaybg",
+			output_id,
+			oc->background,
+			oc->background_option,
+			NULL,
+		};
 		if (fork() == 0) {
-			execl("/bin/sh", "/bin/sh", "-c", cmd, (void *)NULL);
+			execvp(cmd[0], cmd);
 		}
-		free(cmd);
 	}
 }