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.

Login

Who's Online

300 user(s) are online (49 user(s) are browsing Support Forums)


Members: 0


Guests: 300


more...

Donat-O-Meter

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

Latest GitHub Commits