CAT.NET : 취약점 또는 거짓 긍정? 가끔 시리즈 2

Question

: Here's the first one

는 CAT.NET 다음은 ASP.NET에서 정품 취약점하거나 거짓 긍정적이라고 맞습니까?

var myInt = Int32.Parse(txtUserInput.Text); 

Response.Redirect(string.Format("myPage.aspx?myId={0}", myInt); 

CAT.NET은 myInt 인코딩을 통한 치료가 필요한 리디렉션 취약점으로보고합니다.

Answer 1

그 위험하지만 전화를하지 않을 자신이 아닌 구문 분석에 실패하면 나는 때문에 나쁜 사용자 입력에 늘 예외를 슬로우로

int myInt; 
if(Int32.TryParse(txtUserInput.Text,out myInt)){ 
    Response.Redirect(string.Format("myPage.aspx?myId={0}", myInt); 
    } 

내 마음 청소기가 그것을 자신을 작성합니다 우리는 얼마나 int를 명시 적으로 입력합니다.

모든 오류 처리 코드는 끝에 else 문으로 묶을 수 있습니다.

Answer 2

저는 그렇게 생각하지 않으므로 예외가 발생할 수 있으므로 TryParse가 더 나은 방법 일 수 있습니다. 사용자 입력 및 리디렉션을 기반으로하기 때문에 고함을 지르고 있습니다. 아마도 조금 나빠질 수 있습니다.

Answer 3

이 코드로 인해 악용 될 수있는 취약점이 없습니다. 모든 취약점은 myPage.aspx가 myId 값으로 수행하는 작업의 결과이며 사용자 URL은 작성되지 않습니다. 누구나 myPage.aspx를 쿼리 문자열에서 원하는대로 쉽게 명중시킬 수 있습니다.

그러나 두 행 사이의 코드에서 아무 것도 남겨 두지 않은 경우를 가정하면 좋지 않습니다. txtUserInput.Text에 숫자 만 포함하고 허용 가능한 값 내에 있는지 확인해야합니다.

부적절한 URL 생성이 아닌 게시 ​​된 페이지의 사용자 제공 데이터를 부적절하게 파싱하여 악용 될 수 있습니다. 양식에 입력 된 내용으로 인해 웹 사이트가 깨진 URL을 쓰지 않도록하려면 좋은 생각이지만 프런트 엔드의 입력 유효성 검사는 보안과 관련이 없습니다. 중요한 것은 모든 게시물이나 쿼리 문자열이 위조 될 수 있으므로 이을 허용하는 코드입니다.