Насколько безопасен логин Dreamweaver? - PullRequest
Новая
       5

Насколько безопасен логин Dreamweaver?

2 голосов
/ 22 марта 2012

Когда мы используем dreamweaver для создания автоматически сгенерированного входа в систему - насколько он безопасен?

при этом Dreamweaver создает 3 папки: - 

_mmServerScripts
_notes
Connections

и добавляет это кстраница login.php: - 

<?php require_once('../Connections/da.php'); ?>
<?php
if (!function_exists("GetSQLValueString")) {
function GetSQLValueString($theValue, $theType, $theDefinedValue = "", $theNotDefinedValue = "") 
{
  if (PHP_VERSION < 6) {
    $theValue = get_magic_quotes_gpc() ? stripslashes($theValue) : $theValue;
  }

  $theValue = function_exists("mysql_real_escape_string") ? mysql_real_escape_string($theValue) : mysql_escape_string($theValue);

  switch ($theType) {
    case "text":
      $theValue = ($theValue != "") ? "'" . $theValue . "'" : "NULL";
      break;    
    case "long":
    case "int":
      $theValue = ($theValue != "") ? intval($theValue) : "NULL";
      break;
    case "double":
      $theValue = ($theValue != "") ? doubleval($theValue) : "NULL";
      break;
    case "date":
      $theValue = ($theValue != "") ? "'" . $theValue . "'" : "NULL";
      break;
    case "defined":
      $theValue = ($theValue != "") ? $theDefinedValue : $theNotDefinedValue;
      break;
  }
  return $theValue;
}
}
?>
<?php
// *** Validate request to login to this site.
if (!isset($_SESSION)) {
  session_start();
}

$loginFormAction = $_SERVER['PHP_SELF'];
if (isset($_GET['accesscheck'])) {
  $_SESSION['PrevUrl'] = $_GET['accesscheck'];
}

if (isset($_POST['username'])) {
  $loginUsername=$_POST['username'];
  $password=$_POST['password'];
  $MM_fldUserAuthorization = "";
  $MM_redirectLoginSuccess = "../index.php";
  $MM_redirectLoginFailed = "index.php";
  $MM_redirecttoReferrer = false;
  mysql_select_db($database_da, $da);

  $LoginRS__query=sprintf("SELECT user_id, user_password FROM users WHERE user_id=%s AND user_password=%s",
    GetSQLValueString($loginUsername, "int"), GetSQLValueString($password, "text")); 

  $LoginRS = mysql_query($LoginRS__query, $da) or die(mysql_error());
  $loginFoundUser = mysql_num_rows($LoginRS);
  if ($loginFoundUser) {
     $loginStrGroup = "";

    if (PHP_VERSION >= 5.1) {session_regenerate_id(true);} else {session_regenerate_id();}
    //declare two session variables and assign them
    $_SESSION['MM_Username'] = $loginUsername;
    $_SESSION['MM_UserGroup'] = $loginStrGroup;       

    if (isset($_SESSION['PrevUrl']) && false) {
      $MM_redirectLoginSuccess = $_SESSION['PrevUrl'];  
    }
    header("Location: " . $MM_redirectLoginSuccess );
  }
  else {
    header("Location: ". $MM_redirectLoginFailed );
  }
}
?>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
<title>Untitled Document</title>
</head>

<body>
<form name="login" action="<?php echo $loginFormAction; ?>" method="POST" target="_self">
<input name="username" type="text" />
<input name="password" type="password" />
<input name="login_button" type="submit" />
</form>
</body>
</html>

Также он использует mysql_pconnect для подключения mysql: - 

<?php
# FileName="Connection_php_mysql.htm"
# Type="MYSQL"
# HTTP="true"
$hostname_da = "localhost";
$database_da = "database";
$username_da = "root";
$password_da = "password";
$dacreint = mysql_connect($hostname_da, $username_da, $password_da) or trigger_error(mysql_error(),E_USER_ERROR); 
?>

Это 100% эффективно и безопасно?И если нет, мы можем внести в него некоторые изменения, чтобы сделать его максимально безопасным ... Потому что это действительно ускоряет процесс.

Ответы [ 2 ]

3 голосов
/ 22 марта 2012

Редактировать: В принятом ответе на этот вопрос (от того же автора? Почему?) Указаны две уязвимости, которые я пропустил: PHP_SELF уязвим для XSS и возвращаться к mysql_escape_string() не нужно (хотя в любом случае это не является реальной проблемой). Вместо того, чтобы украсть точки у MrCode, я призываю всех закрыть это и одобрить лучший ответ MrCode :)

Это не очень красивый код, но с точки зрения безопасности и эффективности он выглядит довольно хорошо. Принимается во внимание, что магические кавычки в PHP 6 сняты с производства, похоже, что они корректно экранируют все входящие данные формы, и проверяют наличие переменных перед тем, как получить к ним доступ.

Единственное, что может вызвать проблемы - это лишний разрыв строки: 

<?php require_once('../Connections/da.php'); ?>   <---- here
<?php

, который в некоторых ситуациях будет мешать перенаправлению заголовка. Я бы избавился от этого и просто сделал 

<?php require_once('../Connections/da.php'); 
      if (!function_exists("GetSQLValueString")) {

также вместо 

  .... or die(mysql_error());

можно использовать 

  ....  or trigger_error(mysql_error(), E_USER_ERROR);

для предотвращения отображения сообщений об ошибках SQL в производственных средах.

1 голос
/ 22 марта 2012

Возможно, это не очень хорошая идея: 

mysql_query($LoginRS__query, $da) or die(mysql_error());

Это показывает клиенту объяснение ошибки mysql, возможно, раскрывая детали схемы БД вашей пользовательской таблицы.

...