fix psalm-internal risky casts

kkmuffme · Sep 19, 2022 · da68e9d · da68e9d
1 parent 7cdad99
commit da68e9d
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 3 deletions.
diff --git a/src/Psalm/Internal/Cli/LanguageServer.php b/src/Psalm/Internal/Cli/LanguageServer.php
@@ -32,6 +32,7 @@
 use function in_array;
 use function ini_set;
 use function is_array;
+use function is_numeric;
 use function is_string;
 use function preg_replace;
 use function realpath;
@@ -298,7 +299,7 @@ function () use ($current_dir, $options, $vendor_dir): ?\Composer\Autoload\Class
             $find_unused_code = 'auto';
         }
 
-        if (isset($options['disable-on-change'])) {
+        if (isset($options['disable-on-change']) && is_numeric($options['disable-on-change'])) {
             $project_analyzer->onchange_line_limit = (int) $options['disable-on-change'];
         }
 

diff --git a/src/Psalm/Internal/Cli/Psalter.php b/src/Psalm/Internal/Cli/Psalter.php
@@ -45,6 +45,7 @@
 use function ini_set;
 use function is_array;
 use function is_dir;
+use function is_numeric;
 use function is_string;
 use function microtime;
 use function pathinfo;
@@ -230,7 +231,7 @@ function () use ($current_dir, $options, $vendor_dir): ?\Composer\Autoload\Class
             chdir($current_dir);
         }
 
-        $threads = isset($options['threads']) ? (int)$options['threads'] : 1;
+        $threads = isset($options['threads']) && is_numeric($options['threads']) ? (int)$options['threads'] : 1;
 
         if (isset($options['no-cache'])) {
             $providers = new Providers(

diff --git a/src/Psalm/Internal/Cli/Refactor.php b/src/Psalm/Internal/Cli/Refactor.php
@@ -35,6 +35,7 @@
 use function in_array;
 use function ini_set;
 use function is_array;
+use function is_numeric;
 use function is_string;
 use function max;
 use function microtime;
@@ -284,7 +285,7 @@ function () use ($current_dir, $options, $vendor_dir): ?\Composer\Autoload\Class
             chdir($current_dir);
         }
 
-        $threads = isset($options['threads'])
+        $threads = isset($options['threads']) && is_numeric($options['threads'])
             ? (int)$options['threads']
             : max(1, ProjectAnalyzer::getCpuCount() - 2);