我怎樣才能提高這個php代碼的安全性？

Question

我創建的PHP代碼：

<?php 

    include("../config.php"); 

    if(!isset($_COOKIE["sessionid"])) { 
     header("location:../error_pages/403.php"); 
    } else { 
     $value = $_COOKIE["sessionid"]; 
     $query = "SELECT * FROM users WHERE sessionid = ?"; 

     $stmt = $conn->prepare($query); 
     $stmt->bind_param("s",$value); 
     $stmt->execute(); 
     $result = $stmt->get_result(); 

     $rowcount = $result->num_rows; 
     if ($rowcount != 1) { 
      header("location:../error_pages/403.php"); 
     } else { 
      $file = "/path/to/file/"; 
      header('Content-Description: File Transfer'); 
      header('Content-Disposition: attachment; filename='.basename("File_Name")); 
      ob_clean(); 
      flush(); 
      readfile($file); 
     } 
    }; 
?> 

我不知道是否有一種方法的會話ID的cookie可以被操縱，以允許SQL注入。以及此下載是否安全。該文件不存儲在可公開訪問的文件夾中。

Answer 1

通過使用查詢參數，您可以安全地進行SQL注入。

您的代碼可能容易受到會話劫持。以下是一些提示和資源：

• 對所有請求使用TLS（https），這會破壞數據包嗅探器。
• http://phpsecurity.org/有關於會話管理的免費樣章。或者拿到整本書，這很好。
• What is the best way to prevent session hijacking?
• https://www.owasp.org/index.php/Session_Management_Cheat_Sheet