[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [leafnode-list] fix SEGV in nntpd.c



Jonathan Larmour <jlarmour@xxxxxxxxxx> writes:

> Jonathan Larmour wrote:
>> 
>> This is again against 1.9.20.rel. If fetchnews runs, it causes any existing
>> running clients to SEGV because the active file is stamped, causing it 
>
> Actually this patch evidently wasn't quite right, so here it is fixed -
> xovergroup is only meant to be set to group when the xover data in memory
> matches the group. So just NULLify it instead, so that getxover() will be
> run again.
>
> It does seem a bit odd to reread the entire xover data again though.

Well, running concurrent processes will always increase the overhead,
and if we're reading from a current xover file, we're not wasting too
much time, but presumedly, we're reading from the buffer cache, so it
should not be too expensive.

> Perhaps someone should consider a different approach? Anyway, this patch
> should do the trick now:
>
> 2002-04-01  Jonathan Larmour  <jlarmour@xxxxxxxxxx>
>
>         * nntpd.c (rereadactive): Preserve group between rereadings.

Applied, thanks.

-- 
Matthias Andree

-- 
leafnode-list@xxxxxxxxxxxxxxxxxxxxxxxxxxxx -- mailing list for leafnode
To unsubscribe, send mail with "unsubscribe" in the subject to the list