미처서의미처서만든블로그

http://min-blog.tistory.com/1544

http://www.designboard.net/dboard/files/main.php

http://joasoft.net/?r=home

http://getbootstrap.com/

SQL Injection 은 사용자가 요상한 문자열을 입력해서 쿼리를 요상하게 바꿔 버려서 아무 아이디로나 로그인이 훅 되버린다던지 하는 쪼금은 싱거운 해킹방법중의 하나라고 나는 알고 있다.


뭐 예를들어 아이디 비번을 입력하고 로그인 할때

아이디로  admin' OR '1'='1 요런걸 입력하고 로그인을 했을때 admin 으로 싹 로그인 되 버리는 어처구니 없는 일이 발생하는 것이 SQL Injection 처리를 쪽바로 해주지 않았을때 일어나는 참혹한 일이다.


SQL Injection 이 발생하는 전형적인 코드는 다음과 같다.

위 코드처럼 사용자 정보를 받아서 쿼리를 실행하고 실행결과가 1 이면 로그인 시켜주는 코드가 있다.

그런데 사용자가 아디디로 admin' OR '1'='1  을 입력하고 비밀번호로 abcd 를 입력한 후 로그인 시켜주셈~ 이라고 하면 우째될까?

위 코드대로라면  select count(*) from member where userid='amdin' OR '1'='1' and password = 'abcd'   이 쿼리가 실행되 버린다 -_-;;;

무섭다. 만약 사용자 테이블에 userid 가 admin 이라는 레코드가 있다면 admin 으로 로그인 되버리는 것이다. 

이런 어처구니 없는일이 놀랍게도 몇몇 허접한 사이트에서는 더러 통하는데도 있다 . -_-



아무튼 헛소리는 고만하고 본론으로 들어가서 SQL Injection 을 방지하려면 우째해야 될까? 

Statement 대신  PreparedStatement 를 쓰면 된다. 요렇게.

PreparedStatement 만 쓰면 해결된다고 해서 또 요렇게는 쓰지 말길 -.-;

※ 하이버네이트는 한번도 안써봐서 잘 모르겠는데 mybatis 나 ibatis 같은 경우는 무난하게 #{xxxx} 요런것만 썼다면 SQL Injection 이 방지된다. 

그런데 WHERE 절에 ${xxxx} 이렇게 있다면 다시 한번 생각해 보기 바란다. 

${xxxx} 이거는 문자열 더하는것과 똑같기 때문에 위험하다. ${xxxx} 요런건 ORDER BY 절 같은데만 사용하자.

XML

<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
    android:orientation="vertical"
    android:layout_width="fill_parent"
    android:layout_height="fill_parent"
    android:background="@drawable/그림파일이름"> 

<div class="row row-form">

<div class="col-md-12">

<a

onclick="mqigoWritePage('mainQnA.do?nowpage=1&start_row=1&finish_row=10');"

class="btn btn-success pull-right">목록으로</a> <a

onclick="delqnaPage('mainQnaDelete.do?qna_id=${qna.qna_id}');"

class="btn btn-success pull-right">질문삭제</a>

</div>

</div>

<!-- 코멘트삭제 -->

<script type="text/javascript">

function delCommentPage(updateurl) {

$.ajax({

url : updateurl,

success : function(resultData) {

qloadtabpage('mainQnAInner.do?qna_id=${qna.qna_id}');

}

});

}

</script>

<!-- 질문삭제 -->

<script type="text/javascript">

function delqnaPage(updateurl) {

$.ajax({

url : updateurl,

success : function(resultData) {

qloadtabpage('mainQnA.do?nowpage=1&start_row=1&finish_row=10');

}

});

}

</script>

<script type="text/javascript">

function qloadtabpage(loadingurl) {

$.ajax({

url : loadingurl,

type : 'get',//'post',

timeout : 1000,

traditional : true,

async : false,

error : function() {

alert('네트워크가 불안정합니다.');

},

success : function(resultData) {

$("#mainqnainnerview").html(resultData);

}

});

}

</script>

맵퍼

<select id="selectByLecIdwithRownums" parameterType="map"

resultType="databoardvo">

select databoard_id, databoard_title,

databoard_regid,databoard_content,databoard_filename, databoard_uploaddate, lecture_id,

databoard_regname,category

from

(

select rownum rn, ln.*

from

(select *

from databoard

where lecture_id=#{lecture_id}

order by databoard_id desc) ln

)

where rn between #{start_row} and #{finish_row}

order by databoard_id desc

</select>

컨트롤러

@RequestMapping(value = "databoard.do", method = RequestMethod.GET)

public ModelAndView dataBoard(HttpServletRequest request,

HttpSession session) {

ModelAndView mv = new ModelAndView();

int start_row = Integer.parseInt(request.getParameter("start_row"));

int finish_row = Integer.parseInt(request.getParameter("finish_row"));

int nowpage = Integer.parseInt(request.getParameter("nowpage"));

LectureVO lecture = (LectureVO) session.getAttribute("lecture");

int lecture_id = 0;

lecture_id = lecture.getLecture_id();

System.out.println(lecture_id);

System.out.println(start_row + "~~~~" + finish_row + "~~~~"

+ lecture_id);

ArrayList<DataBoardVO> datalist = dataBoardService

.selectByLecIdwithRownums(lecture_id, start_row, finish_row);

// System.out.println(dataBoardService.selectByLecId(lecture_id));

for (DataBoardVO aa : datalist) {

System.out.println(aa);

}

// 게시글 총합 구하기

int boardcount = dataBoardService.countBoardByLecId(lecture_id);

mv.addObject("databoardtotal", boardcount);

mv.addObject("datalist", datalist);

mv.addObject("nowpage", nowpage);

mv.setViewName("lecture/dataBoard");

return mv;

}

JSP

  <c:set var="previous" value=""></c:set>

<c:set var="next" value=""></c:set>

<c:set var="pid" value='id="previous"'></c:set>

<c:set var="nid" value='id="next"'></c:set>

<c:set var="pages" value="${databoardtotal/10}"></c:set>

<c:if test="${nowpage<=1}">

<c:set var="previous" value='class="disabled"'></c:set>

<c:set var="pid" value=""></c:set>

</c:if>

<c:if test="${pages<nowpage}">

<c:set var="next" value='class="disabled"'></c:set>

<c:set var="nid" value=""></c:set>

</c:if>

<div class="row">

<ul class="pager">

<li ${previous }><a ${pid } >Previous</a></li>

<li>${nowpage}</li>

<li ${next }><a ${nid } >Next</a></li>

</ul>

</div>

<!-- jQuery (necessary for Bootstrap's JavaScript plugins) -->

<script

src="https://ajax.googleapis.com/ajax/libs/jquery/1.11.1/jquery.min.js"></script>

  <script type="text/javascript">

var nowpage =${nowpage};

var start_row ;

var finish_row ;

$(document).ready(function() {

$("#previous").on("click", function() {

--nowpage;

start_row = nowpage * 10 - 9;

finish_row = nowpage * 10;

loadtabpage('databoard.do?start_row='+start_row+'&finish_row='+finish_row+'&nowpage='+nowpage);

});

$("#next").on("click", function() {

++nowpage;

start_row = nowpage * 10 - 9;

finish_row = nowpage * 10;

loadtabpage('databoard.do?start_row='+start_row+'&finish_row='+finish_row+'&nowpage='+nowpage);

});

});

</script> 

Server.xml line 65    

<Connector URIEncoding = "UTF-8" connectionTimeout="20000" port="8088" protocol="HTTP/1.1" redirectPort="8443"/>