<!doctype html public "-//W3C//DTD W3 HTML//EN">
<html><head><style type="text/css"><!--
blockquote, dl, ul, ol, li { padding-top: 0 ; padding-bottom: 0 }
--></style><title>Re: [Live-devel] segfault in
DelayQueue::removeEntry()</title></head><body>
<blockquote type="cite" cite>should the sanity check, at the beginning
of the function, be patched in the following way?<br>
<br>
- if (entry == <a
href=
"http://www.live555.com/liveMedia/doxygen/html/UsageEnvironment_8hh.html#070d2ce7b6bb7e5c05602aa8c308d0c4"><span
></span>NULL</a> || entry-><a
href=
"http://www.live555.com/liveMedia/doxygen/html/classDelayQueueEntry.html#3a2ccdcdcac760623833136b5ced2410"><span
></span>fNext</a> == <a
href=
"http://www.live555.com/liveMedia/doxygen/html/UsageEnvironment_8hh.html#070d2ce7b6bb7e5c05602aa8c308d0c4"><span
></span>NULL</a>) return;<br>
</blockquote>
<blockquote type="cite" cite>+ if (entry == <a
href=
"http://www.live555.com/liveMedia/doxygen/html/UsageEnvironment_8hh.html#070d2ce7b6bb7e5c05602aa8c308d0c4"><span
></span>NULL</a> || entry-><a
href=
"http://www.live555.com/liveMedia/doxygen/html/classDelayQueueEntry.html#3a2ccdcdcac760623833136b5ced2410"><span
></span>fNext</a> == <a
href=
"http://www.live555.com/liveMedia/doxygen/html/UsageEnvironment_8hh.html#070d2ce7b6bb7e5c05602aa8c308d0c4"><span
></span>NULL</a> || entry-><a
href=
"http://www.live555.com/liveMedia/doxygen/html/classDelayQueueEntry.html#3a2ccdcdcac760623833136b5ced2410"><span
></span>fPrev</a> == <a
href=
"http://www.live555.com/liveMedia/doxygen/html/UsageEnvironment_8hh.html#070d2ce7b6bb7e5c05602aa8c308d0c4"><span
></span>NULL</a>) return;</blockquote>
<div><br></div>
<div>In principle, this shouldn't be necessary, as it should never be
the case that "entry->fNext" is non-NULL, but
"entry->fPrev" is NULL.</div>
<div><br></div>
<div>So, I don't understand how your error could have occurred.
I hope you're not doing something dumb like trying to access a queue
from multiple threads?</div>
<div><br></div>
<div>Puzzling...</div>
<x-sigsep><pre>--
</pre></x-sigsep>
<div><br>
Ross Finlayson<br>
Live Networks, Inc.<br>
http://www.live555.com/</div>
</body>
</html>