More updates to take care of warnings on Ubuntu systems (fixes for common sense warnings that should have come up om my Fedora system but didn't).

git-svn-id: file:///home/mbr/svn/fwknop/trunk@223 510a4753-2344-4c79-9c09-4d669213fbeb

server/extcmd.c

@@ -75,8 +75,11 @@ run_extcmd(char *cmd, char *so_buf, char *se_buf, size_t so_buf_sz, size_t se_bu
     /* Create the pipes we will use for getting stdout and stderr
      * from the child process.
     */
-    pipe(so);
-    pipe(se);
+    if(pipe(so) != 0)
+        return(EXTCMD_PIPE_ERROR);
+
+    if(pipe(se) != 0)
+        return(EXTCMD_PIPE_ERROR);
 
     /* Fork off a child process to run the command and provide its outputs.
     */

server/extcmd.h

@@ -33,7 +33,8 @@
  * may end up in.
 */
 enum {
-    EXTCMD_SELECT_ERROR             =   -2,
+    EXTCMD_SELECT_ERROR             =   -3,
+    EXTCMD_PIPE_ERROR               =   -2,
     EXTCMD_FORK_ERROR               =   -1,
     EXTCMD_SUCCESS_ALL_OUTPUT       = 0x00,
     EXTCMD_SUCCESS_PARTIAL_STDOUT   = 0x01,

server/fw_util.c

@@ -438,7 +438,8 @@ process_spa_request(fko_srv_options_t *opts, spa_data_t *spadat)
     struct fw_chain *snat_chain; /* We assign this later (if we need to). */
 
     int             status, res;
-    time_t          now, exp_ts;
+    time_t          now;
+    unsigned int    exp_ts;
 
     /* Parse and expand our access message.
     */