Very nearly a useful tool. Still generating zombies, however; much work in

wrapper to get this far, but still no bananas!

Makefile

@@ -5,7 +5,7 @@ CC= gcc
 CFLAGS= -g
 LD= gcc
 LDFLAGS= -g 
-BINDIR= /usr/local/bin
+HOMEDIR= /usr/local/etc/gild
 TARGETS= gild
 COMPONENTS = gild.o wrapper.o config.o log.o
 
@@ -21,7 +21,9 @@ clean:
 	rm core *.o $(TARGETS)
 
 install: $(TARGETS)
-	install --strip $(TARGETS) $(BINDIR)
+	install --strip $(TARGETS) gild.conf $(HOMEDIR)
+	install handlers/* $(HOMEDIR)/handlers
 
 version: $(TARGETS)
-	cvs commit gild.c wrapper.c config.c log.c Makefile gild.h gild.conf
+	cvs commit gild.c wrapper.c config.c log.c Makefile gild.h \
+	    gild.conf handlers

config.c

@@ -12,6 +12,8 @@
 *                                                                          *
 \**************************************************************************/
 
+/* $Header$ */
+
 #include "gild.h"
 
 extern char errorBuff[];	/* where I assemble logging messages */

gild.c

@@ -23,8 +23,6 @@ int port = DEFAULT_PORT_NO;	/* the port I shall listen on */
 char errorBuff[ 1024];		/* somewhere to assemble error messages */
 extern handler * handlers;
 				/* the list of handlers I support */
-FILE * logstream = stderr;	/* where I stick log messages */
-
 
 void error( int severity)
 /* log the current contents of errorBuff and then if severity is bad die */
@@ -49,8 +47,6 @@ int main( int argc, char * argv[])
 	  ( struct sockaddr_in *)malloc( sizeof( struct sockaddr_in));
 				/* the address I bind the keyhole to */
 
-     fprintf( stderr, "%s starting...", GILD_ID);
-
      for ( arg = 1; argc > arg; arg ++)
      {				/* process arguments */
 	  switch ( argv[ arg][ 0])
@@ -115,8 +111,11 @@ int main( int argc, char * argv[])
 
 #ifndef DEBUG
 				/* we become a real class one daemon */
-     if ( fork() != 0) exit(0);             
+     if ( fork() != 0)		/* we're the parent */
-				/* the parent exits */
+     {
+	  exit(0);		/* so die */
+     }
+				/* otherwise (we're the child)... */
      setsid();			/* release the controlling tty */
 #endif
 
@@ -171,12 +170,15 @@ int main( int argc, char * argv[])
 
 	       wrapper( conversation, client_addr);
 				/* wrapper (q.v.) handles the conversation */
+
+	       exit( 0);	/* after completing wrapper, die */
 	       break;
 	  default:		/* I'm the parent */
 	       close( conversation); 
 	       free( client);	/* prevent memory bloat */
 	       if ( client_addr != null)
 		    free( client_addr);
+	       break;
 	  }
      }
 }

gild.conf

@@ -1,5 +1,6 @@
 # Sample GILD configuration file
 # $Header$
 # protocol  pattern	    handler			timeout
+
 http        ^.*HTTP/[0-1]   handlers/http     5
 crp	    CRP/1           handlers/crp		10

gild.h

@@ -40,7 +40,6 @@
 #define STDOUT_FILENO 1
 #define STDERR_FILENO 2
 
-
 #define ever (;;)
 #define null (((void *) 0))
 

handlers/http

@@ -73,14 +73,14 @@ then
 	case $command in
 	    "HEAD"|"Head"|"head")   exit 0;;
 	    "GET"|"Get"|"get"	)   cat $file;;
-	    *			)   echo "Whoops! Unexpected error!";;
 	esac
 
 	echo "$now: $REMOTE_HOST: $command: $rq_file" \
 		    >> $SERVER_ROOT/logs/access_log
-    else
+    else			    # didn't find it; report and log
 	cat $SERVER_ROOT/error/404.html
 	echo "$now: $REMOTE_HOST: No such file [$rq_file]" \
 		    >> $SERVER_ROOT/logs/error_log
     fi
 fi
+exit 0				    # yes, I know it _shouldn't_ be necessary

log.c

@@ -21,8 +21,12 @@
 int log( int level, char *message)
 /* hand this message over to the syslog daemon for recording */
 {
+#ifdef DEBUG
+     fprintf( stderr, "%s: DEBUG: %s\n", GILD_ID, message);
+#else
      openlog( GILD_NAME, 0, LOG_DAEMON);
      syslog( level, message);
      closelog();
+#endif
 }
      

wrapper.c

@@ -16,26 +16,27 @@
 
 #include "gild.h"
 
-char errorBuff[ 1024];
+extern char errorBuff[ 1024];
+extern char **environ;
 
-void die( void)			/* inherited from cgi-lib; a way of
+void die( void)			
-				   getting rid of errant programs */
+/* inherited from cgi-lib; a way of getting rid of errant programs */
 {
      sprintf( errorBuff, "potentially serious: handler over-ran alloted time");
      error( LOG_ERR);
+     exit( 1);			/* belt and braces; should never be called */
 }
 
 
 void wrapper( int conversation, char * client_id)
 /* conversation is the handle on an open socket communicating with a
-   client. What I want to do is quite simple, and there must be a
+   client. */
-   straightforward way of doing it: attach the conversation to both
-   the standard input and the standard output of a process, and then
-   exec the handler within that process... I can't (at present) find
-   an easy way of doing that, however */
 {
      char firstln[ 1024];
+     char * exec_args[1];	/* 'arguments' to pass to execve */
      handler * command = null;
+     pid_t child_pid;		/* the process number of the spawned child */
+     int child_status;		/* the status returned by wait() */
 
      recv( conversation, firstln, 80, MSG_PEEK);
 				/* get the first thing the client
@@ -67,11 +68,8 @@ void wrapper( int conversation, char * client_id)
      }
      else			/* did find one */
      {
-	  sprintf( errorBuff, 
+	  setenv( "REMOTE_HOST", client_id, 1);
-		  "using handler '%s' to handle %s request from %s", 
+				/* set up the handler environment */
-		  command->command, command->protocol, client_id);
-	  error( LOG_NOTICE);
-				/* log the request */
 
 	  if ( command->timeout != 0)
 				/* prevent runaway processes; if a
@@ -82,10 +80,46 @@ void wrapper( int conversation, char * client_id)
 	       alarm( command->timeout);
 	  } 
 
-	  setenv( "REMOTE_HOST", client_id, 1);
+	  child_pid = fork();
-				/* set up the handler environment */
 
-	  system( command->command);
+	  switch ( child_pid)
+	  {
+	  case -1:		/* blew it; whinge and die */
+	       sprintf( errorBuff, "failed to fork in wrapper()");
+	       error( LOG_ERR);
+	       break;
+	  case 0:		/* we're the child process */
+	       sprintf( errorBuff, 
+		       "using handler '%s' [%d] to handle %s request from %s", 
+		       command->command, ( int)getpid(), command->protocol, 
+		       client_id);
+	       error( LOG_NOTICE);
+				/* log the request, and... */
+
+	       exec_args[ 0] = command->command;
+	       exec_args[ 1] = null;
+				/* set up the dummy arguments */
+
+	       if ( execve( command->command, 
+			   exec_args, environ) == -1)
+				/* ...execute the command (shouldn't return) */
+	       {		/* if it did we've got an error */
+		    sprintf( errorBuff, 
+			"error [errno %d] whislt execing handler '%s' [%d]\n",
+			    errno, command->command, child_pid),
+		    error( LOG_ERR);
+	       }		    
+	       break;
+
+	  default:		/* we're the parent */
+	       wait( &child_status);
+				/* hang around for the child to die */
+	       sprintf( errorBuff, 
+		       "handler '%s' [%d] completed; wrapper exiting normally",
+		       command->command, child_pid),
+	       error( LOG_NOTICE);
+	       break;
+	  }
      }
 
      exit( 0);
@@ -94,4 +128,3 @@ void wrapper( int conversation, char * client_id)
 
 
 
-