THREAD curiosity

Ken Murchison murch at andrew.cmu.edu
Wed Oct 8 18:16:15 EDT 2008


David Carter wrote:
> On Wed, 8 Oct 2008, David Carter wrote:
> 
>> It needs all three messages: this was the minimal test case. If I cut 
>> a few lines out of one of the DKIM-Signature: lines it starts to work 
>> again.
> 
> Think that I've found it:
> 
> index.c: index_get_ids()
> 
>     /* grab the References header */
>     if ((refstr = stristr(headers, "references:"))) {
> 
> Isn't a sensible thing to do given the following input:
> 
>> From dpc22 at magenta.csi.cam.ac.uk Tue Oct  7 16:23:52 2008 +0100
> DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple;
>     d=qualcomm.com; i=hardie at qualcomm.com; q=dns/txt;
>     s=qcdkim; t=1223306812; x=1254842812;
>     h=mime-version:message-id:in-reply-to:references:date:to:
>     from:subject:cc:content-type:x-ironport-av;
>     z=MIME-Version:=201.0|Message-ID:=20<p06240602c50fdb702e5d
>     @[10.227.68.106]>|In-Reply-To:=20<20081006101429.DF8E23A6
>     89D at core3.amsl.com>|References:=20<20081006101429.DF8E23A
>     689D at core3.amsl.com>|Date:=20Mon,=206=20Oct=202008=2008:2
>     6:46=20-0700|To:=20IAB=20Chair=20<iab-chair at ietf.org>,=20
>     <ietf at ietf.org>|From:=20Ted=20Hardie=20<hardie at qualcomm.c
>     om>|Subject:=20Re:=20Call=20for=20Comments:=20Principles
>     =20of=20Internet=20Host=20Configuration|CC:=20"iab at iab.or
>     g"=20<iab at iab.org>|Content-Type:=20text/plain=3B=20charse
>     t=3D"us-ascii"|X-IronPort-AV:=20E=3DMcAfee=3Bi=3D"5300,27
>     77,5398"=3B=20a=3D"10116819";
>     bh=1QWtA80vJrN81vPOqPHundKPVJ/ygOFCJOCyqNmmpjY=;
>     b=Gzi2jfH6WJXXTk0PCI8EAiAM7cGAixb4RaDe9uIkFLz3wtBHqTlfyM9J
>     a5VH3w2b7OjTY6IbsJ11cpC9yJK0A3WsAb6I4gp65WZKeZnfWwu6WvPGR
>     RjPUPcxjqoseCOPettCxdE9eOlAjTJtQXMo0PQGj5QEtMkrP8cqi/WnxK w=;
> Message-ID: <p06240602c50fdb702e5d@[10.227.68.106]>
> In-Reply-To: <20081006101429.DF8E23A689D at core3.amsl.com>
> References: <20081006101429.DF8E23A689D at core3.amsl.com>

Changing it to the following fixes it as long as References isn't the 
first header field:

if ((refstr = stristr(headers, "\r\nreferences:"))) {


If we're worried about it being the first header field, there is an easy 
test for that.  Comments?

-- 
Kenneth Murchison
Systems Programmer
Project Cyrus Developer/Maintainer
Carnegie Mellon University


More information about the Cyrus-devel mailing list