Hi Torsten,
While building a private mail2news gateway I stumbled upon a limitation
regarding the number of header lines inews accepts. In lines 258 ff. of
inews.c, the following lines can be found:
| /* Too many headers? */
| if (++i > 5 * HEADER_DELTA)
| die("more than %d lines of header", i);
HEADER_DELTA is #defined to 20 near the beginning. Thus, any article
containing more than 100 header lines is rejected by inews.
In fact, it was 50 because i was incremented twice in the loop. And it was
not header lines but header fields.
But never mind, because it was not documented.
In my opinion, this value should be a configurable parameter. At the
very least, this arbitrarily chosen limit should be clearly documented
(it isn't at the moment).
You're right.
It is now configurable.
Here is a patch. It will be shipped with INN 2.5.2.
http://inn.eyrie.org/trac/changeset/8933/branches
A new option has been added to inews. When the "-m" flag is specified,
the corresponding value will be used to determine the maximum number of
header fields that are accepted for an article. The default was 50
and it is now configurable. Thanks to Torsten Jerzembeck for the bug report.
--
Julien ÉLIE
« Je ne crains rien, je n'espère rien, je suis libre. »
--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
