diff options
| author | Alexander von Gluck IV <[email protected]> | 2013-09-24 18:46:53 -0500 |
|---|---|---|
| committer | Alexander von Gluck IV <[email protected]> | 2013-09-24 18:46:53 -0500 |
| commit | b4b58f54b640165ef82dcabeb8d70fb4ade475f0 (patch) | |
| tree | 5f6ebdea491315f792036d0ec7a83dea7f1087cc | |
| parent | dfe505522c630863438c3649e2a028f14e1138fa (diff) | |
| download | pluma-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.c | 2 | ||||
| -rw-r--r-- | plugins/filebrowser/pluma-file-browser-widget.c | 2 |
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 */ |