Comment 2 for bug 1427286

Revision history for this message
Lars Karlitski (larsu) wrote :

This is an issue in telephony-service: it calls g_variant_unref() on a variant that it doesn't own a reference to [1].

g_variant_new_strv() returns a floating reference which is consumed by the call to messaging_menu_message_add_action(). Another bug in that code is g_variant_unref() might be called with a NULL variant when the condition in line 212 is false. Both of those things will be fixed when removing the unref call.

[1] http://bazaar.launchpad.net/~phablet-team/telephony-service/trunk/view/head:/indicator/messagingmenu.cpp#L239