diff options
| author | Matthieu Herrb <matthieu@herrb.eu> | 2015-11-07 18:51:17 +0100 |
|---|---|---|
| committer | Matthieu Herrb <matthieu@herrb.eu> | 2015-11-07 18:51:17 +0100 |
| commit | 1be2bc04b11834db2c1dd8587fd725ef9106016d (patch) | |
| tree | d42b5f079614736991338366572a62b4e03f3b23 /src/menus.c | |
| parent | 7755c564b9b7f306fd8d47f35f37ae06e6ea0acd (diff) | |
| parent | c5abcb619eb0b01f074fd9acb9128f19af774834 (diff) | |
Merge remote-tracking branch 'origin/master' into obsd
Diffstat (limited to 'src/menus.c')
| -rw-r--r-- | src/menus.c | 32 |
1 files changed, 18 insertions, 14 deletions
diff --git a/src/menus.c b/src/menus.c index a390444..8aa99c7 100644 --- a/src/menus.c +++ b/src/menus.c @@ -2020,7 +2020,7 @@ ExecuteFunction(int func, const char *action, Window w, TwmWindow *tmp_win, "%s: unable to open cut file \"%s\"\n", ProgramName, tmp); } - if (ptr != tmp) free (ptr); + free (ptr); } } else { XFree(ptr); @@ -2171,21 +2171,25 @@ ExecuteFunction(int func, const char *action, Window w, TwmWindow *tmp_win, case F_FILE: ptr = ExpandFilename(action); - fd = open(ptr, O_RDONLY); - if (fd >= 0) - { - count = read(fd, buff, MAX_FILE_SIZE - 1); - if (count > 0) - XStoreBytes(dpy, buff, count); + if (ptr) { + fd = open(ptr, O_RDONLY); + if (fd >= 0) + { + count = read(fd, buff, MAX_FILE_SIZE - 1); + if (count > 0) + XStoreBytes(dpy, buff, count); - close(fd); - } - else - { - fprintf (stderr, "%s: unable to open file \"%s\"\n", - ProgramName, ptr); + close(fd); + } + else + { + fprintf (stderr, "%s: unable to open file \"%s\"\n", + ProgramName, ptr); + } + free(ptr); + } else { + fprintf (stderr, "%s: error expanding filename\n", ProgramName); } - if (ptr != action) free(ptr); break; case F_REFRESH: |