1
kaotik
Content 1.3 module, adding sanitation
  • 2009/3/16 19:14

  • kaotik

  • Just can't stay away

  • Posts: 861

  • Since: 2004/2/19


Replace:
le="color: #000000"><?php $id = isset($_GET['id']) ? intval($_GET['id']) : 0; $currentPage = isset($_GET['page']) ? intval($_GET['page']) : 0;


With:
le="color: #000000"><?php $id = isset($_GET['id']) ? intval($_GET['id']) : 0; $currentPage = isset($_GET['page']) ? intval($_GET['page']) : 0; //Sanitize User Input Added by Kaotik 16-03-09 $myts = myTextSanitizer::getInstance(); $id=$myts->addslashes($id); $currentPage=$myts->addslashes($currentPage); // End change


Content is still one of my preferred modules, small, simple and effective, great for simple page wraps

2
trabis
Re: Content 1.3 module, adding sanitation
  • 2009/3/16 19:22

  • trabis

  • Core Developer

  • Posts: 2269

  • Since: 2006/9/1 1


Sorry, Why is that?

3
ghia
Re: Content 1.3 module, adding sanitation
  • 2009/3/17 10:46

  • ghia

  • Community Support Member

  • Posts: 4953

  • Since: 2008/7/3 1


I can also not see what is left to improve after intval().
It seems superfluous code to me.

Login

Donat-O-Meter

Stats
Goal: $15.00
Due Date: Aug 31
Gross Amount: $0.00
Net Balance: $0.00
Left to go: $15.00
Make donations with PayPal!