From 638cb105df33546d1e530de16c5b51ae40b03040 Mon Sep 17 00:00:00 2001 From: Laurent Destailleur Date: Fri, 18 Feb 2011 10:45:38 +0000 Subject: [PATCH] Fix: HTML tags must be escaped --- htdocs/adherents/liste.php | 5 ++-- htdocs/adherents/type.php | 5 ++-- htdocs/admin/tools/listevents.php | 2 +- htdocs/admin/tools/listsessions.php | 33 ------------------------- htdocs/comm/clients.php | 4 +-- htdocs/comm/contact.php | 2 +- htdocs/comm/mailing/cibles.php | 5 ++-- htdocs/comm/mailing/fiche.php | 1 - htdocs/comm/mailing/liste.php | 2 +- htdocs/comm/propal.php | 2 +- htdocs/comm/prospect/prospects.php | 2 +- htdocs/commande/liste.php | 2 +- htdocs/compta/bank/account.php | 2 +- htdocs/compta/bank/search.php | 2 +- htdocs/compta/clients.php | 2 +- htdocs/compta/facture.php | 2 +- htdocs/compta/facture/impayees.php | 2 +- htdocs/compta/paiement/cheque/liste.php | 2 +- htdocs/compta/paiement/liste.php | 2 +- htdocs/compta/param/comptes/liste.php | 2 +- htdocs/compta/prelevement/bons.php | 2 +- htdocs/compta/prelevement/demandes.php | 2 +- htdocs/compta/prelevement/liste.php | 2 +- htdocs/compta/propal.php | 2 +- htdocs/compta/sociales/index.php | 2 +- htdocs/compta/ventilation/lignes.php | 2 +- htdocs/contact/index.php | 5 ++-- htdocs/contrat/liste.php | 2 +- htdocs/contrat/services.php | 2 +- htdocs/fichinter/index.php | 2 +- htdocs/fourn/commande/liste.php | 2 +- htdocs/fourn/facture/impayees.php | 2 +- htdocs/fourn/facture/index.php | 2 +- htdocs/fourn/liste.php | 2 +- htdocs/fourn/product/liste.php | 5 ++-- htdocs/product/liste.php | 4 +-- htdocs/product/stock/mouvement.php | 8 +++--- htdocs/projet/liste.php | 2 +- htdocs/societe/lien.php | 2 +- htdocs/societe/societe.php | 4 +-- 40 files changed, 55 insertions(+), 82 deletions(-) diff --git a/htdocs/adherents/liste.php b/htdocs/adherents/liste.php index 5da5e4f4db2..4b221c6d40a 100644 --- a/htdocs/adherents/liste.php +++ b/htdocs/adherents/liste.php @@ -242,8 +242,9 @@ if ($resql) print ' '; print ''; - print ''; - print '  '; + print ''; + print '  '; + print ''; print ''; print "\n"; diff --git a/htdocs/adherents/type.php b/htdocs/adherents/type.php index 98af9040d79..3f950dad3d6 100644 --- a/htdocs/adherents/type.php +++ b/htdocs/adherents/type.php @@ -460,8 +460,9 @@ if ($rowid > 0) print ' '; print ''; - print ''; - print '  '; + print ''; + print '  '; + print ''; print ''; print "\n"; diff --git a/htdocs/admin/tools/listevents.php b/htdocs/admin/tools/listevents.php index 3fb1e240e66..3355d8a4455 100644 --- a/htdocs/admin/tools/listevents.php +++ b/htdocs/admin/tools/listevents.php @@ -145,7 +145,7 @@ if ($result) print ''; print ''; - print ''; + print ''; print ''; print "\n"; diff --git a/htdocs/admin/tools/listsessions.php b/htdocs/admin/tools/listsessions.php index 56287857188..2d48dd8c2d0 100644 --- a/htdocs/admin/tools/listsessions.php +++ b/htdocs/admin/tools/listsessions.php @@ -134,39 +134,6 @@ if ($savehandler == 'files') print_liste_field_titre('','',''); print "\n"; - - // Lignes des champs de filtre - /* - print '
'; - - print ''; - - print ' '; - - print ''; - print ''; - print ''; - - print ''; - print ''; - print ''; - - print ''; - print ''; - print ''; - - print ''; - print ''; - print ''; - - print ''; - print ''; - print ''; - - print "\n"; - print '
'; - */ - $var=True; foreach ($listofsessions as $key => $sessionentry) diff --git a/htdocs/comm/clients.php b/htdocs/comm/clients.php index 4d79e0730af..5fe7da7608e 100644 --- a/htdocs/comm/clients.php +++ b/htdocs/comm/clients.php @@ -183,9 +183,9 @@ if ($result) print ''; print ''; print ''; - print ''; + print ''; print '  '; - print ''; + print ''; print ''; print "\n"; diff --git a/htdocs/comm/contact.php b/htdocs/comm/contact.php index f3a297d38f8..3776867b457 100644 --- a/htdocs/comm/contact.php +++ b/htdocs/comm/contact.php @@ -144,7 +144,7 @@ if ($resql) print ''; print ''; print ' '; - print ''; + print ''; print "\n"; print ''; diff --git a/htdocs/comm/mailing/cibles.php b/htdocs/comm/mailing/cibles.php index 96b6e8b9f07..abf06b6e66f 100644 --- a/htdocs/comm/mailing/cibles.php +++ b/htdocs/comm/mailing/cibles.php @@ -419,8 +419,9 @@ if ($mil->fetch($_REQUEST["id"]) >= 0) print ''; // Source print ''; - print ''; - print '  '; + print ''; + print '  '; + print ''; print ''; print ''; diff --git a/htdocs/comm/mailing/fiche.php b/htdocs/comm/mailing/fiche.php index 4ff84ea25ac..c27de12a53f 100644 --- a/htdocs/comm/mailing/fiche.php +++ b/htdocs/comm/mailing/fiche.php @@ -915,7 +915,6 @@ else foreach($listofpaths as $key => $val) { print img_mime($listofpaths[$key]['name']).' '.$listofpaths[$key]['name']; - //print ' '; print '
'; } } diff --git a/htdocs/comm/mailing/liste.php b/htdocs/comm/mailing/liste.php index 827d7e4a6eb..6183dafb72c 100644 --- a/htdocs/comm/mailing/liste.php +++ b/htdocs/comm/mailing/liste.php @@ -127,7 +127,7 @@ if ($result) print ' '; if (! $filteremail) print ' '; print ' '; - print ''; + print ''; print ""; print "\n"; print ''; diff --git a/htdocs/comm/propal.php b/htdocs/comm/propal.php index 100b150630a..e1cbcd25717 100644 --- a/htdocs/comm/propal.php +++ b/htdocs/comm/propal.php @@ -1759,7 +1759,7 @@ else print ''; $html->select_propal_statut($viewstatut); print ''; - print ''; + print ''; print ''; print "\n"; print ''; diff --git a/htdocs/comm/prospect/prospects.php b/htdocs/comm/prospect/prospects.php index 28f0b281712..f3d0d14210d 100644 --- a/htdocs/comm/prospect/prospects.php +++ b/htdocs/comm/prospect/prospects.php @@ -340,7 +340,7 @@ if ($resql) // Print the search button print ''; - print ''; + print ''; print ''; print "\n"; diff --git a/htdocs/commande/liste.php b/htdocs/commande/liste.php index 49bfe93747c..a5c100e4bf1 100644 --- a/htdocs/commande/liste.php +++ b/htdocs/commande/liste.php @@ -194,7 +194,7 @@ if ($resql) print ' '; print ' '; print ''; - print ''; + print ''; print ''; print ''; $var=True; diff --git a/htdocs/compta/bank/account.php b/htdocs/compta/bank/account.php index c8a2b7baf77..e68d8edc5f9 100644 --- a/htdocs/compta/bank/account.php +++ b/htdocs/compta/bank/account.php @@ -422,7 +422,7 @@ if ($account || $_GET["ref"]) print ''; print ''; print ' '; - print ''; + print ''; print "\n"; print "\n"; diff --git a/htdocs/compta/bank/search.php b/htdocs/compta/bank/search.php index 213007a89af..fe1c86b214e 100644 --- a/htdocs/compta/bank/search.php +++ b/htdocs/compta/bank/search.php @@ -170,7 +170,7 @@ if ($resql) print ''; print ''; if (! empty($_REQUEST['bid'])) print ''; - print ''; + print ''; print ''; print ''; diff --git a/htdocs/compta/clients.php b/htdocs/compta/clients.php index 6a34816ccf9..1ee0afc4cfb 100644 --- a/htdocs/compta/clients.php +++ b/htdocs/compta/clients.php @@ -187,7 +187,7 @@ if ($resql) print ''; print ''; - print ''; + print ''; print ''; print "\n"; diff --git a/htdocs/compta/facture.php b/htdocs/compta/facture.php index ccc19484906..f9e985d24c9 100644 --- a/htdocs/compta/facture.php +++ b/htdocs/compta/facture.php @@ -3045,7 +3045,7 @@ else print ''; print ' '; print ''; - print ''; + print ''; print "\n"; if ($num > 0) diff --git a/htdocs/compta/facture/impayees.php b/htdocs/compta/facture/impayees.php index 54b66763e99..c508e65182f 100644 --- a/htdocs/compta/facture/impayees.php +++ b/htdocs/compta/facture/impayees.php @@ -300,7 +300,7 @@ if ($result) print ''; print ''; print ''; - print ''; + print ''; print ''; print ''; if ($conf->use_javascript_ajax) print ''.$langs->trans("All").' / '.$langs->trans("None").''; diff --git a/htdocs/compta/paiement/cheque/liste.php b/htdocs/compta/paiement/cheque/liste.php index c9fca7c773d..b5d75c25658 100644 --- a/htdocs/compta/paiement/cheque/liste.php +++ b/htdocs/compta/paiement/cheque/liste.php @@ -101,7 +101,7 @@ if ($resql) print ''; print ''; print ''; - print ''; + print ''; print ''; print "\n"; diff --git a/htdocs/compta/paiement/liste.php b/htdocs/compta/paiement/liste.php index 18954b60456..b560624867e 100644 --- a/htdocs/compta/paiement/liste.php +++ b/htdocs/compta/paiement/liste.php @@ -179,7 +179,7 @@ if ($resql) print ''; print ''; print ''; - print ''; + print ''; print ''; if ($conf->global->BILL_ADD_PAYMENT_VALIDATION) { diff --git a/htdocs/compta/param/comptes/liste.php b/htdocs/compta/param/comptes/liste.php index 87c5bea61b0..ff3d646e241 100644 --- a/htdocs/compta/param/comptes/liste.php +++ b/htdocs/compta/param/comptes/liste.php @@ -92,7 +92,7 @@ if ($resql) print ''; print ''; print ''; - print ''; + print ''; print ''; print ''; print ''; diff --git a/htdocs/compta/prelevement/bons.php b/htdocs/compta/prelevement/bons.php index 78ce041b684..62ef5030f4b 100644 --- a/htdocs/compta/prelevement/bons.php +++ b/htdocs/compta/prelevement/bons.php @@ -89,7 +89,7 @@ if ($result) print '
'; print ''; print ' '; - print ''; + print ''; print '
'; print ''; diff --git a/htdocs/compta/prelevement/demandes.php b/htdocs/compta/prelevement/demandes.php index 3cdf32aee7a..6190a5edf5a 100644 --- a/htdocs/compta/prelevement/demandes.php +++ b/htdocs/compta/prelevement/demandes.php @@ -111,7 +111,7 @@ if ($resql) print '
'; print ''; print ''; - print ''; + print ''; print ''; print '
'; diff --git a/htdocs/compta/prelevement/liste.php b/htdocs/compta/prelevement/liste.php index 32956d52d0c..a08809a7b62 100644 --- a/htdocs/compta/prelevement/liste.php +++ b/htdocs/compta/prelevement/liste.php @@ -123,7 +123,7 @@ if ($result) print ''; print ' '; print ' '; - print ''; + print ''; print ''; print ''; diff --git a/htdocs/compta/propal.php b/htdocs/compta/propal.php index a159d3cc523..c1414dae039 100644 --- a/htdocs/compta/propal.php +++ b/htdocs/compta/propal.php @@ -634,7 +634,7 @@ else print ''; $html->select_propal_statut($viewstatut); print ''; - print ''; + print ''; print ''; print "\n"; print ''; diff --git a/htdocs/compta/sociales/index.php b/htdocs/compta/sociales/index.php index fa3230f86b8..daafa781843 100644 --- a/htdocs/compta/sociales/index.php +++ b/htdocs/compta/sociales/index.php @@ -166,7 +166,7 @@ if ($resql) print ' '; print ' '; print ''; - print ''; + print ''; print ''; print "\n"; diff --git a/htdocs/compta/ventilation/lignes.php b/htdocs/compta/ventilation/lignes.php index ba8721582a0..00e30b23623 100644 --- a/htdocs/compta/ventilation/lignes.php +++ b/htdocs/compta/ventilation/lignes.php @@ -92,7 +92,7 @@ if ($result) print ' '; print ' '; print ''; - print ''; + print ''; print ''; print "\n"; diff --git a/htdocs/contact/index.php b/htdocs/contact/index.php index 0317aa6f793..1015553c7b9 100644 --- a/htdocs/contact/index.php +++ b/htdocs/contact/index.php @@ -279,8 +279,9 @@ if ($result) print $form->selectarray('search_priv',$selectarray,$search_priv,1); print ''; print ''; - print ''; - print '  '; + print ''; + print '  '; + print ''; print ''; print ''; diff --git a/htdocs/contrat/liste.php b/htdocs/contrat/liste.php index e03ecb75403..86f75354933 100644 --- a/htdocs/contrat/liste.php +++ b/htdocs/contrat/liste.php @@ -124,7 +124,7 @@ if ($resql) print ''; print ' '; //print ' '; - print ''; + print ''; print ""; print "\n"; print ''; diff --git a/htdocs/contrat/services.php b/htdocs/contrat/services.php index 4ca1c059c7d..11a57ae87ab 100644 --- a/htdocs/contrat/services.php +++ b/htdocs/contrat/services.php @@ -175,7 +175,7 @@ if ($resql) $filter_date2=dol_mktime(0,0,0,$_REQUEST['op2month'],$_REQUEST['op2day'],$_REQUEST['op2year']); print $form->select_date($filter_date2,'op2',0,0,1); print ''; - print ''; + print ''; print ""; print "\n"; print ''; diff --git a/htdocs/fichinter/index.php b/htdocs/fichinter/index.php index aca2231cc76..c3eba98e937 100644 --- a/htdocs/fichinter/index.php +++ b/htdocs/fichinter/index.php @@ -121,7 +121,7 @@ if ($result) print ''; print ' '; print ' '; - print ''; + print ''; print "\n"; $companystatic=new Societe($db); diff --git a/htdocs/fourn/commande/liste.php b/htdocs/fourn/commande/liste.php index 452968a06da..2865c71d2bb 100644 --- a/htdocs/fourn/commande/liste.php +++ b/htdocs/fourn/commande/liste.php @@ -141,7 +141,7 @@ if ($resql) print ''; print ''; print ''; - print ''; + print ''; print ''; print ''; diff --git a/htdocs/fourn/facture/impayees.php b/htdocs/fourn/facture/impayees.php index 2effbbf87df..b0a4d8fdba1 100644 --- a/htdocs/fourn/facture/impayees.php +++ b/htdocs/fourn/facture/impayees.php @@ -183,7 +183,7 @@ if ($user->rights->fournisseur->facture->lire) print ''; print ''; print ''; - print ''; + print ''; print ''; print "\n"; print ''; diff --git a/htdocs/fourn/facture/index.php b/htdocs/fourn/facture/index.php index d6bfbc7ba1a..f5b5a816c82 100644 --- a/htdocs/fourn/facture/index.php +++ b/htdocs/fourn/facture/index.php @@ -219,7 +219,7 @@ if ($resql) print ''; print ''; print ''; - print ''; + print ''; print ''; print "\n"; diff --git a/htdocs/fourn/liste.php b/htdocs/fourn/liste.php index 104000fe4a3..ce7882dfa7c 100644 --- a/htdocs/fourn/liste.php +++ b/htdocs/fourn/liste.php @@ -157,7 +157,7 @@ if ($resql) print ''; print ''; - print ''; + print ''; print ''; diff --git a/htdocs/fourn/product/liste.php b/htdocs/fourn/product/liste.php index 5d90e5e9581..a77cdcdea03 100644 --- a/htdocs/fourn/product/liste.php +++ b/htdocs/fourn/product/liste.php @@ -207,8 +207,9 @@ if ($resql) print ''; print ''; print ''; - print ''; - print '  '; + print ''; + print '  '; + print ''; print ''; print ''; print ''; diff --git a/htdocs/product/liste.php b/htdocs/product/liste.php index d542e48d3ec..0a4dbfb0a0b 100644 --- a/htdocs/product/liste.php +++ b/htdocs/product/liste.php @@ -340,8 +340,8 @@ if ($resql) print ' '; print ''; print ''; - print ''; - print ''; + print ''; + print ''; print ''; print ''; diff --git a/htdocs/product/stock/mouvement.php b/htdocs/product/stock/mouvement.php index be79c6e419c..01bf675e822 100644 --- a/htdocs/product/stock/mouvement.php +++ b/htdocs/product/stock/mouvement.php @@ -53,7 +53,7 @@ $offset = $conf->liste_limit * $page; if (! $sortfield) $sortfield="m.datem"; if (! $sortorder) $sortorder="DESC"; -if ($_REQUEST["button_removefilter"]) +if (GETPOST("button_removefilter")) { $year=''; $month=''; @@ -280,8 +280,10 @@ if ($resql) print ''; print ''; print ''; - print ''; - print '  '; + print ''; + print ''; + print '  '; + print ''; print ''; print "\n"; print ''; diff --git a/htdocs/projet/liste.php b/htdocs/projet/liste.php index 5f159ccb28b..b23eeec9eea 100644 --- a/htdocs/projet/liste.php +++ b/htdocs/projet/liste.php @@ -134,7 +134,7 @@ if ($resql) print ''; print ''; print ' '; - print ''; + print ''; print "\n"; while ($i < $num) diff --git a/htdocs/societe/lien.php b/htdocs/societe/lien.php index de5ce624aa5..466db0de1cd 100644 --- a/htdocs/societe/lien.php +++ b/htdocs/societe/lien.php @@ -233,7 +233,7 @@ if($_GET["socid"]) print ''; print ''; print ''; - print ''; + print ''; print ''; print "\n"; print ''; diff --git a/htdocs/societe/societe.php b/htdocs/societe/societe.php index fb6973894fa..1140db8f343 100644 --- a/htdocs/societe/societe.php +++ b/htdocs/societe/societe.php @@ -364,9 +364,9 @@ if ($resql) print ''; // Type (customer/prospect/supplier) print ''; - print ''; + print ''; print '  '; - print ''; + print ''; print ''; print "\n";