compose: Ensure we mmap only regular files

Author: wismill

src/compose/table.c

@@ -176,31 +176,31 @@ xkb_compose_table_new_from_locale(struct xkb_context *ctx,
         return NULL;
 
     path = get_xcomposefile_path(ctx);
-    if (path) {
+    if (path && is_regular_file(path)) {
         file = fopen(path, "rb");
         if (file)
             goto found_path;
     }
     free(path);
 
     path = get_xdg_xcompose_file_path(ctx);
-    if (path) {
+    if (path && is_regular_file(path)) {
         file = fopen(path, "rb");
         if (file)
             goto found_path;
     }
     free(path);
 
     path = get_home_xcompose_file_path(ctx);
-    if (path) {
+    if (path && is_regular_file(path)) {
         file = fopen(path, "rb");
         if (file)
             goto found_path;
     }
     free(path);
 
     path = get_locale_compose_file_path(ctx, table->locale);
-    if (path) {
+    if (path && is_regular_file(path)) {
         file = fopen(path, "rb");
         if (file)
             goto found_path;

src/utils.c

@@ -23,14 +23,14 @@
 
 #include "config.h"
 
+#include <sys/stat.h>
 #include "utils.h"
 
 #ifdef HAVE_MMAP
 
 #include <fcntl.h>
 #include <unistd.h>
 #include <sys/mman.h>
-#include <sys/stat.h>
 #include <sys/types.h>
 
 bool
@@ -111,6 +111,16 @@ unmap_file(char *str, size_t size)
 
 #endif
 
+bool
+is_regular_file(const char *path)
+{
+    struct stat stat_buf;
+    int err = stat(path, &stat_buf);
+    if (err != 0) return false;
+
+    return S_ISREG(stat_buf.st_mode);
+}
+
 // ASCII lower-case map.
 static const unsigned char lower_map[] = {
     0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 19, 20,

src/utils.h

@@ -47,6 +47,9 @@
 # ifndef S_ISDIR
 #  define S_ISDIR(m) (((m) & S_IFMT) == S_IFDIR)
 # endif
+# ifndef S_ISREG
+#  define S_ISREG(m) (((m) & S_IFMT) == S_IFREG)
+# endif
 typedef SSIZE_T ssize_t;
 #endif
 
@@ -263,6 +266,9 @@ check_eaccess(const char *path, int mode)
     return true;
 }
 
+bool
+is_regular_file(const char *path);
+
 #if defined(HAVE_SECURE_GETENV)
 # define secure_getenv secure_getenv
 #elif defined(HAVE___SECURE_GETENV)

test/compose.c

@@ -23,6 +23,7 @@
 
 #include "config.h"
 #include <time.h>
+#include <errno.h>
 
 #include "xkbcommon/xkbcommon-compose.h"
 
@@ -499,6 +500,18 @@ test_XCOMPOSEFILE(struct xkb_context *ctx)
     struct xkb_compose_table *table;
     char *path;
 
+    /* Error: directory */
+    path = test_get_path("locale/en_US.UTF-8");
+    setenv("XCOMPOSEFILE", path, 1);
+    free(path);
+
+    table = xkb_compose_table_new_from_locale(ctx, "blabla",
+                                              XKB_COMPOSE_COMPILE_NO_FLAGS);
+    assert_printf(errno != ENODEV && errno != EISDIR,
+                  "Should not be an error from `map_file`\n");
+    assert(!table);
+
+    /* OK: regular file */
     path = test_get_path("locale/en_US.UTF-8/Compose");
     setenv("XCOMPOSEFILE", path, 1);
     free(path);