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:
$id = isset($_GET['id']) ? intval($_GET['id']) : 0;
$currentPage = isset($_GET['page']) ? intval($_GET['page']) : 0;

With:
$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.