From:	CRDGW2::CRDGW2::MRGATE::"SMTP::VM1.NODAK.EDU::ANU-NEWS" 23-JAN-1992 02:26:42.50
To:	ARISIA::EVERHART
CC:	
Subj:	Bug report and fix for NNTP_XFER

From:	ANU-NEWS@VM1.NoDak.EDU@SMTP@CRDGW2
To:	EVERHART@ARISIA@MRGATE

Received:  by crdgw1.ge.com (5.57/GE 1.122)
	 id AA28273; Thu, 23 Jan 92 01:35:45 EST
Message-Id:  <9201230635.AA28273@crdgw1.ge.com>
Received: from NDSUVM1.BITNET by VM1.NoDak.EDU (IBM VM SMTP V2R2)
   with BSMTP id 7709; Wed, 22 Jan 92 23:42:17 CST
Received: from NDSUVM1.BITNET by NDSUVM1.BITNET (Mailer R2.07) with BSMTP id
 4911; Wed, 22 Jan 92 23:42:10 CST
Date:         Wed, 22 Jan 1992 10:28:58 GMT
Reply-To: "Karsten Nyblad, TFL" <karsten@TFL.DK>
Sender: ANU-NEWS Discussion <ANU-NEWS@VM1.NoDak.EDU>
From: "Karsten Nyblad, TFL" <karsten@TFL.DK>
Subject:      Bug report and fix for NNTP_XFER
To: Multiple recipients of list ANU-NEWS <ANU-NEWS@VM1.NoDak.EDU>

Hi,

NNTP_XFER handles Message-IDs longer than 59 characters erroneously.
NEWS ADD FILE store only 59 (IDLEN-1) characters in
NEWS_ROOT:NEWS.ITEMS, but NNTP_XFER assumes NEWS ADD FILE stores 60
characters.  Unfortunately NEWSBUILD purged my old version of
NNTP_XFER, so you will have to take the corrections like this.  (when
do people learn not to let software building command procedure purge
while building new versions, and not to purge without warning?)

id_check(id)
  char *id;
{
  char l_id[IDLEN + 4];
  int i;

  for (i = 0; i < (IDLEN + 4); ++i) l_id[i] = '\0';
  strncpy(l_id,id,IDLEN-1);   /* changed */
  l_id[IDLEN-1] = 0;          /* added */
  if (itm_check(l_id) || hist_check(l_id)) return(1);
  return(0);
}


Karsten Nyblad
TFL, A Danish Telecommunication Research Laboratory
E-mail: karsten@tfl.dk
