Jump to content

Recommended Posts

It's not too bad, though I do have a few issues with it.

 

First off it's completely JavaScript dependent. Then with JavaScript on, although the effects and pop ups may look pretty, they slow down the whole process quite considerably.

 

There's no human validation like captcha images or email verification, so the whole thing's wide open to spamming!

 

If you upload a file, then upload another with the same name, the old one is automatically over-written, this could cause some serious frustration!

 

Also the links in the top right hand corner seem to vary on every page.

 

Adam

you need to work on injection

 

something like

 

<?php
function clean($text){
$text=strip_trags(trim(mysql_real_escape_string($text)));
}
$passwordview=clean($_POST['passwordview']);
$userview=clean($_POST['userview']);
$mysqli= new mysqli('localhost','username','password','database'); //connect
$sql= $mysqli ->prepare("INSERT* INTO users WHERE username=? AND password=?");//prepare statement
$sql ->bind_param('ss',$userview,$passwordview);//bind parameters to replace ? in statement
$sql ->execute();//execute statement
?>

 

 

 

i have used an example of the MYSQLI prepared statements to weed out any injection before it gets submitted to the database also before i even prepare the statement i use my clean() function to weed out most if not all of the injection attempts.

 

i have explained in the example how the code works if you need to refer to something feel free to google MYSQLI prepared statements. ;)

Hello,

 

thank you, But i'm not sure what is injection? could you please tell me about it and the code ?

 

SQL Injection is the biggest one you want to watch out for: Read here

Cross-site Scripting (XSS) Exploit is another issue: Read here

 

For file uploading you really need to filter the names, as on some operating system a / is allowed but renders that file like a folder and is un-deletable or a ' can cause a file issue. So filtering that as well will save you some major headaches.

×
×
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue.