사용자를 자신의 프로필 페이지로 제한

Question

내 웹 사이트에는 사용자 로그인 시스템이 있습니다. 로그인하면 자신의 프로필 페이지로 이동합니다.이 프로필 페이지는 해당 사용자가 사용하지 않은 것입니다. 문제는 사용자가 uid를 변경하고 다른 사용자의 프로필로 이동하고 해당 사용자로 행동하는 것을 막을 수있는 방법이 없었기 때문입니다. 이를 막기 위해 사용자의 uid가 URL의 uid가 아닌 경우 자신의 프로필로 리디렉션되는 URL/MySQL 시스템을 구현했습니다. 여기서 문제는 프로필에 URL을 변경하는 양식이 있으므로 uid 쿼리를 제거하면 uid가 없어 페이지가 생성되어 프로필로 이동하고 양식 입력을 무시한다는 것입니다.

코드는 다음과 같습니다

<?php 
mysql_connect ('x', 'x', 'x'); 
mysql_select_db ('x'); 

if(isset($_COOKIE['wd_un'])) { 
    $un = $_COOKIE['wd_un']; 
    $pass = $_COOKIE['wd_pass']; 

    $cook = "SELECT * FROM x WHERE username = '$un' AND password = '$pass' limit 1"; 
    $cookr = mysql_query($cook) or die (mysql_error()); 
     if(mysql_num_rows($cookr) == 0) { 
      header ("Location: index.php"); 
     } 
     else { 
      $urluid = mysql_real_escape_string($_GET['uid']); 
      $uidcheck = "SELECT * FROM x WHERE username = '$un' AND password = '$pass'"; 
      $uidcheckq = mysql_query($uidcheck) or die (mysql_error()); 
      while($rcu = mysql_fetch_assoc($uidcheckq)) { 
       $dbuid = $rcu['uid']; 
         if($urluid != $dbuid) { 
          header ("location: home.php?uid=$dbuid"); 
         } 
         else { 
         } 
      } 
     } 
    } 
mysql_close(); 
?> 

작업 주위에 있습니까?

Answer 1

여기에있는이 코드 블록은 불량으로 가득합니다.

먼저 쿠키에 사용자 암호를 절대로 저장하지 않아야합니다. 쿠키에 세션 ID 만 저장하고 나머지 세션 데이터를 사용자 ID 및 기본 액세스 권한이 필요한 다른 항목이 포함 된 세션 테이블에 저장해야합니다. 이 테이블에도 있지 않아.

이제 URL에 user_id를 사용하여 상호 참조로 인해 사람들을 안전하게 보호 할 수 있습니다.

물론 쿠키 ID를 기반으로 가져온 세션 테이블에서 mysql 결과를 참조하십시오. 분명히 그들이 일치하지 않으면 그들을 부팅합니다.

리디렉션 양식의 경우 게시를 처리하는 방법을 다시 구성해야합니다. 프로필 페이지는 항상 쿠키의 세션 ID와 관련된 프로필 만 가져올 수 있습니다. 그러면 URL에 대한 의존성이 제거되고이 문제를 완전히 해결할 수 있습니다.

또한 입력 내용을 살균하려면 mysql_real_escape_string()을 살펴보십시오. 맹목적으로 mysql 쿼리에 대한 쿠키 정보를 받아들이는 것은 엄청나게 위험합니다. 당신이 실제로 당신의 위치에있는 거대한 주입 구멍을 남기는 것을 조준하면 않는 한.

Answer 2

URL에서 사용자를 추출하는 대신 사용자를 식별하려면 쿠키 ID를 사용해야합니다. cookieID가 UserID와 일치하지 않으면 자신의 프로필로 리디렉션합니다.

기본적으로 개인 정보 용 사용자 ID를 전달하는 데 URL을 사용하지 마십시오. 항상 쿠키를 참조합니다.

사용자를 식별하기 위해 URL 문자열에만 의존하면 설명 된 사용성 문제 위에 거대한 보안 구멍이 있습니다.