From: Frederic M. <fre...@wo...> - 2006-02-14 09:52:37
|
Matthias Andree wrote: > Rob MacGregor <rob...@gm...> writes: > > >> On 2/13/06, Frederic Marchal <fre...@wo...> wrote: >> >>> Hello, >>> >>> I found 15 lines like this in my log this morning and 3 more earlier >>> last week: >>> >>> fetchmail[1226]: incorrect header line found while scanning headers >>> >> You'll find threads about this in the list archive. I don't remember >> the outcome, but I've a feeling that 6.3.x handles these (illegal) >> messages differently. >> > > I don't recall having a fix for this particular issue, but 6.3.2 > generally has some dozens of bugfixes worth having anyways. I'll put > this on my 6.3.3 TODO, but don't think it will appear within the next > three weeks (unless someone else sends a patch). Don't hold your breath, > I'm busy ATM. > Well, I'm a C programmer and I may be able to help but I'm mainly a windows programmer (yes, I know, shame on me :-) ) and I'm not yet all too familiar with the Linux programming and especially with the patch mechanism and none at all with the diff tool. Nevertheless, I have identified the core of the problem (its the call to readheaders in drive.c and the response to PS_REFUSED). I still have to check what happens if an invalid FROM or TO header is passed after readheaders but I think I can fix the program. I'll also need some direction about the proper way of doing it for a good integration in the current source. If you think it is reasonable, we can discuss this further on the fetchmail-devel list. Frederic |