forked from Wavyzz/dolibarr
Revert code because it does not fix security hole completely. Also it
does work on origin but at a transition level. Sanitizing for command line data must not appears inside a function used for http data. I prefer fixing this at the source and also using a rule that clean all attacks completely instead of a rule that clean "most problem but not all".
This commit is contained in:
@@ -127,7 +127,7 @@ if ($what == 'mysql')
|
|||||||
if (! empty($dolibarr_main_db_port)) $param.=" -P ".$dolibarr_main_db_port;
|
if (! empty($dolibarr_main_db_port)) $param.=" -P ".$dolibarr_main_db_port;
|
||||||
if (! GETPOST("use_transaction")) $param.=" -l --single-transaction";
|
if (! GETPOST("use_transaction")) $param.=" -l --single-transaction";
|
||||||
if (GETPOST("disable_fk")) $param.=" -K";
|
if (GETPOST("disable_fk")) $param.=" -K";
|
||||||
if (GETPOST("sql_compat") && GETPOST("sql_compat") != 'NONE') $param.=" --compatible=".GETPOST("sql_compat","alpha");
|
if (GETPOST("sql_compat") && GETPOST("sql_compat") != 'NONE') $param.=" --compatible=".preg_replace('/[^a-zA-Z0-9]/','',GETPOST("sql_compat","alpha"));
|
||||||
if (GETPOST("drop_database")) $param.=" --add-drop-database";
|
if (GETPOST("drop_database")) $param.=" --add-drop-database";
|
||||||
if (GETPOST("sql_structure"))
|
if (GETPOST("sql_structure"))
|
||||||
{
|
{
|
||||||
|
|||||||
Reference in New Issue
Block a user