Merge lp:~ted/indicator-messages/lp1216758 into lp:~indicator-applet-developers/indicator-messages/trunk.13.10

[Ted Gould]

Protect against invalid actions

Retrace isn't great, but it turns out there's only one call to that function in the codebase :-)

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:360
http://jenkins.qa.ubuntu.com/job/indicator-messages-ci/80/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/indicator-messages-saucy-amd64-ci/33
    SUCCESS: http://jenkins.qa.ubuntu.com/job/indicator-messages-saucy-armhf-ci/33
        deb: http://jenkins.qa.ubuntu.com/job/indicator-messages-saucy-armhf-ci/33/artifact/work/output/*zip*/output.zip

Click here to trigger a rebuild:
http://s-jenkins:8080/job/indicator-messages-ci/80/rebuild

review: Approve (continuous-integration)

[Lars Karlitski (larsu)]

Yep, indeed. Thanks.

review: Approve

=== modified file 'src/im-application-list.c'
--- src/im-application-list.c	2013-08-22 13:16:15 +0000
+++ src/im-application-list.c	2013-08-26 13:28:27 +0000
@@ -223,6 +223,8 @@
   GVariant * draw;
 
   state = g_action_group_get_action_state (G_ACTION_GROUP(app->source_actions), action_name);
+  if (state == NULL)
+    return FALSE;
 
   /* uxsb */
   draw = g_variant_get_child_value(state, 3);