diff options
author | Marco Trevisan (Treviño) <mail@3v1n0.net> | 2022-04-21 18:29:55 +0200 |
---|---|---|
committer | Marco Trevisan (Treviño) <mail@3v1n0.net> | 2022-04-25 19:13:22 +0200 |
commit | c4d5ea8baff20bd6594faa310b38e8d2feb2d54d (patch) | |
tree | f5ed945849c974be09753300ac51e4934f176be2 /libnotify/notification.c | |
parent | d348a36a053bee9d45fdd3f90b0d0682e0ffdf25 (diff) | |
download | libnotify-c4d5ea8baff20bd6594faa310b38e8d2feb2d54d.tar.gz |
notification: Do not prepend SNAP prefix when is already present
We used to just add it again, to eventually figure out that the file
did not exists, but may not work properly in some scenarios.
Also always fill the path_filename when using a file-name only, as that
may be refer to something under the SNAP path, and in such case we want
to try check if that exists before giving up.
Diffstat (limited to 'libnotify/notification.c')
-rw-r--r-- | libnotify/notification.c | 21 |
1 files changed, 16 insertions, 5 deletions
diff --git a/libnotify/notification.c b/libnotify/notification.c index 55cc46f..be8bfac 100644 --- a/libnotify/notification.c +++ b/libnotify/notification.c @@ -443,16 +443,27 @@ try_prepend_path (const char *base_path, path_filename = g_strdup (base_path); } else { path_filename = realpath (base_path, NULL); + + if (path_filename == NULL) { + /* File path is not existing, but let's check + * if it's under the base path before giving up + */ + path_filename = g_strdup (base_path); + } } } - g_debug ("Trying to look at file '%s' in the '%s' prefix.", - base_path, - path); - - path_ret = g_build_filename (path, path_filename, NULL); + if (g_str_has_prefix (path_filename, path)) { + path_ret = g_strdup (path_filename); + } else { + g_debug ("Trying to look at file '%s' in the '%s' prefix.", + base_path, + path); + path_ret = g_build_filename (path, path_filename, NULL); + } if (!g_file_test (path_ret, G_FILE_TEST_EXISTS)) { + g_debug ("Nothing found at %s", path_ret); g_free (path_ret); path_ret = NULL; } |