Pass binary filename to sqlite3_open

c_src/sqlite3_nif.c

@@ -191,33 +191,45 @@ exqlite_open(ErlNifEnv* env, int argc, const ERL_NIF_TERM argv[])
 {
     assert(env);
 
+    unsigned char* filename;
     int flags;
     int rc             = 0;
     int size           = 0;
     connection_t* conn = NULL;
     sqlite3* db        = NULL;
     ErlNifMutex* mutex = NULL;
-    char filename[MAX_PATHNAME];
+    ErlNifBinary filename_binary;
     ERL_NIF_TERM result;
 
     if (argc != 2) {
         return enif_make_badarg(env);
     }
 
-    size = enif_get_string(env, argv[0], filename, MAX_PATHNAME, ERL_NIF_LATIN1);
-    if (size <= 0) {
+    if (!enif_inspect_binary(env, argv[0], &filename_binary)) {
         return make_error_tuple(env, "invalid_filename");
     }
 
+    filename = (unsigned char*)sqlite3_malloc(filename_binary.size + sizeof(unsigned char));
+    if (!filename) {
+        return make_error_tuple(env, "out_of_memory ");
+    }
+
+    memcpy(filename, filename_binary.data, filename_binary.size);
+    filename[filename_binary.size] = '\0';
+
     if (!enif_get_int(env, argv[1], &flags)) {
-        return make_error_tuple(env, "invalid flags");
+        sqlite3_free(filename);
+        return make_error_tuple(env, "invalid_flags");
     }
 
     rc = sqlite3_open_v2(filename, &db, flags, NULL);
     if (rc != SQLITE_OK) {
+        sqlite3_free(filename);
         return make_error_tuple(env, "database_open_failed");
     }
 
+    sqlite3_free(filename);
+
     mutex = enif_mutex_create("exqlite:connection");
     if (mutex == NULL) {
         sqlite3_close_v2(db);

lib/exqlite/sqlite3.ex

@@ -35,7 +35,7 @@ defmodule Exqlite.Sqlite3 do
   @spec open(String.t(), [open_opt()]) :: {:ok, db()} | {:error, reason()}
   def open(path, opts \\ []) do
     mode = Keyword.get(opts, :mode, :readwrite)
-    Sqlite3NIF.open(String.to_charlist(path), flags_from_mode(mode))
+    Sqlite3NIF.open(path, flags_from_mode(mode))
   end
 
   defp flags_from_mode(:readwrite),

test/exqlite/connection_test.exs

@@ -46,6 +46,16 @@ defmodule Exqlite.ConnectionTest do
       assert state.db
     end
 
+    test "connects to a file with an accented character" do
+      path = Temp.path!(prefix: "databasé")
+      {:ok, state} = Connection.connect(database: path)
+
+      assert state.path == path
+      assert state.db
+
+      File.rm(path)
+    end
+
     test "fails to write a file from URL with mode=ro" do
       path = Temp.path!()