기본 콘텐츠로 건너뛰기

CVE-2017-12061_CVE-2017-12062

CVE-2017-12061, CVE-2017-12062

MantisBT는 오픈소스 이슈 트래킹 도구 중 하나이며 간편한 사용법으로 많은 사용자를 보유하고 있다. 최근 Cross Site Scripting 취약점 (CVE-2017-12061, CVE-2017-12062)이 발생하였고 조치가 완료되었다. 수정된 소스코드를 통해서 해당 취약점을 살펴보자.

CVE-2017-12061

취약점이 발생된 곳은 /admin/install.php 소스이다.

  • 영향 버전: 1.3.11 and older, 2.5.1 and older
  • 조치 버전: 1.3.12, 2.5.2, 2.6.0 (not yet released*)

POC

http://mantis.server/admin/install.php?install=3&database_name=%3Ch1%3EXSS&admin_username=%3Ch1%3EXSS

http://mantis.server//admin/install.php?install=3&database_name=%3Cscript%3Ealert(%27XSS%27)%3C/script%3E&admin_username=%3Cscript%3Ealert(%27XSS%27)%3C/script%3E

Fix

# the check only works on mysql if the database is open
        $t_version_info = @$g_db->ServerInfo();
    } else {
-       print_test_result( BAD, true, 'Does administrative user have access to the database? ( ' . db_error_msg() . ' )' );
+       print_test_result(
+           BAD,
+           true,
+           'Does administrative user have access to the database? ( ' . string_attribute( db_error_msg() ) . ' )'
+       );
        $t_version_info = null;
    }
    ?>
@@ -469,7 +473,11 @@ function print_test( $p_test_description, $p_result, $p_hard_fail = true, $p_mes
                print_test_result( GOOD );
            }
        } else {
-           print_test_result( BAD, false, 'Database user doesn\'t have access to the database ( ' . db_error_msg() . ' )' );
+           print_test_result(
+               BAD,
+               false,
+               'Database user doesn\'t have access to the database ( ' . string_attribute( db_error_msg() ) . ' )'
+           );
        }
        ?>
 </tr>
@@ -791,7 +799,11 @@ function print_test( $p_test_description, $p_result, $p_hard_fail = true, $p_mes

                if( !$t_rs ) {
                    $t_result = false;
-                   print_test_result( BAD, true, 'Does administrative user have access to create the database? ( ' . db_error_msg() . ' )' );
+                   print_test_result(
+                       BAD,
+                       true,
+                       'Does administrative user have access to create the database? ( ' . string_attribute( db_error_msg() ) . ' )'
+                   );
                    $t_install_state--; # db creation failed, allow user to re-enter user/password info
                } else {
                    print_test_result( GOOD );
@@ -814,9 +826,18 @@ function print_test( $p_test_description, $p_result, $p_hard_fail = true, $p_mes
                    }

                    if( $t_db_exists ) {
-                       print_test_result( BAD, false, 'Database already exists? ( ' . db_error_msg() . ' )' );
-                   } else {
-                       print_test_result( BAD, true, 'Does administrative user have access to create the database? ( ' . db_error_msg() . ' )' );
+                       print_test_result(
+                           BAD,
+                           false,
+                           'Database already exists? ( ' . string_attribute( db_error_msg() ) . ' )'
+                       );
+                   }
+                   else {
+                       print_test_result(
+                           BAD,
+                           true,
+                           'Does administrative user have access to create the database? ( ' . string_attribute( db_error_msg() ) . ' )'
+                       );
                        $t_install_state--; # db creation failed, allow user to re-enter user/password info
                    }
                }
@@ -847,7 +868,11 @@ function print_test( $p_test_description, $p_result, $p_hard_fail = true, $p_mes
        if( $t_result == true ) {
            print_test_result( GOOD );
        } else {
-           print_test_result( BAD, false, 'Database user doesn\'t have access to the database ( ' . db_error_msg() . ' )' );
+           print_test_result(
+               BAD,
+               false,
+               'Database user doesn\'t have access to the database ( ' . string_attribute( db_error_msg() ) . ' )'
+           );
        }
        $g_db->Close();
    ?>
@@ -1242,7 +1267,11 @@ function print_test( $p_test_description, $p_result, $p_hard_fail = true, $p_mes
    if( $t_result == true ) {
        print_test_result( GOOD );
    } else {
-       print_test_result( BAD, false, 'Database user does not have access to the database ( ' . db_error_msg() . ' )' );
+       print_test_result(
+           BAD,
+           false,
+           'Database user does not have access to the database ( ' . string_attribute( db_error_msg() ) . ' )'
+       );
    }

    if( $f_db_type == 'db2' ) {
@@ -1264,7 +1293,11 @@ function print_test( $p_test_description, $p_result, $p_hard_fail = true, $p_mes
    if( $t_result != false ) {
        print_test_result( GOOD );
    } else {
-       print_test_result( BAD, true, 'Database user does not have SELECT access to the database ( ' . db_error_msg() . ' )' );
+       print_test_result(
+           BAD,
+           true,
+           'Database user does not have SELECT access to the database ( ' . string_attribute( db_error_msg() ) . ' )'
+       );
    }
    ?>
 </tr>
@@ -1279,7 +1312,11 @@ function print_test( $p_test_description, $p_result, $p_hard_fail = true, $p_mes
    if( $t_result != false ) {
        print_test_result( GOOD );
    } else {
-       print_test_result( BAD, true, 'Database user does not have INSERT access to the database ( ' . db_error_msg() . ' )' );
+       print_test_result(
+           BAD,
+           true,
+           'Database user does not have INSERT access to the database ( ' . string_attribute( db_error_msg() ) . ' )'
+       );
    }
    ?>
 </tr>
@@ -1294,7 +1331,11 @@ function print_test( $p_test_description, $p_result, $p_hard_fail = true, $p_mes
    if( $t_result != false ) {
        print_test_result( GOOD );
    } else {
-       print_test_result( BAD, true, 'Database user does not have UPDATE access to the database ( ' . db_error_msg() . ' )' );
+       print_test_result(
+           BAD,
+           true,
+           'Database user does not have UPDATE access to the database ( ' . string_attribute( db_error_msg() ) . ' )'
+       );
    }
    ?>
 </tr>
@@ -1309,7 +1350,11 @@ function print_test( $p_test_description, $p_result, $p_hard_fail = true, $p_mes
    if( $t_result != false ) {
        print_test_result( GOOD );
    } else {
-       print_test_result( BAD, true, 'Database user does not have DELETE access to the database ( ' . db_error_msg() . ' )' );
+       print_test_result(
+           BAD,
+           true,
+           'Database user does not have DELETE access to the database ( ' . string_attribute( db_error_msg() ) . ' )'
+       );
    }
    ?>
 </tr>

소스코드에서 확인할 수 있듯이 db_error_msg()함수를 string_attribute()함수로 감싸는 형식으로 조치한 것을 볼 수 있다. POC Code에서 스크립트 구문 삽입 시 사용된 database_name, admin_username 파라미터의 값을 db_error_msg()함수가 그대로 사용자에게 반환하여 판단되며 스크립트 관련 특수 문자를 걸러내기 위해서 string_attribute()함수를 적용시킨 것으로 보인다.

이제 string_attribute()함수를 살펴보자. string_attribute()함수는 mantisbt/core/string_api.php 231번 줄에 위치하고 있다.

function string_attribute( $p_string ) {
    return string_html_specialchars( $p_string );
}

string_html_specialchars()함수를 호출하는 것 외에는 다른 내용이 없다. string_html_specialchars()함수는 동일한 파일의 908번 줄에 위치하고 있다.

function string_html_specialchars( $p_string ) {
    # Remove any invalid character from the string per XML 1.0 specification
    # http://www.w3.org/TR/2008/REC-xml-20081126/#NT-Char
    $p_string = preg_replace( '/[^\x9\xA\xD\x20-\x{D7FF}\x{E000}-\x{FFFD}\x{10000}-\x{10FFFF}]+/u', '', $p_string );
    # achumakov: @ added to avoid warning output in unsupported codepages
    # e.g. 8859-2, windows-1257, Korean, which are treated as 8859-1.
    # This is VERY important for Eastern European, Baltic and Korean languages
    return preg_replace( '/&amp;(#[0-9]+|[a-z]+);/i', '&$1;', @htmlspecialchars( $p_string, ENT_COMPAT, 'utf-8' ) );
}

Cross Site Scripting에 관련된 부분은 @htmlspecialchars( $p_string, ENT_COMPAT, 'utf-8' )이 부분으로 $p_string변수의 문자열 값 중 특수문자를 HTML 엔터티 값으로 변환하는 데, ENT_COMPAT을 설정하였으므로 따옴표 중 겹따옴표만 변환하며 문자셋은 utf-8을 사용하게 된다.

  • 치환 특수문자
    • ’&’ : &amp;
    • ‘”’ : &quot;
    • ”’ : &#039;
    • ‘<’ : &lt;
    • ‘>’ : &gt;

CVE-2017-12062

취약점이 발생된 곳은 manage_user_page.php 소스이다.

영향 버전: 2.1.0 through 2.5.1
조치 버전: 2.5.2, 2.6.0 (not yet released*)

POC

http://localhost/mantisbt/manage_user_page.php
?sort=username
&dir=desc
&save=1
&hideinactive=0
&showdisabled=0
&filter=ALL"><SVG ONLOAD=&#97&<a href="/bugs/view.php?id=108" title="[종료된 이슈] When adding a bugnote can we have at some other detail on the page." class="resolved">0000108</a>&#101&<a href="/bugs/view.php?id=114" title="[종료된 이슈] Make date format displayed configurable." class="resolved">0000114</a>&#116(1)><IMG SRC="

<svg onload=&#97&#108&#101&#114&#116(1)>

Fix

        <input type="hidden" name="sort" value="<?php echo $c_sort ?>" />
            <input type="hidden" name="dir" value="<?php echo $c_dir ?>" />
            <input type="hidden" name="save" value="1" />
-           <input type="hidden" name="filter" value="<?php echo $f_filter ?>" />
+           <input type="hidden" name="filter" value="<?php echo string_attribute( $f_filter ); ?>" />
            <label class="inline">
            <input type="checkbox" class="ace" name="hideinactive" value="<?php echo ON ?>" <?php check_checked( (int)$c_hide_inactive, ON ); ?> />
            <span class="lbl"> <?php echo lang_get( 'hide_inactive' ) ?></span>

조치된 방법은 CVE-2017-12061 취약점과 동일하다.

이 블로그의 인기 게시물

X-Frame-Options-Test

X-Frame-Options 테스트하기 X-Frame-Options 페이지 구성 시 삽입된 프레임의 출처를 검증하여 허용하지 않는 페이지 URL일 경우 해당 프레임을 포함하지 않는 확장 응답 헤더이다. 보안 목적으로 사용되는 확장 헤더로 아직 적용되지 않은 사이트들이 많지만 앞으로 점차 적용될 것으로 보인다. X-Frame OptionsDENY, SAMEORIGIN, ALLOW-FROM 옵션을 이용하여 세부 정책을 설정한다. 옵션 설명 DENY Frame 비허용 SAMEORIGIN 동일한 ORIGIN에 해당하는 Frame만 허용 ALLOW-FROM 지정된 ORIGIN에 해당하는 Frame만 허용 크롬 4.1 , IE 8 , 오페라 10.5 , 사파리 4.0 , 파이어폭스 3.6.9 이상에서는 DENY , SAMEORIGIN 이 적용되며, ALLOW-FROM 은 각 브라우저 마다 지원 현황이 다르다. https://developer.mozilla.org/ko/docs/Web/HTTP/Headers/X-Frame-Options 해당 확장헤더는 브라우저에서 처리하는 응답 헤더이므로 미지원 브라우저 사용 시 설정과 무관하게 페이지 내 포함된 모든 Frame을 출력한다. (검증 테스트: Opera 5.0.0) 테스트 코드 DENY <!DOCTYPE html> < html lang = "en" > < head > < meta http-equiv = "X-Frame-Options" content = "deny" /> < title > Deny option Test </ title > </ head > < bod

C-lang-vulnerabilities

C 언어 공통 취약점 해당 글은 CERN Computer Security의 Common vulnerabilities guide for C programmers 글을 참고하여 작성하였습니다. C언어에서 발생하는 대부분의 취약점은 버퍼 오버플로우와 문자열 처리 미흡과 관련되어 있다. 이는 segmentation fault를 유발하고 입력 값을 조작할 경우 임의 코드 실행으로 이어질 수 있다. 이에 대부분의 에러와 조치 방안을 살펴보자고 한다. gets stdio gets() 함수는 버퍼 길이를 검증하지 않아 사용 시 항상 취약성을 야기한다. Vulnerable Code #include<stdio.h> int main() { char username[ 8 ]; int allow = 0 ; printf ( "Enter your username, please: " ); gets(username); //악의적인 값 삽입 if (grantAccess(username)) { allow = 1 ; } if (allow != 0 ) { //username을 오버플로우하여 덮어씀 privilegeAction(); } return 0 ; } Mitigation fgets() 함수 사용 및 동적 메모리 할당 #include <stdio.h> #include <stdlib.h> #define LENGTH 8 int main () { char * username, *nlptr; int allow = 0 ; username = malloc (LENGTH * sizeof (*username)); if (!username) return EXIT_FAILURE; printf ( "Enter your username, please:

HTML/CSS를 활용하여 카카오톡 클론 만들기

시간을 내어 HTML과 CSS를 공부한 것은 대학생 때가 마지막이었던 것으로 기억한다. 그 동안 사이드 프로젝트로 진행했던 여러 아이디어들을 결국 서비스하지 못했던 결정적인 이유는 프론트 엔드 기술 부족이었다고 생각하고 우선 HTML과 CSS 학습을 진행하였다. 프론트 엔드 기술은 많은 발전을 거듭하여 예전에 비해 큰 복잡성을 가지게 되었다. 빠른 시간 안에 숙지하지 못한 기법들에 대해서 알아보고 구현하고자 하는 아이디어에 활용할 수 있을 정도로 진행해보고자 한다. 또한 보안적 관점에서 발생할 수 있는 프론트 엔드 위협에 대해 파악할 수 있는 좋은 밑거름이 되길 기대해본다. 우선적으로 진행한 카카오 톡 디자인 클론은 노마드 아카데미의 강의를 수강하며 진행하였고 결과는 아래의 링크에서 확인할 수 있다. 소스코드 저장소 구현된 웹 페이지