diff options
| author | Alexander von Gluck IV <[email protected]> | 2013-09-24 18:46:53 -0500 |
|---|---|---|
| committer | infirit <[email protected]> | 2013-12-17 17:52:35 +0100 |
| commit | 8d897b8ecac49212c761752ea50d1f890f962441 (patch) | |
| tree | 8e695cf99ab22098785a44c50bcc4b24f153d224 | |
| parent | 51ce5064d95f2184ce364f6294d1fc1510963779 (diff) | |
| download | pluma-8d897b8ecac49212c761752ea50d1f890f962441.tar.bz2 pluma-8d897b8ecac49212c761752ea50d1f890f962441.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 */ |