> - return node ? NODE_NAME (node) : "<unknown>"; > + return node ? (NODE_NAME (node) == NULL ? "<unknown>" : NODE_NAME (node)) : "<unknown>";
This might be simpler: return node && NODE_NAME (node) ? NODE_NAME (node) : "<unknown>"; -Evan C Wang wrote: > This patch will not fix the bug from the root, the root cause seems to > be the gstreamer is not thread safe. > However, with this patch, the pidgin will not crash if problem occurred > by not allowing NULL pointer pass to > g_log. > > The pidgin will not crash on Linux is because printf(NULL) will not > cause SEGV fault, but this happens on Solaris > > Chris > >
