Hi Jukka,
> +send_reply:
> + if (dbus_message_get_no_reply(message) == FALSE &&
> + reply == NULL) {
>
> reply = dbus_message_new_method_return(message);
> if (reply == NULL)
> return DBUS_HANDLER_RESULT_NEED_MEMORY;
> + }
>
> + if (reply != NULL) {
> dbus_connection_send(connection, reply, NULL);
>
> dbus_message_unref(reply);
> }
Why not just move to upper "if (reply == NULL)" one down?
cheers,
daniel
_______________________________________________
connman mailing list
connman@connman.net
http://lists.connman.net/listinfo/connman
