Hi Darren,

>          g_string_append(tooltip_markup, "\n");
>      }
>  
> +  }
> +  if ( iface_name != NULL ) {
>      g_string_append_printf(tooltip_markup, _("<b>%-20s</b>: %s"), _("Network 
> Connection"), iface_name );
>    }
>    else {

Was the execution flow changed? I would expect a diff as follows for the
null check.

+  if ( iface_name != NULL ) {
     g_string_append_printf(tooltip_markup, _("<b>%-20s</b>: %s"), _("Network 
Connection"), iface_name );
+  }
   }


-Evan

Darren Kenny wrote:
> Simple fix, to check if iface_name is not NULL before using it.
>
>
>   

Reply via email to