	[x] Posted via EE Mobile
	Search, ask, and monitor your questions on the go with EE Mobile. Visit Experts Exchange from your mobile device and never be out of touch again.

Question

	[x] Attachment Details

[x]

The Solution Rating System

With so many solutions, how can you tell which solutions are most likely to help you and which ones are not? To provide you with a tool to use, we rate our solutions based on various elements that most accurately determine if a solution is a quality solution. To explain what factors affect the solution rating, here are the elements we take into consideration when formulating our solution rating.

The Grade of the Solution
The Zone Rank of the Expert Providing the Solution
The Number of Author and Expert Comments
The Number of Experts Contributing
The Feedback of the Community

Your Input Matters
Because of the way the system is set up, the most important variable in this equation is you. As a member of Experts Exchange, you are able to cast your vote on the quality of the solutions in regard to how complete, accurate, helpful and easy to understand each solution is. When you provide your feedback, each rating is adjusted accordingly. So, if you see a solution that has a poor rating that you think is a good solution, let us know by rating it. As you do, the rating will be adjusted and will become more accurate for other members of our site.

If you have any suggestions that you would like to make for our rating system, please ask a question in the Suggestions Zone of Community Support.

Thank you!

9.3

PHP secure login code

Asked by paul_taylor_22 in Security Issues in Programming, PHP and Databases

Tags: php, sessions, security, login

Hi,
Apology's if the answer to this is found on another post, not found it so far.
I am looking to implement a system which includes a login portal, for users (stored in db) to login using PHP sessions.
Below is the code that i have used, which works, but the security aspect is concerning me.
Could you let me know if this is a secure way to implement this, and if i can improve it in any way?

thanks in advance

View the Solution FREE for 30 Days

         
           
             1:
2:
3:
4:
5:
6:
7:
8:
9:
10:
11:
12:
13:
14:
15:
16:
17:
18:
19:
20:
21:
22:
23:
24:
25:
26:
27:
28:
29:
30:
31:
32:
33:
34:
35:
36:
37:
38:
39:
40:
41:
42:
43:
44:
45:
46:
47:
48:
49:
50:
51:
52:
53:
54:
55:
56:
57:
58:
59:
60:
61:
62:
63:
64:
65:
66:
67:
68:
69:
70:
71:
72:
73:

           
           
             All files include check_login.php:
<?php
        session_start();
        if (!isset($_SESSION['user_id'])){
                include("login.php");
                exit();
        }
?>
 
 
 
login.php:
<html>
<head>
</head>
<body>
<form action="login_submit.php" method="POST">
Username: <input type="text" name="username">
Password: <input type="password" name="password">
</form>
</body>
</html>
 
 
 
login_submit.php
<?php
        include("includes/db_connect.php");
 
        $username=$_POST['username'];
        $password=$_POST['password'];
 
        $qryGetUser="SELECT * FROM user WHERE username='$username' LIMIT 1";
        $rsGetUser=mysql_query($qryGetUser) or die(mysql_error());
 
        //USERNAME CORRECT?
        if (mysql_num_rows($rsGetUser)==0){
                echo "Username or password incorrect.";
                exit();
        }
        $rowUser=mysql_fetch_array($rsGetUser);
 
        //PASSWORD CORRECT?
        if ($rowUser['password']!=md5($password)){
                echo "Username or password incorrect.";
 
                //LOCK ACCOUNT IF THIS IS THIRD ATTEMPT
                if ($rowUser['failed_attempts']>1){
                        $qryUpdateUser="UPDATE user SET failed_attempts=failed_attempts+1, locked='1' WHERE user_id='".$rowUser['user_id']."'";
                }
                //ELSE INCREMENT FAILED ATTEMPTS
                else{
                        $qryUpdateUser="UPDATE user SET failed_attempts=failed_attempts+1 WHERE user_id='".$rowUser['user_id']."'";
                }
                $rsUpdateUser=mysql_query($qryUpdateUser) or die(mysql_error());
                exit();
        }
 //IF ACCOUNT LOCKED
        if ($rowUser['locked']=="1"){
                $qryUpdateUser="UPDATE user SET failed_attempts=failed_attempts+1 WHERE user_id='".$rowUser['user_id']."'";
                $rsUpdateUser=mysql_query($qryUpdateUser) or die(mysql_error());
                echo "<b>Error: </b>Your account has been disabled.<br/>Please contact your systems administrator.";
                exit();
        }
 
        //LOGGED IN
        session_start();
        $_SESSION['username']=$rowUser['username'];
        $_SESSION['user_id']=$rowUser['user_id'];
        $_SESSION['login_time']=date(" H:i:s d-m-Y");
 
        header("Location: main.php");
?>

           
         

Related Solutions

Keywords: PHP secure login code

	Title
1	How would you set up a new parallel E…

Loading Advertisement...

20091111-EE-VQP-92 - Hierarchy / EE_QW_3_20080625

Hall of Fame

1. richrumble4,100
2. MuraliKanth4,000
3. mrcoffee...3,600
4. jakopriit2,600
5. NovaDeni...2,500
6. rlandquist2,000
6. Locklear2,000
6. JimU2,000
6. MunterMan2,000
6. JBlond2,000
6. egrep2,000
6. leegclystv...2,000
6. hiteshgold...2,000
6. matthews...2,000
6. sdesign2,000
6. azadisaryev2,000
6. MSWarrior2,000
6. bec0s2,000
6. cminear2,000
6. GiftsonDJ...2,000
6. kevthedude2,000
6. i_t2,000
6. twocandles2,000
6. wht19862,000
6. AhmedC...2,000

1. mrcoffee...5,600
2. richrumble4,600
3. MuraliKanth4,000
4. jason11783,700
5. ahoffmann3,500
6. jakopriit2,600
7. TheLearn...2,500
7. Squinky2,500
7. NovaDeni...2,500
10. CoccoBill2,150
11. dvz2,000
11. rlandquist2,000
11. Locklear2,000
11. JimU2,000
11. Mikal6132,000
11. tedbilly2,000
11. WyTyz2,000
11. MunterMan2,000
11. cs97jjm32,000
11. BestWay2,000
11. LSMCons...2,000
11. gatorvip2,000
11. JBlond2,000
11. amahesh...2,000
11. egrep2,000
11. Ray_Pas...2,000
11. nedfine2,000
11. leegclystv...2,000
11. hiteshgold...2,000
11. franked_it2,000
11. mkielar2,000
11. matthews...2,000
11. sdesign2,000
11. azadisaryev2,000
11. khaledf2,000
11. MSWarrior2,000
11. bec0s2,000
11. cminear2,000
11. GiftsonDJ...2,000
11. kevthedude2,000
11. i_t2,000
11. twocandles2,000
11. wht19862,000
11. AhmedC...2,000

PHP secure login code

Asked by paul_taylor_22 in Security Issues in Programming, PHP and Databases

Tags: php, sessions, security, login

About this solution