Menu: Typecheck before dereferencing to get ->priv
authorDaniel Boles <dboles.src@gmail.com>
Tue, 17 Apr 2018 18:51:53 +0000 (19:51 +0100)
committerDaniel Boles <dboles.src@gmail.com>
Tue, 17 Apr 2018 19:40:21 +0000 (20:40 +0100)
gtk/gtkmenu.c

index 3e58d7e15275ed6f8a97f66c2b099cdc7c5051d8..686c09d69d3eb116f2cfb5e84954c985a37fdb98 100644 (file)
@@ -2212,12 +2212,14 @@ gtk_menu_popdown (GtkMenu *menu)
 GtkWidget*
 gtk_menu_get_active (GtkMenu *menu)
 {
-  GtkMenuPrivate *priv = menu->priv;
+  GtkMenuPrivate *priv;
   GtkWidget *child;
   GList *children;
 
   g_return_val_if_fail (GTK_IS_MENU (menu), NULL);
 
+  priv = menu->priv;
+
   if (!priv->old_active_menu_item)
     {
       child = NULL;
@@ -2254,12 +2256,14 @@ void
 gtk_menu_set_active (GtkMenu *menu,
                      guint    index)
 {
-  GtkMenuPrivate *priv = menu->priv;
+  GtkMenuPrivate *priv;
   GtkWidget *child;
   GList *tmp_list;
 
   g_return_if_fail (GTK_IS_MENU (menu));
 
+  priv = menu->priv;
+
   tmp_list = g_list_nth (GTK_MENU_SHELL (menu)->priv->children, index);
   if (tmp_list)
     {
@@ -2291,9 +2295,12 @@ void
 gtk_menu_set_accel_group (GtkMenu       *menu,
                           GtkAccelGroup *accel_group)
 {
-  GtkMenuPrivate *priv = menu->priv;
+  GtkMenuPrivate *priv;
+
   g_return_if_fail (GTK_IS_MENU (menu));
 
+  priv = menu->priv;
+
   if (priv->accel_group != accel_group)
     {
       if (priv->accel_group)
@@ -2371,9 +2378,12 @@ void
 gtk_menu_set_accel_path (GtkMenu     *menu,
                          const gchar *accel_path)
 {
-  GtkMenuPrivate *priv = menu->priv;
+  GtkMenuPrivate *priv;
+
   g_return_if_fail (GTK_IS_MENU (menu));
 
+  priv = menu->priv;
+
   if (accel_path)
     g_return_if_fail (accel_path[0] == '<' && strchr (accel_path, '/')); /* simplistic check */
 
@@ -4626,10 +4636,12 @@ void
 gtk_menu_set_monitor (GtkMenu *menu,
                       gint     monitor_num)
 {
-  GtkMenuPrivate *priv = menu->priv;
+  GtkMenuPrivate *priv;
 
   g_return_if_fail (GTK_IS_MENU (menu));
 
+  priv = menu->priv;
+
   if (priv->monitor_num != monitor_num)
     {
       priv->monitor_num = monitor_num;
@@ -4752,13 +4764,14 @@ void
 gtk_menu_set_reserve_toggle_size (GtkMenu  *menu,
                                   gboolean  reserve_toggle_size)
 {
-  GtkMenuPrivate *priv = menu->priv;
+  GtkMenuPrivate *priv;
   gboolean no_toggle_size;
 
   g_return_if_fail (GTK_IS_MENU (menu));
 
-  no_toggle_size = !reserve_toggle_size;
+  priv = menu->priv;
 
+  no_toggle_size = !reserve_toggle_size;
   if (priv->no_toggle_size != no_toggle_size)
     {
       priv->no_toggle_size = no_toggle_size;