fix: Time-Based Blind SQL Injection via options[stato] Parameter

Author: Pek5892

modules/contratti/ajax/select.php

@@ -53,7 +53,10 @@
 
         if (empty($elements)) {
             $where[] = '`an_anagrafiche`.`idanagrafica`='.prepare($superselect['idanagrafica']);
-            $stato = !empty($superselect['stato']) ? $superselect['stato'] : 'is_pianificabile';
+            $allowed_stati = ['is_pianificabile', 'is_completato', 'is_fatturabile'];
+            $stato = !empty($superselect['stato']) && in_array($superselect['stato'], $allowed_stati)
+                ? $superselect['stato']
+                : 'is_pianificabile';
             $where[] = '`idstato` IN (SELECT `id` FROM `co_staticontratti` WHERE '.$stato.' = 1)';
         }
 

modules/ordini/ajax/select.php

@@ -48,7 +48,10 @@
             if (empty($elements)) {
                 $where[] = '`an_anagrafiche`.`idanagrafica`='.prepare($superselect['idanagrafica']);
 
-                $stato = !empty($superselect['stato']) ? $superselect['stato'] : 'is_fatturabile';
+                $allowed_stati = ['is_fatturabile', 'is_evadibile', 'is_completato'];
+                $stato = !empty($superselect['stato']) && in_array($superselect['stato'], $allowed_stati)
+                    ? $superselect['stato']
+                    : 'is_fatturabile';
                 $where[] = '`or_statiordine`.'.$stato.' = 1';
             }
         }

modules/preventivi/ajax/select.php

@@ -56,7 +56,10 @@
                 $where[] = '`an_anagrafiche`.`idanagrafica`='.prepare($superselect['idanagrafica']);
                 $where[] = '`co_preventivi`.`default_revision`=1';
 
-                $stato = !empty($superselect['stato']) ? $superselect['stato'] : 'is_pianificabile';
+                $allowed_stati = ['is_pianificabile', 'is_completato', 'is_fatturabile', 'is_concluso'];
+                $stato = !empty($superselect['stato']) && in_array($superselect['stato'], $allowed_stati)
+                    ? $superselect['stato']
+                    : 'is_pianificabile';
                 $where[] = '('.$stato.' = 1)';
             }