From 878d97068cb6c8ae044332ac3c8218fcad635b42 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Bert=20M=C3=BCnnich?= Date: Thu, 20 Oct 2016 10:21:55 +0200 Subject: Only open regular files; fixes issue #252 --- Makefile | 2 +- image.c | 5 ++++- thumbs.c | 16 ++++++++++------ 3 files changed, 15 insertions(+), 8 deletions(-) diff --git a/Makefile b/Makefile index 2339272..c9b140c 100644 --- a/Makefile +++ b/Makefile @@ -1,4 +1,4 @@ -VERSION := git-20160928 +VERSION := git-20161020 PREFIX := /usr/local MANPREFIX := $(PREFIX)/share/man diff --git a/image.c b/image.c index ebd522d..801bbe5 100644 --- a/image.c +++ b/image.c @@ -19,6 +19,7 @@ #include #include #include +#include #include #include @@ -296,8 +297,10 @@ bool img_load_gif(img_t *img, const fileinfo_t *file) bool img_load(img_t *img, const fileinfo_t *file) { const char *fmt; + struct stat st; - if (access(file->path, R_OK) < 0 || + if (access(file->path, R_OK) == -1 || + stat(file->path, &st) == -1 || !S_ISREG(st.st_mode) || (img->im = imlib_load_image(file->path)) == NULL) { if (file->flags & FF_WARN) diff --git a/thumbs.c b/thumbs.c index 3988bd2..43bb50c 100644 --- a/thumbs.c +++ b/thumbs.c @@ -239,6 +239,7 @@ bool tns_load(tns_t *tns, int n, bool force, bool cache_only) float zw, zh; thumb_t *t; fileinfo_t *file; + struct stat st; Imlib_Image im = NULL; if (n < 0 || n >= *tns->cnt) @@ -330,12 +331,15 @@ bool tns_load(tns_t *tns, int n, bool force, bool cache_only) } } - if (im == NULL && (access(file->path, R_OK) < 0 || - (im = imlib_load_image(file->path)) == NULL)) - { - if (file->flags & FF_WARN) - error(0, 0, "%s: Error opening image", file->name); - return false; + if (im == NULL) { + if (access(file->path, R_OK) == -1 || + stat(file->path, &st) == -1 || !S_ISREG(st.st_mode) || + (im = imlib_load_image(file->path)) == NULL) + { + if (file->flags & FF_WARN) + error(0, 0, "%s: Error opening image", file->name); + return false; + } } imlib_context_set_image(im); -- cgit v1.2.3