Quoting Christian Biere <[EMAIL PROTECTED]> from ml.softs.gtk-gnutella.devel:
:I don't see what lookup_is_alive() is good for. It is not valid C if the
:pointer actually references a freed object as per C standard section 6.2.4. If
:you want to use weak references, look at core/nodes.c and "node_id". You have
:to use numeric IDs, not pointers to track objects beyond their life-cycle.

This code is perfectly valid as the pointer is used only as an integer here,
as a key in the hash table, and is never de-referenced unless it is valid
(presence in the table means the pointer is valid).

Anyway, this is not the problem here.  The problem is the missing else block,
most probably.  I need to review the code to make sure this was my original
intent.

One thing is sure: as written, this part of the code is broken and can
lead to the assertion failure.

Raphael

-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
gtk-gnutella-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/gtk-gnutella-devel

Reply via email to