Possible bugs in rnews (inn 2.4.1)

[Russ Allbery]

Russ Allbery <rra at stanford.edu> writes:

> I think rnews needs to be studied a little for nul handling; it looks
> like at least the wire format conversion doesn't work properly when the
> article contains a nul character.  I haven't made any more fixes to this
> at the moment, though.

This should now be fixed.

>> 2) rnews stops processing the newsbatch when it gets to an article that
>>    does have an empty Subject (/^Subject: $/). It logs the message
>>    "rnews: bad_article missing Subject", but then doesn't process the
>>    rest of the newsbatch.

> I'm a little confused how this is supposed to work.  rnews *does* have a
> way of rejecting one message and then continuing on to process the rest
> of the batch, and intentionally doesn't use it for this case.  I'm not
> sure why that is... it seems to me that for this, and also for
> unterminated message ID headers, only the one article should be rejected
> rather than pushing aside the whole batch.

I've made this change in CURRENT and STABLE.

> It really shouldn't be putting it in bad either; it looks like rnews -U
> behaves slightly differently than regular rnews, which is odd.

This has also now been fixed.

-- 
Russ Allbery (rra at stanford.edu)             <http://www.eyrie.org/~eagle/>

    Please send questions to the list rather than mailing me directly.
     <http://www.eyrie.org/~eagle/faqs/questions.html> explains why.