(execute): Prompted for a missing argument, but then passed a null

pointer to prexec() instead.  Some systems (GNU, Windows) deal
gracefully with printing null strings, others crash.

diff --git a/src/lib/player/player.c b/src/lib/player/player.c
index dc919f478f21e0695120ae4e56811f81efde34cf..3b9e789ed44fbb9f7422a06cc811b622f50db7ab 100644 (file)
--- a/src/lib/player/player.c
+++ b/src/lib/player/player.c
@@ -278,13 +278,11 @@ execute(void)
     failed = 0;
     redir = NULL;
 
+    /* FIXME should use raw argument here, to support UTF-8 file names */
     p = getstarg(player->argp[1], "File? ", buf);
-
     if (p == NULL || *p == '\0')
        return RET_SYN;
-
-    /* FIXME should use raw argument here, to support UTF-8 file names */
-    prexec(player->argp[1]);
+    prexec(p);
 
     while (!failed && status()) {
        if (recvclient(buf, sizeof(buf)) < 0)