src/io.c: Use fork_exec() instead of system()

Speeds up execution, prevents failures when pager or log file contains
spaces, fixes a warning seen with "-Wunused-result".

Signed-off-by: Lukas Fleischer <calcurse@cryptocrack.de>

1 files changed, 4 insertions, 3 deletions

diff --git a/src/io.c b/src/io.c
index 1aa431b..6c981d4 100644
--- a/src/io.c
+++ b/src/io.c
@@ -1230,10 +1230,11 @@ io_log_display (struct io_file *log, char *msg, char *pager)
       ans = fgetc (stdin);
       if (ans == 'y')
         {
-          char cmd[BUFSIZ];
+          char *arg[] = { pager, log->name, NULL };
+          int pid;
 
-          snprintf (cmd, BUFSIZ, "%s %s", pager, log->name);
-          system (cmd);
+          if ((pid = fork_exec (NULL, NULL, pager, arg)))
+            child_wait (NULL, NULL, pid);
         }
     }
   else