On Wed, Aug 01, 2007 at 09:40:31PM +0200, Christof Meerwald wrote:
> On Mon, 30 Jul 2007 22:14:24 +0200, Christof Meerwald wrote:
> > just had a quick look through the recent SVN changes. I don't think the
> > changes in mio_impl.h are correct.
> >
> > With the current version you will probably reference freed memory in
> > _mio_run (which I think was the reason for the "deferred closing fd" in
> > _mio_run, but I am not sure if the "skip dead slots" in _mio_run was
> > needed).
> I have now confirmed with valgrind that 2.1.12 does reference memory that
> has been freed (as expected).

You might, of course, also want to revert the mio_impl.h changes from change
311 (as you have at least reverted 313 now) - but then again, you could also
wait another week before fixing it...


Christof

-- 

http://cmeerw.org                              sip:cmeerw at cmeerw.org
mailto:cmeerw at cmeerw.org                   xmpp:cmeerw at cmeerw.org
_______________________________________________
Jabberd2 mailing list
Jabberd2@lists.xiaoka.com
http://lists.xiaoka.com/listinfo.cgi/jabberd2-xiaoka.com

Reply via email to