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

319 user(s) are online (248 user(s) are browsing Support Forums)


Members: 0


Guests: 319


more...

Donat-O-Meter

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

Latest GitHub Commits