Weird bug related to message ids

---------

From: Jose Kahan (jose.kahan@w3.org)
Date: Tue Jun 10 2003 - 09:21:39 CDT


Maybe you know what is wrong here.

-----
The test
-----
I'm testing hypermail as follows:

1. Convert a mailbox with some messages as follows:

        hypermail -d dirname -m /path/to/mailbox

2. I then try to add a new message as follows:

        hypermail -d dirname -u -m message

-----
The problem
-----

One of the messages in the mailbox has the following msgid (showing how
it is stored in hypermail):

<!-- id="000e01bfd579$76981260$956bfac1@c6y7d4" -->

It corresponds to the original value.

When doing 2), I get the following warning message:

Message-ID is missing, ignoring message with subject ....

------
I traced this bug to the following code on parse.c:2836

else if (!strcasecmp(command, "id")) {
    char *raw_msgid = getvalue(line);
    msgid = unspamify(raw_msgid);
    if (raw_msgid) free(raw_msgid);
    if (msgid && (!isalnum(*msgid) || !strchr(msgid,'.')
                  || !strstr(line,"-->")) && set_linkquotes)
       msgid = NULL;/* old version of Hypermail wrote junk? */ }

Any idea why we're using the isalnum function there? I converted
this archive from zero with the same version of hypermail that's
complaining about this test against an old version of hypermail.

I don't have enough context to fix this problem. The msgid value
looks correct to me.

-jose


---------

This archive was generated by hypermail 2.1.7.