*BSD News Article 47752


Return to BSD News archive

Xref: sserve comp.unix.bsd.bsdi.misc:498 news.software.nntp:14721
Path: sserve!newshost.anu.edu.au!harbinger.cc.monash.edu.au!simtel!vtc.tacom.army.mil!news1.oakland.edu!news.concourse.com!braintree!news.sprintlink.net!howland.reston.ans.net!news.cac.psu.edu!news.math.psu.edu!barr
From: barr@math.psu.edu (Dave Barr)
Newsgroups: comp.unix.bsd.bsdi.misc,news.software.nntp
Subject: INN Bad Overview Problem
Supersedes: <3vm8s4$80k@dodgson.math.psu.edu>
Date: 1 Aug 1995 22:14:56 GMT
Organization: Mathematics Department, The Pennsylvania State University
Lines: 38
Distribution: inet
Message-ID: <3vm910$80n@dodgson.math.psu.edu>
References: <3vlt6f$g16@salt.ncinter.net> <3vlv26$g6d@salt.ncinter.net> <3vlvb4$g8c@salt.ncinter.net>
NNTP-Posting-Host: augusta.math.psu.edu

In article <3vlvb4$g8c@salt.ncinter.net>,
Ian Smith <iansmith@pepper.ncinter.net> wrote:
>The problem is in the news article itself.  There is a ctrl-m inside
>the refrences line.. obviously a bad header.  Shouldn't this be handled
>by inn or the overview system though?  I don't want to refuse articles
>with bad refrences lines, but would like the mangled line to be deleted
>before being processed.  Any way to do this?

Hm.  I haven't tested this, but try the following patch:
(depending on your INN version you may have some fuzz)

This will only fix expireover so it can correct the problem, a
corresponding fix to overchan (or innd) is needed.

*** expireover.c.orig	Tue Aug  1 18:09:18 1995
--- expireover.c	Tue Aug  1 18:10:12 1995
***************
*** 477,483 ****
  	    }
  	    (void)strcpy(hp->Header, p);
  	    for (p = hp->Header; *p; p++)
! 		if (*p == '\t' || *p == '\n')
  		    *p = ' ';
  	    hp->HasHeader = TRUE;
  	}
--- 477,483 ----
  	    }
  	    (void)strcpy(hp->Header, p);
  	    for (p = hp->Header; *p; p++)
! 		if (*p == '\t' || *p == '\n' || *p == '\r')
  		    *p = ' ';
  	    hp->HasHeader = TRUE;
  	}


Please let me know if this fixes the problem.

--Dave