tags:

views:

28

answers:

2

hi,

$page = $_GET['page'];
if(isset($page))
if(!preg_match('/[\w\d_]+/i', $page)) die("Error");

I want to allow alphanum and underscore,

above code works but let say i set 123..., this works too. Is preg_match will not validate behind entry?

A: 

You need to use anchors as:

/^\w+$/

\w already has \d and _

codaddict
And `\w` does already contain uppercase and lowercase variants.
Gumbo
@Gumbo: Thanks for pointing.
codaddict
+2  A: 

The regex will match as long as an alphanumeric appears as a substring of $page. Since 123... contains the substring 123 it will pass your regex.

Use

/^\w+$/

to match the whole string. (\w already means [a-zA-Z0-9_] so your \d, _ and the i modifier are redundant.)

KennyTM