validate string before parsing

elixir-lsp · Nov 17, 2023 · 3ad9449 · 3ad9449
1 parent 22de3f6
commit 3ad9449
Showing 1 changed file with 11 additions and 6 deletions.
diff --git a/lib/elixir_sense/core/parser.ex b/lib/elixir_sense/core/parser.ex
@@ -16,12 +16,17 @@ defmodule ElixirSense.Core.Parser do
   def parse_file(file, try_to_fix_parse_error, try_to_fix_line_not_found, cursor_line_number) do
     case File.read(file) do
       {:ok, source} ->
-        parse_string(
-          source,
-          try_to_fix_parse_error,
-          try_to_fix_line_not_found,
-          cursor_line_number
-        )
+        if String.valid?(source) do
+          parse_string(
+            source,
+            try_to_fix_parse_error,
+            try_to_fix_line_not_found,
+            cursor_line_number
+          )
+        else
+          Logger.warning("Invalid encoding in #{file}")
+          create_metadata("", {:error, :invalid_encoding})
+        end
 
       error ->
         Logger.warning("Unable to read file #{file}: #{inspect(error)}")