summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAlexander von Gluck IV <[email protected]>2013-09-24 18:46:53 -0500
committerAlexander von Gluck IV <[email protected]>2013-09-24 18:46:53 -0500
commitb4b58f54b640165ef82dcabeb8d70fb4ade475f0 (patch)
tree5f6ebdea491315f792036d0ec7a83dea7f1087cc
parentdfe505522c630863438c3649e2a028f14e1138fa (diff)
downloadpluma-b4b58f54b640165ef82dcabeb8d70fb4ade475f0.tar.bz2
pluma-b4b58f54b640165ef82dcabeb8d70fb4ade475f0.tar.xz
filemanager: Add assertion checks to prevent dereferencing ptrs
* clang static analysis shows gaps where these calls could rereferencce pointers. Perform an assertion check.
-rw-r--r--plugins/filebrowser/pluma-file-browser-store.c2
-rw-r--r--plugins/filebrowser/pluma-file-browser-widget.c2
2 files changed, 4 insertions, 0 deletions
diff --git a/plugins/filebrowser/pluma-file-browser-store.c b/plugins/filebrowser/pluma-file-browser-store.c
index 6f611105..ba45feff 100644
--- a/plugins/filebrowser/pluma-file-browser-store.c
+++ b/plugins/filebrowser/pluma-file-browser-store.c
@@ -2480,6 +2480,8 @@ set_virtual_root_from_node (PlumaFileBrowserStore * model,
GSList *copy;
GtkTreePath *empty = NULL;
+ g_assert (node != NULL);
+
prev = node;
next = prev->parent;
diff --git a/plugins/filebrowser/pluma-file-browser-widget.c b/plugins/filebrowser/pluma-file-browser-widget.c
index f2c54fe9..22c3c3b0 100644
--- a/plugins/filebrowser/pluma-file-browser-widget.c
+++ b/plugins/filebrowser/pluma-file-browser-widget.c
@@ -1640,6 +1640,8 @@ jump_to_location (PlumaFileBrowserWidget * obj, GList * item,
obj->priv->changing_location = TRUE;
+ g_assert (obj->priv->current_location != NULL);
+
loc = (Location *) (obj->priv->current_location->data);
/* Set the new root + virtual root */