More work on Def-settings.
[egate.git] / kernel / urp.c
index 81c4c34..0e5ed8d 100644 (file)
@@ -1,8 +1,54 @@
-/* Gateway kernel
+/* Gateway kernel - User Request Processor
  * Europagate, 1995
  *
  * $Log: urp.c,v $
- * Revision 1.20  1995/03/03 17:19:17  adam
+ * Revision 1.34  1995/05/04 10:40:08  adam
+ * More work on Def-settings.
+ *
+ * Revision 1.33  1995/05/03  16:34:19  adam
+ * CCL def command, i.e. user definitions - saved as resource files.
+ *
+ * Revision 1.32  1995/05/03  07:37:46  adam
+ * CCL commands stop/continue implemented. New functions gw_res_{int,bool}
+ * are used when possible.
+ *
+ * Revision 1.31  1995/05/01  12:43:38  adam
+ * First work on resource monitor program.
+ *
+ * Revision 1.30  1995/04/20  16:10:47  adam
+ * Modified to work with non-blocking zass-api. Not using non-blocking
+ * facility yet.
+ *
+ * Revision 1.29  1995/04/19  16:01:58  adam
+ * Some hacks to get the FIFO communication work!! Isn't reliable.
+ * Resource gw.account added - default account info.
+ *
+ * Revision 1.28  1995/04/19  13:19:09  adam
+ * New command: account - for authentication.
+ *
+ * Revision 1.27  1995/04/19  10:46:19  adam
+ * Persistency works much better now. New command: status - history-like
+ *
+ * Revision 1.26  1995/04/19  07:31:12  adam
+ * First work on Z39.50 persistence.
+ *
+ * Revision 1.25  1995/04/17  09:34:33  adam
+ * Timeout (idletime) adjustable. Minor changes in kernel.
+ *
+ * Revision 1.24  1995/03/28  11:42:35  adam
+ * First use of string-queue utility.
+ *
+ * Revision 1.23  1995/03/28  08:01:28  adam
+ * FIFO existence is used to test for a running kernel.
+ *
+ * Revision 1.22  1995/03/27  12:51:05  adam
+ * New log level in use: GW_LOG_ERRNO.
+ *
+ * Revision 1.21  1995/03/27  08:24:04  adam
+ * First use of gip interface and gw-db.
+ * First work on eti program.
+ *
+ * Revision 1.20  1995/03/03  17:19:17  adam
  * Smarter presentation. Bug fix in email header interpretation.
  *
  * Revision 1.19  1995/03/02  09:32:11  adam
  *
  */
 
+/*
+   Todo:
+     info/status (other name?)
+     better persistence diagnostics
+     resource gw.path - use chdir call instead
+
+   Optional:
+     automatic information about target-aliases: name, query-support, etc.
+ */
+
 #include <stdio.h>
 #include <stdlib.h>
 #include <assert.h>
 #include <unistd.h>
 #include <fcntl.h>
 
+#include <lgets.h>
+#include <ttyemit.h>
+#include <strqueue.h>
 #include "kernel.h"
 
-#define LINE_MAX 256
+static void present (const char *set, int offset, int number,
+                     struct ccl_token *format_token);
+
+static char line_buf[LINE_MAX+1];
 
 static void put_esc_str (const char *s)
 {
@@ -88,14 +150,18 @@ static void put_esc_str (const char *s)
         tty_emit (*s++);
 }
 
-static int reopen_target (void)
+int reopen_target (void)
 {
     const char *v;
     if (info.zass)
         gw_log (GW_LOG_WARN, KERNEL_LOG, "Zass free...");
-    info.zass = zass_open (info.hostname, info.port);
+    gw_log (GW_LOG_DEBUG, KERNEL_LOG, "reopen_target");
+    info.zass = zass_open (info.hostname, info.port, NULL, /* complete */
+                           *info.account ? info.account : NULL);
     if (!info.zass)
     {
+        gw_log (GW_LOG_DEBUG, KERNEL_LOG, "Cannot connect to target %s:%d",
+                info.hostname, info.port);
         fprintf (reply_fd, "%s %s:%d\n", 
                  gw_res_get (info.kernel_res, "gw.err.connect",
                              "Cannot connect to target"),
@@ -113,35 +179,25 @@ static int reopen_target (void)
                          "Connection established to"),
              info.hostname, info.port);
     if (*info.databases)
-        fprintf (reply_fd, "%s:\n%s\n",
+        fprintf (reply_fd, "%s: %s\n",
                  gw_res_get (info.kernel_res, "gw.msg.databases",
                              "Available databases"),
                  info.databases);
     if (*info.database)
-        fprintf (reply_fd, "%s:\n%s\n",
+        fprintf (reply_fd, "%s: %s\n",
                  gw_res_get (info.kernel_res, "gw.msg.database",
                              "Selected databases"),
                  info.database);
+    if (info.setno >= 0)
+        fprintf (reply_fd, "set=%d\n", info.setno);
+    else
+        fprintf (reply_fd, "set=Default\n");
     return 0;
 }
 
-static char line_buf[LINE_MAX+1];
-
-static struct command_word {
+struct command_word {
     char *default_value;
     char *resource_suffix;
-} command_tab [] = 
-{
-{   "find", "find"},
-{   "show", "show"},
-{   "base", "base" },
-{   "help", "help" },
-{   "info", "info" },
-{   "continue", "continue" },
-{   "status", "status" },
-{   "cancel", "cancel" },
-{   "target", "target" },
-{   NULL, NULL }
 };
 
 static int command_search (struct command_word *tab, struct ccl_token *cmd,
@@ -215,17 +271,18 @@ static char *error_no_search (struct error_no_struct *tab, int no)
     return NULL;
 }
 
-static int email_header (FILE *inf, char *from_str, char *subject_str)
+static int email_header (struct str_queue *sq,
+                         char *from_str, char *subject_str)
 {
     *from_str = '\0';
-    *subject_str = '\0';
-    while (fgets (line_buf, LINE_MAX, inf))
+    *subject_str = '\0';    
+    while (str_queue_deq (sq, line_buf, LINE_MAX))
     {
         if (line_buf[0] == '\n')
             return 0;
-        if (strncmp (line_buf, "From ", 5) == 0)
+        if (memcmp (line_buf, "From ", 5) == 0)
             sscanf (line_buf+4, "%s", from_str);
-        if (strncmp (line_buf, "Subject: ", 9) == 0 &&
+        if (memcmp (line_buf, "Subject: ", 9) == 0 &&
             sscanf (line_buf+9, "%s", subject_str+1) == 1)
             strcpy (subject_str, line_buf+9);
     }
@@ -240,22 +297,25 @@ static void help_general (void)
 
 static int exec_help (struct ccl_token *list)
 {
+    static char *sep = "-------------------------------\\n";
     help_general ();
 
-#if 1
+    put_esc_str (sep);
     put_esc_str (gw_res_get (info.kernel_res, "gw.help.target",
                              "target <name> - selects a given target\n"));
 
+    put_esc_str (sep);
     put_esc_str (gw_res_get (info.kernel_res, "gw.help.base",
                              "base <base>..  - selects databases\n"));
 
+    put_esc_str (sep);
     put_esc_str (gw_res_get (info.kernel_res, "gw.help.find",
                              "find <query>   - performs a search request\n"));
 
+    put_esc_str (sep);
     put_esc_str (gw_res_get (info.kernel_res, "gw.help.show",
                              "show <spec>    - retrieves and displays "
                              "records\n"));
-#endif
     return 0;
 }
 
@@ -271,16 +331,24 @@ static void display_diag_error (int code, const char *addinfo)
              gw_res_get (info.kernel_res, str, ""), addinfo);
 }
 
-static int exec_find (struct ccl_token *list)
+static int exec_find (struct ccl_token *list, const char *search_str)
 {
     const struct zass_searchent *p;
     struct gw_user_set *us;
+    char setname[32];
+    int autoshow;
 
     struct ccl_rpn_node *rpn;
     int error;
     const char *pos;
 
-    us = user_set_add ("Default", -1);
+    if (info.setno == -1)
+        strcpy (setname, "Default");
+    else
+        sprintf (setname, "%d", info.setno);
+    info.next_position = 1;
+    if (info.setno >= 0)
+        info.setno++;
     rpn = ccl_find (info.bibset, list, &error, &pos);
     if (!rpn)
     {
@@ -313,17 +381,50 @@ static int exec_find (struct ccl_token *list)
     gw_log (GW_LOG_DEBUG, KERNEL_LOG, "Searching in database %s",
             info.database );
     assert (info.zass);
-    p = zass_search (info.zass, rpn, "Default", info.database );
+    p = zass_p_search (info.zass, rpn, setname, info.database, info.sets);
     if (!p)
+    {
+        fprintf (reply_fd, "%s\n",
+                 gw_res_get (info.kernel_res, "gw.err.search.fail",
+                             "Search fail. Connection closed"));
         return -1;
+    }
     if (p->errcode != -1)
     {
         display_diag_error (p->errcode, p->errstring);
         return -2;
     }
-    fprintf (reply_fd, "%d %s\n", p->num,
-             gw_res_get (info.kernel_res, "gw.msg.hits", "hit(s)"));
-    us->hits = p->num;
+    fprintf (reply_fd, "%d %s %s\n", p->num,
+             gw_res_get (info.kernel_res, "gw.msg.hits", "hit(s) in set "),
+             setname);
+    us = user_set_add (setname, p->num, info.database, rpn, 1, search_str);
+    autoshow = gw_res_int (info.kernel_res, "gw.auto.show", 0);
+    if (autoshow && p->num > 0)
+    {
+        if (autoshow > p->num)
+        {
+            present (setname, 1, p->num, NULL);
+            info.next_position = 1+p->num;
+        }
+        else
+        {
+            present (setname, 1, autoshow, NULL);
+            info.next_position = 1+autoshow;
+        }
+    }
+    return 0;
+}
+
+static int exec_account (struct ccl_token *list)
+{
+    if (list->kind != CCL_TOK_EOL)
+    {
+        int len = list->len;
+        memcpy (info.account, list->name, len);
+        info.target[len] = '\0';
+    }
+    else
+        *info.account = '\0';
     return 0;
 }
 
@@ -340,6 +441,25 @@ static int exec_target (struct ccl_token *list)
     return reopen_target ();
 }
 
+static void exec_status_r (struct gw_user_set *sp)
+{
+    if (!sp)
+        return;
+    exec_status_r (sp->prev);
+    fprintf (reply_fd, "%6s %7d %12.12s   %.50s\n", sp->name, sp->hits,
+             sp->database, sp->search_str);
+}
+
+static int exec_status (struct ccl_token *list)
+{
+    fprintf (reply_fd, "%s\n",
+             gw_res_get (info.kernel_res,
+                         "gw.msg.statusline",
+                         "  Name    Hits     Database   Find"));
+    exec_status_r (info.sets);
+    return 0;
+}
+
 static int exec_base (struct ccl_token *list)
 {
     struct ccl_token *li = list;
@@ -373,13 +493,6 @@ static int exec_base (struct ccl_token *list)
     return 0;
 }
 
-struct command_word show_tab [] = 
-{
-{   "f", "format"},
-{   "p", "position"},
-{   NULL, NULL }
-};
-
 static void present (const char *set, int offset, int number,
                      struct ccl_token *format_token)
 {
@@ -388,14 +501,13 @@ static void present (const char *set, int offset, int number,
     int max_number;
     char format_str[16];
     
-    max_number = atoi (gw_res_get (info.kernel_res, "gw.max.show", 
-                                   "200"));
+    max_number = gw_res_int (info.kernel_res, "gw.max.show", 200);
     if (number > max_number)
         number = max_number;
     gw_log (GW_LOG_DEBUG, KERNEL_LOG, "present in set %s", set);
     gw_log (GW_LOG_DEBUG, KERNEL_LOG, "present of %d records from offset %d",
             number, offset);
-    zp = zass_present(info.zass, (char *) set, offset, number);
+    zp = zass_p_present(info.zass, (char *) set, offset, number);
     if (zp)
     {
         int i;
@@ -413,12 +525,12 @@ static void present (const char *set, int offset, int number,
                      record_log_name );
             record_log_fd = open (path, O_WRONLY|O_CREAT|O_APPEND, 0666);
             if (record_log_fd == -1)
-                gw_log (GW_LOG_WARN, "Cannot open %s", path);
+                gw_log (GW_LOG_WARN|GW_LOG_ERRNO, "Cannot open %s", path);
         }
         fprintf (reply_fd, gw_res_get (info.kernel_res,
                                        "gw.msg.records",
-                                       "Got %d records"),
-                 zp->num);
+                                       "Got %d records from set %s"),
+                 zp->num, set);
         fprintf (reply_fd, "\n");
         for (i = 0, pp = zp->records; pp; pp = pp->next, i++)
         {
@@ -448,6 +560,9 @@ static void present (const char *set, int offset, int number,
             if (rec)
             {
 #if USE_FML
+                strcpy (format_str,
+                        gw_res_get (info.kernel_res,
+                                    "gw.display.format", ""));
                 if (format_token)
                 {
                     len = format_token->len;
@@ -456,7 +571,7 @@ static void present (const char *set, int offset, int number,
                     memcpy (format_str, format_token->name, len);
                     format_str[len] = '\0';
                 }
-                if (info.fml && format_token && 
+                if (info.fml && *format_str && 
                     (!strcmp (format_str, "0") || !strcmp (format_str, "1")
                     || !strcmp(format_str, "2")))
                 {
@@ -484,6 +599,12 @@ static void present (const char *set, int offset, int number,
 
 static int exec_show (struct ccl_token *list)
 {
+    static struct command_word show_tab [] = 
+    {
+    {   "f", "format"},
+    {   "p", "position"},
+    {   NULL, NULL }
+    };
     char tmp_str[20];
     struct ccl_token *set_token = NULL;
     struct ccl_token *format_token = NULL;
@@ -624,12 +745,20 @@ static int exec_show (struct ccl_token *list)
         us = user_set_search (NULL);
         if (us && us->hits != -1)    /* proper result-set? */
         {
-            default_show = atoi (gw_res_get (info.kernel_res,
-                                             "gw.default.show", "20"));
-            if (us->hits > default_show)
-                present (us->name, 1, default_show, format_token);
-            else if (us->hits > 0)
-                present (us->name, 1, us->hits, format_token);
+            default_show = gw_res_int (info.kernel_res, "gw.default.show", 20);
+            if (us->hits >= info.next_position + default_show)
+           {
+                present (us->name, info.next_position, default_show, 
+                        format_token);
+               info.next_position += default_show;
+           }
+            else if (us->hits >= info.next_position)
+           {
+                present (us->name, info.next_position, 
+                        us->hits - info.next_position + 1, 
+                        format_token);
+               info.next_position = us->hits + 1;
+            }
         }
         else                         /* display error message */
         {
@@ -642,37 +771,175 @@ static int exec_show (struct ccl_token *list)
     return 0;
 }
 
-static int exec_command (const char *str)
+static void show_def (void)
 {
+    fprintf (reply_fd, "format       %s\n", gw_res_get
+             (info.kernel_res, "gw.display.format", ""));
+    fprintf (reply_fd, "autoshow     %s\n", gw_res_get
+             (info.kernel_res, "gw.auto.show", ""));
+    fprintf (reply_fd, "defaultshow  %s\n", gw_res_get
+             (info.kernel_res, "gw.default.show", ""));
+    fprintf (reply_fd, "language     %s\n", gw_res_get
+             (info.kernel_res, "gw.language", ""));
+}
+
+static int exec_def (struct ccl_token *list)
+{
+    const char *name = NULL;
+    char value[128];
+    char fname[256];
+    static struct command_word def_tab [] =
+    {
+    {   "reset",       "reset" },
+    {   "show",        "show" },
+    {   "f",           "format"},
+    {   "autoshow",    "autoshow" },
+    {   "defaultshow", "defaultshow" },
+    {   "lang",        "language" },
+    {   NULL, NULL }
+    };
+
+    if (info.userid < 0)
+        return -1;
+    sprintf (fname, "%s/user.%d.r", gw_res_get (info.kernel_res,
+                                                "gw.path", "."),
+             info.userid);
+
+    if (list->kind == CCL_TOK_EOL) 
+        show_def ();
+    else
+    {
+        int setting_no = command_search (def_tab, list, "ccl.token.");
+
+        if (!setting_no)
+        {
+            fprintf (reply_fd, "Unknown setting in def\n");
+            return -1;
+        }
+        if (setting_no == 1)
+        {
+            unlink (fname);
+            read_kernel_res ();
+            return 0;
+        }
+        else if (setting_no == 2)
+        {
+            show_def ();
+            return 0;
+        }
+        list = list->next;
+        if (list->kind == CCL_TOK_EOL)
+            return 0;
+        strncpy (value, list->name, 127);
+        value[(size_t) list->len] = '\0';
+        switch (setting_no)
+        {
+        case 3:
+            name = "gw.display.format";
+            break;
+        case 4:
+            name = "gw.auto.show";
+            break;
+        case 5:
+            name = "gw.default.show";
+            break;
+        case 6:
+            name = "gw.language";
+            break;
+        default:
+            return 0;
+        }
+        gw_log (GW_LOG_DEBUG, KERNEL_LOG, "update file %s with %s=%s",
+                fname, name, value);
+        gw_res_put (info.kernel_res, "gw.username", info.from_str, fname);
+        gw_res_put (info.kernel_res, name, value, fname);
+        gw_res_commit (info.kernel_res, fname);
+        if (setting_no == 6)
+            read_kernel_res ();
+    }
+    return 0;
+}
+
+
+/*
+ * exec_command: parse and execute ccl command in str.
+ * str:      ccl command string
+ * stop_flag pointer to integer. On completion the integer
+ *           is 1 (stop) or 2 (continue); 0 (other command)
+ * return: 0 success; non-zero otherwise
+ */
+static int exec_command (const char *str, int *stop_flag)
+{
+    static struct command_word command_tab [] = 
+    {
+    {   "find",     "find"},
+    {   "show",     "show"},
+    {   "base",     "base" },
+    {   "help",     "help" },
+    {   "info",     "info" },
+    {   "continue", "continue" },
+    {   "status",   "status" },
+    {   "stop",     "stop" },
+    {   "target",   "target" },
+    {   "def",      "def" },
+    {   "account",  "account" },
+    {   NULL, NULL }
+    };
+
     struct ccl_token *cmd = ccl_tokenize (str);
     int no;
 
+    *stop_flag = 0;
     if (cmd->kind != CCL_TOK_EOL &&
         (no = command_search (command_tab, cmd, "ccl.command.")))
     {
-        if (!info.zass && no != 9 && no != 4)
-            reopen_target ();
-        fprintf (reply_fd, "\n> %s\n", str);
-        if (!info.zass && (no == 1 || no == 2 || no == 3))
+        if (no == 1 || no == 2 || no == 3) /* find/show/base? */
         {
-            fprintf (reply_fd, "%s\n",
-                     gw_res_get (info.kernel_res, "gw.err.no.target",
-                                 "No connection established - "
-                                 "command ignored"));
-            return 0;
+            if (!info.zass)                /* open if not already opened */
+                reopen_target ();
+            fprintf (reply_fd, "\n> %s\n", str);
+            if (!info.zass)                /* fail? */
+            {
+                fprintf (reply_fd, "%s\n",
+                         gw_res_get (info.kernel_res, "gw.err.no.target",
+                                     "No connection established - "
+                                     "command ignored"));
+                return 0;
+            }
         }
+        else
+            fprintf (reply_fd, "\n> %s\n", str);
+#if 0
+        ccl_token_and = gw_res_get (info.kernel_res, "ccl.token.and", "and");
+        ccl_token_or = gw_res_get (info.kernel_res, "ccl.token.or", "or");
+        ccl_token_not = gw_res_get (info.kernel_res, "ccl.token.not", "not");
+        ccl_token_set = gw_res_get (info.kernel_res, "ccl.token.set", "set");
+#endif
         switch (no)
         {
         case 1:
-            return exec_find (cmd->next);
+            return exec_find (cmd->next, str);
         case 2:
             return exec_show (cmd->next);
         case 3:
             return exec_base (cmd->next);
         case 4:
+            fprintf (reply_fd, "\n> %s\n", str);
             return exec_help (cmd->next);
+       case 6: /* continue */
+           *stop_flag = 2;
+           return 0;
+        case 7:
+            return exec_status (cmd->next);
+       case 8: /* stop */
+           *stop_flag = 1;
+           return 0;
         case 9:
             return exec_target (cmd->next);
+        case 10:
+            return exec_def (cmd->next);
+        case 11:
+            return exec_account (cmd->next);
         default:
             fprintf (reply_fd, "%s\n",
                      gw_res_get (info.kernel_res, "gw.err.unimplemented",
@@ -690,31 +957,31 @@ static int exec_command (const char *str)
     return 0;
 }
 
-int urp (FILE *inf)
+int urp_start (int continuation, struct str_queue *queue)
 {
-    char from_str[128];
     char subject_str[128];
-    int command_no = 0;
-    char *reply_fname = NULL;
 
-    if (email_header (inf, from_str, subject_str))
+    info.command_no = 0;
+    info.reply_fname = NULL;
+
+    if (email_header (queue, info.from_str, subject_str))
     {
         gw_log (GW_LOG_WARN, KERNEL_LOG, "No message body");
         return -1;
     }
     tty_init (stdout, 40, 70);
-    if (*from_str)
+    if (*info.from_str)
     {
-        reply_fname = tempnam (gw_res_get (info.kernel_res,
-                                           "gw.reply.tmp.dir", NULL),
-                               gw_res_get (info.kernel_res,
-                                           "gw.reply.tmp.prefix", "gwr"));
-                                                 
-        reply_fd = fopen (reply_fname, "w");
+        info.reply_fname = (char *)
+            tempnam (gw_res_get (info.kernel_res, "gw.reply.tmp.dir", NULL),
+                     gw_res_get (info.kernel_res,
+                                 "gw.reply.tmp.prefix", "gwr"));
+
+        reply_fd = fopen (info.reply_fname, "w");
         if (!reply_fd)
         {
-            gw_log (GW_LOG_FATAL, KERNEL_LOG, "Cannot create %s",
-                    reply_fname);
+            gw_log (GW_LOG_FATAL|GW_LOG_ERRNO, KERNEL_LOG, "Cannot create %s",
+                    info.reply_fname);
             return -1;
         }
         tty_init (reply_fd, 0, 0);
@@ -728,59 +995,83 @@ int urp (FILE *inf)
                                                    "gw.msg.subject",
                                                    "Your Query"));
         fprintf (reply_fd, "\n");
-        gw_log (GW_LOG_ACCT, KERNEL_LOG, "User start %s", from_str);
+        gw_log (GW_LOG_ACCT, KERNEL_LOG, "User start %s", info.from_str);
     }
     else
         gw_log (GW_LOG_WARN, KERNEL_LOG, "No From in email header");
     fprintf (reply_fd, "%s\n", gw_res_get (info.kernel_res, "gw.msg.greeting",
                                            "Email->Z39.50 gateway"));
-    while (fgets (line_buf, LINE_MAX, inf))
-    {
-        char *cp;
+    if (continuation)
+        fprintf (reply_fd, "%s\n", gw_res_get (info.kernel_res,
+                                              "gw.msg.cont",
+                                              "Continued..."));
+    return 0;
+}
 
+int urp_command (struct str_queue *queue)
+{
+    char *cp;
+    int stop_flag;
+
+    while (str_queue_deq (queue, line_buf, LINE_MAX))
+    {
         if (line_buf[0] == '\n')
-            if (command_no)
+            if (info.command_no)
+           {
+               while (str_queue_deq (queue, 0, 0))
+                   ;
                 break;
+           }
             else 
                 continue;
         if ((cp = strchr (line_buf, '\n')))
             *cp = '\0';
         gw_log (GW_LOG_ACCT, KERNEL_LOG, "cmd: %s", line_buf);
-        ccl_token_and = gw_res_get (info.kernel_res, "ccl.token.and", "and");
-        ccl_token_or = gw_res_get (info.kernel_res, "ccl.token.or", "or");
-        ccl_token_not = gw_res_get (info.kernel_res, "ccl.token.not", "not");
-        ccl_token_set = gw_res_get (info.kernel_res, "ccl.token.set", "set");
         if (isalpha (line_buf[0]))
-            exec_command (line_buf);
-        command_no++;
+       {
+            exec_command (line_buf, &stop_flag);
+           if (stop_flag)
+           {
+               info.command_no++;     /* prevent help! */
+                while (str_queue_deq (queue, line_buf, LINE_MAX))
+                    ;
+               return stop_flag;
+            }
+        }
+        info.command_no++;
     }
-    if (!command_no)
+    return 0;
+}
+
+void urp_end (void)
+{
+    if (!info.command_no)
     {
-        fprintf (reply_fd, "%s\n", gw_res_get (info.kernel_res,
-                                               "gw.err.nullbody",
-                                               "No body"));
+        fprintf (reply_fd, "%s\n", 
+                gw_res_get (info.kernel_res, "gw.err.nullbody", "No body"));
         help_general ();
     }
-    if (*from_str)
+    if (*info.from_str)
     {
         const char *mta;
         char cmd[256];
         int mta_code;
 
-        assert (reply_fname);
+        assert (info.reply_fname);
         fclose (reply_fd);
         reply_fd = stdout;
 
         mta = gw_res_get (info.kernel_res, "gw.reply.mta",
-                          "/usr/lib/sendmail");
-        sprintf (cmd, "%s %s < %s", mta, from_str, reply_fname);
+                         "/usr/lib/sendmail");
+        sprintf (cmd, "%s %s < %s", mta, info.from_str, info.reply_fname);
         
         mta_code = system (cmd);
         if (mta_code)
-            gw_log (GW_LOG_FATAL, KERNEL_LOG, "Reply '%s' got exit code %d",
-                    cmd, mta_code);
-        unlink (reply_fname);        
-        gw_log (GW_LOG_ACCT, KERNEL_LOG, "User end %s", from_str);
+            gw_log (GW_LOG_FATAL, KERNEL_LOG,
+                    "Reply '%s' got exit code %d", cmd, mta_code);
+       else
+            unlink (info.reply_fname);        
+        gw_log (GW_LOG_ACCT, KERNEL_LOG, "User end %s", info.from_str);
     }
-    return 0;
 }
+