diff options
author | Monsta <[email protected]> | 2015-08-21 19:10:02 +0300 |
---|---|---|
committer | infirit <[email protected]> | 2015-08-31 21:57:23 +0200 |
commit | 55463d8851281552f5cc549617a5d6d2927b7c8f (patch) | |
tree | 321e64721e4d83f3b104159f876e61887009743d | |
parent | 8c08661057c66d312bb0b217cbd70f00e25c98c6 (diff) | |
download | mate-utils-55463d8851281552f5cc549617a5d6d2927b7c8f.tar.bz2 mate-utils-55463d8851281552f5cc549617a5d6d2927b7c8f.tar.xz |
logview: add some safety NULL-checks
from
https://git.gnome.org/browse/gnome-system-log/commit/?id=ad9153b8bcd8a2556d7a1f5fcca890680c4677de
-rw-r--r-- | logview/logview-loglist.c | 11 | ||||
-rw-r--r-- | logview/logview-manager.c | 4 |
2 files changed, 11 insertions, 4 deletions
diff --git a/logview/logview-loglist.c b/logview/logview-loglist.c index d7945c64..3bd0c8aa 100644 --- a/logview/logview-loglist.c +++ b/logview/logview-loglist.c @@ -135,7 +135,8 @@ logview_loglist_find_log (LogviewLoglist *list, LogviewLog *log) if (current == log) { retval = gtk_tree_iter_copy (&iter); } - g_object_unref (current); + if (current) + g_object_unref (current); } while (gtk_tree_model_iter_next (model, &iter) != FALSE && retval == NULL); return retval; @@ -488,10 +489,12 @@ logview_loglist_update_lines (LogviewLoglist *loglist, LogviewLog *log) g_assert (LOGVIEW_IS_LOGLIST (loglist)); g_assert (LOGVIEW_IS_LOG (log)); - days = logview_log_get_days_for_cached_lines (log); parent = logview_loglist_find_log (loglist, log); - update_days_and_lines_for_log (loglist, parent, days); - gtk_tree_iter_free (parent); + if (parent) { + days = logview_log_get_days_for_cached_lines (log); + update_days_and_lines_for_log (loglist, parent, days); + gtk_tree_iter_free (parent); + } } diff --git a/logview/logview-manager.c b/logview/logview-manager.c index 6d6bfb05..2448468e 100644 --- a/logview/logview-manager.c +++ b/logview/logview-manager.c @@ -393,6 +393,10 @@ logview_manager_close_active_log (LogviewManager *manager) g_assert (LOGVIEW_IS_MANAGER (manager)); active_log = manager->priv->active_log; + if (active_log == NULL) { + return; + } + log_uri = logview_log_get_uri (active_log); file = logview_log_get_gfile (active_log); |