GNATS does not use the builtin name "originator" in file-pr.c

[Carsten Prescher (EADS)]

Hello,

we have renamed the field Originator and got the following stack trace
of the gnatsd.

#0  0x40078f81 in kill () from /lib/libc.so.6
#1  0x40078c44 in raise () from /lib/libc.so.6
#2  0x40079ffd in abort () from /lib/libc.so.6
#3  0x08058b75 in set_field () at pr.c:963
#4  0x080525fd in createNewPRFile (pr=0x808cd70, flag_autocreate=1,
    err=0xbffff94c) at file-pr.c:145
#5  0x08053222 in submit_pr (database=0x8072410, fp=0x8088308,
err=0xbffff94c)
    at file-pr.c:959
#6  0x0804b797 in GNATS_subm (ac=0, av=0x8085a34) at cmds.c:510
#7  0x0804aa43 in serverMainLoop () at gnatsd.c:929
#8  0x0804ad75 in main (argc=1, argv=0x4017bae4) at gnatsd.c:1073
#9  0x4006754d in __libc_start_main () from /lib/libc.so.6

The bug seems to be in file-pr.c where the field name is used instead of
the builtin name. After applying this patch it seems to work:

--- gnats/file-pr.c.old 2003-03-02 11:15:24.000000000 +0100
+++ gnats/file-pr.c     2004-04-23 14:57:40.000000000 +0200
@@ -130,7 +130,7 @@
   /* If originator wasn't set, try to get the value from the From mail
      header.  This is what users usually expect. */
   {
-    FieldIndex idx_origin = find_field_index (database, "Originator");
+    FieldIndex idx_origin = findBuiltinField (database, "originator");
     const char *value_origin = (idx_origin == InvalidFieldIndex
                                ? NULL
                                : field_value (pr, idx_origin));


We use GNATS 4.0.


Kind regards

        Carsten