summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGeyslan G. Bem <geyslan@gmail.com>2021-04-27 11:42:55 -0300
committerGeyslan G. Bem <geyslan@gmail.com>2021-04-27 11:58:31 -0300
commit1635998ac3482abf6153f857e8a16c8b8cf85158 (patch)
tree780dba8064aefe3ef4ae7f6e12708c43117174a0
parentb23529d940c17bce03a8b8c2c29a11f07800e962 (diff)
downloadgtk+-1635998ac3482abf6153f857e8a16c8b8cf85158.tar.gz
gtkplacessidebar.c: add bookmark null checking
This fixes nautilus crash and perhaps other callers issues. Nautilus (and sometimes glib) crashes with malformed URI inside of the bookmarks file .config/gtk-3.0/bookmarks when it has no LABEL. This is result from the closed glib MR #2065 analysis and agreement. https://gitlab.gnome.org/GNOME/glib/-/merge_requests/2065#note_1091979 Signed-off-by: Geyslan G. Bem <geyslan@gmail.com>
-rw-r--r--gtk/gtkplacessidebar.c4
1 files changed, 4 insertions, 0 deletions
diff --git a/gtk/gtkplacessidebar.c b/gtk/gtkplacessidebar.c
index 43f03069d8..c57031be07 100644
--- a/gtk/gtkplacessidebar.c
+++ b/gtk/gtkplacessidebar.c
@@ -795,6 +795,10 @@ on_bookmark_query_info_complete (GObject *source,
{
/* Don't add non-UTF-8 bookmarks */
bookmark_name = g_file_get_basename (root);
+ if (bookmark_name == NULL)
+ {
+ goto out;
+ }
if (!g_utf8_validate (bookmark_name, -1, NULL))
{
g_free (bookmark_name);