Chuyển hướng PHP này có an toàn không?

Question

Hmm Tôi thực sự không thể phân biệt bất kỳ sự bất an nào nhưng đã tự hỏi liệu bạn có thể, nếu vậy làm thế nào để có thể vá/sửa đổi?

Heres mã:

header("Location: http://example.com/search/{$_POST['term']}/{$_POST['type']}"); 

Các trang web mà tôi đang chuyển hướng quá không xác nhận & sanitization về phía họ, nhưng những gì tôi quan tâm là - không an toàn chuyển hướng này dưới mọi hình thức (trên mặt của tôi - nhìn thấy khi tôi đang sử dụng trực tiếp $_POST 's).

Đánh giá cao tất cả trợ giúp.

PS: Chỉ trở nên tò mò vì tôi luôn nghĩ rằng việc sử dụng dữ liệu nhập của người dùng không được đánh giá là nguy hiểm (hoặc ít nhất là áp dụng cho XSS và SQLi).

Answer 1

Nhìn chung, đối với hầu hết các trang web đang chạy phiên bản PHP hiện đại, là an toàn.

Có hai vấn đề ở bàn tay:

• Một người sử dụng độc hại có thể đánh lừa nạn nhân vào vô tình ghé thăm bất kỳ trang của mẫu đơn /search/*/* trên trang web bằng cách liên kết chúng vào một trang độc hại mà POSTS đến trang với chuyển hướng của bạn. (Lưu ý rằng chúng không chỉ giới hạn ở hai dấu gạch chéo sau /search vì biến POST của chúng có thể chứa dấu gạch chéo.) Điều này tương tự như việc trao cho ai đó một URL bit ngắn được rút ngắn để chuyển hướng chúng, vì vậy nó không quá tệ.
• HTTP response splitting. Nếu người dùng độc hại bao gồm dòng mới (cụ thể là CRLF/\r\n) trong dữ liệu POST của họ, họ có thể thực hiện cuộc gọi header() để xuất nhiều tiêu đề, bao gồm tiêu đề để đặt cookie, v.v. Tuy nhiên, kể từ PHP 5.1.2 this has been fixed.

Answer 2

Có, nó khá không an toàn. Người ta KHÔNG BAO GIỜ BAO GIỜ nên đặt đầu vào người dùng không được kiểm soát vào một tập lệnh. Tôi khuyên bạn nên tối thiểu tối thiểu, sử dụng filter_input_array để khử trùng mảng $ _POST của bạn trước khi thực hiện bất kỳ điều gì với nó.

Answer 3

Hy vọng rằng tầng web nhận URL được chuyển hướng từ ứng dụng khách sẽ thực hiện xác thực trên toàn bộ URL, vì vậy bạn có thể an toàn. Nhưng nó sẽ không làm tổn thương để chạy hai lĩnh vực thông qua xác nhận đơn giản trước khi gửi chuyển hướng.

Answer 4

Đây không phải là vấn đề bảo mật, nhưng có lẽ bạn nên mã hóa URL các đối số này bằng urlencode($_POST['term']).

Answer 5

Tôi nghĩ bạn sẽ tìm thấy điều này là read đáng giá.