[gtk+] quartz: Null check title before setting it



commit e28b10d56f8f22dc5caf52b649e370665f93631b
Author: Alan McGovern <alan xamarin com>
Date:   Wed Mar 6 11:06:09 2013 +0000

    quartz: Null check title before setting it
    
    This avoids a crash in objective-c.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=695278

 gdk/quartz/gdkwindow-quartz.c |    9 ++++++---
 1 files changed, 6 insertions(+), 3 deletions(-)
---
diff --git a/gdk/quartz/gdkwindow-quartz.c b/gdk/quartz/gdkwindow-quartz.c
index a096064..5550fa8 100644
--- a/gdk/quartz/gdkwindow-quartz.c
+++ b/gdk/quartz/gdkwindow-quartz.c
@@ -2642,9 +2642,11 @@ gdk_quartz_window_set_decorations (GdkWindow       *window,
            * NSTitledWindowMask. This behaved extremely oddly when
            * conditionalized upon that and since it has no side effects (i.e.
            * if NSTitledWindowMask is not requested, the title will not be
-           * displayed) just do it unconditionally.
+           * displayed) just do it unconditionally. We also must null check
+           * 'title' before setting it to avoid crashing.
            */
-          [impl->toplevel setTitle:title];
+          if (title)
+            [impl->toplevel setTitle:title];
         }
       else
         {
@@ -2664,7 +2666,8 @@ gdk_quartz_window_set_decorations (GdkWindow       *window,
                                                                    screen:screen];
           [impl->toplevel setHasShadow: window_type_hint_to_shadow (impl->type_hint)];
           [impl->toplevel setLevel: window_type_hint_to_level (impl->type_hint)];
-          [impl->toplevel setTitle:title];
+          if (title)
+            [impl->toplevel setTitle:title];
           [impl->toplevel setBackgroundColor:bg];
           [impl->toplevel setHidesOnDeactivate: window_type_hint_to_hides_on_deactivate (impl->type_hint)];
           [impl->toplevel setContentView:old_view];


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]