(In reply to comment #2)
> Shouldn't it add an "if (pending_return)" instead of removing the assignment?
> (Was it already assigned to earlier in the code?)
There is already a NULL assignment before that:
if (pending_return)
*pending_return = NULL;
CONNECTION_LOCK (connection);
if (!_dbus_connection_get_is_connected_unlocked (connection))
{
CONNECTION_UNLOCK (connection);
(In reply to comment #2)
> Shouldn't it add an "if (pending_return)" instead of removing the assignment?
> (Was it already assigned to earlier in the code?)
There is already a NULL assignment before that:
if (pending_return)
*pending_return = NULL;
CONNECTION_LOCK (connection);
if (!_dbus_ connection_ get_is_ connected_ unlocked (connection)) N_UNLOCK (connection);
{
CONNECTIO
* pending_ return = NULL;
return TRUE;
}