기본 콘텐츠로 건너뛰기

typos

typos

오탈자

도전 1

whlie(*src && left)
{
	*dst++=*src++;
	
	if (left = 0)
		die("badlen");

	left--;
}

if 조건에 관계 연산자 == 대신 대입 연산자 =가 설정되어 있다. 이로 인해 left이 0으로 설정되어도 while 구문을 벗어나지 못하고 left 값을 감소시키며 반복 수행한다.

수정 코드


if (left == 0)

도전 2

int f;
f=get_security_flags(username);

if(f = FLAG_AUTHENTICATED)
{
	return LOGIN_OK;
}

return LOGIN_FAILED;

if 조건에 관계 연산자 == 대신 대입 연산자 =가 설정되어 있어 get_security_flags 결과와 무관하게 로그인 성공을 반환하고 있다.

수정 코드

if(f == FLAG_AUTHENTICATED)

도전 3

for (i==5; src[i] && i<10; i++)
{
	dst[i-5]=src[i];
}

for 조건에 대입 연산자 = 대신에 관계 연산자 ==가 설정되어 있어 i가 10보다 작은 값일 때, 버퍼를 벗어날 위험이 있다.

수정 코드

for (i=5; src[i] && i<10; i++)

도전 4

if (get_string(src) && check_for_overflow(src) & copy_string(dst, src))
	printf("string safely copied\n");

논리 연산자 && 대신에 비트 연산자 &가 조건으로 설정되어 있어 문자열 복사 작업 실패 시에도 성공 문구를 출력하고 있다.

수정 코드

if (get_string(src) && check_for_overflow(src) && copy_string(dst, src))

도전 5

if (len > 0 && len <= sizeof(dst));
	memcpy(dst, src, len);

if; 이 존재하여 길이 검사 결과와 무관하게 memcpy를 실행한다.

수정 코드

if (len > 0 && len <= sizeof(dst))

도전 6

char buf[040];

snprintf(buf, 40, "%s", userinput);

숫자 앞에 0을 접두사로 사용하면 8진수로 인식되므로 의도한 것보다 작은 배열 공간 (buf[32])을 할당하게 된다.

수정 코드

char buf[40];

도전 7

if (len < 0 || len > sizeof(dst)) /* 길이 검사
	die("bad length!");

/* length ok */

memcpy(dst, src, len);

if 뒤 주석을 닫지 않아 길이 검사를 실패할 경우에만 memcpy를 실행한다.

수정 코드

if (len < 0 || len > sizeof(dst)) /* 길이 검사 */

도전 8

if (len > 0 && len <= sizeof(dst))
	copiedflag = 1;
	memcpy(dst, src, len);

if (!copiedflag)
	die("didn't copy");

괄호가 생략되어 있어 길이 검사 결과와 무관하게 memcpy가 실행된다.

수정 코드

if (len > 0 && len <= sizeof(dst))
{
	copiedflag = 1;
	memcpy(dst, src, len);
}

도전 9

if (!strncmp(src, "magicword", 9))
	// report_magic(1);

if (len < 0 || len > sizeof(dst))
	assert("bad length!");

/* length ok */
memcpy(dst, src, len);

if에서 ;를 생략했기 때문에 memcpy는 언제나 실행된다.

수정 코드

if (!strncmp(src, "magicword", 9))
	// report_magic(1);
	;

도전 10

l = msg_hdr.msg_len;
frag_off = msg_hdr.frag_off;
frag_len = msg_hdr.frag_len;
...
if (frag_len > (unsigned long)max)
{
	al=SSL_AD_ILLEGAL_PARAMETER;
	SSLerr(SSL_F_DTLSl_GET_MESSAGE_FRAGMENT, SSL_R_EXCESSIVE_MESSAGE_SIZE);
	goto f_err;
}

if (frag_len + s->init_num > (INT_MAX - DTLSl_HM_HEADER_LENGTH))
{
	al=SSL_AD_ILLEGAL_PARAMETER;
	SSLerr(SSL_F_DTLSl_GET_MESSAGE_FRAGMENT, SSL_R_EXCESSIVE_MESSAGE_SIZE);
	goto f_err;
}

if (frag_len & !BUF_MEM_grow_clean(s->init_buf, (int)frag_len + DTLSl_HM_HEADER_LENGTH + s->init_num))
{
	SSLerr(SSL_F_DTLSl_GET_MESSAGE_FRAGMENT, ERR_R_BUF_LIB);
	goto err;
}

if (s->dl->r_msg_hdr.frag_off == 0)
{
	s->s3->tmp.message_type = msg_hdr.type;
	s->dl->r_msg_hdr.type = msg_hdr.type;
	s->dl->r_msg_hdr.msg_len = 1;
	/* s->dl->r_msg_hdr.seq = seq_num; */
}

/* XDTLS: ressurect this when restart is in place */
s->state=stn;

/* next state (stn) */
p = (unsigned char *))s->init_buf->data;

if (frag_len > 0)
{
	i=s->method->ssl_read_bytes(s, SSL3_RT_HANDSHAKE, &p[s->init_num], frag_len, 0);
	/* XDTLS: fix this-message fragments cannot span multiple packets */

	if (i <= 0)
	{
		s->rwstate=SSL_READING;
		*ok = 0;
		return i;
	}
}
else
	i = 0;

길이 검사 중 하나에서 논리 연산자 &&를 사용해야 하는 곳에 비트 연산자 &를 사용하는 실수를 저질렀다.

수정 코드

if (frag_len && !BUF_MEM_grow_clean(s->init_buf, (int)frag_len + DTLSl_HM_HEADER_LENGTH + s->init_num))

이 블로그의 인기 게시물

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 학습을 진행하였다. 프론트 엔드 기술은 많은 발전을 거듭하여 예전에 비해 큰 복잡성을 가지게 되었다. 빠른 시간 안에 숙지하지 못한 기법들에 대해서 알아보고 구현하고자 하는 아이디어에 활용할 수 있을 정도로 진행해보고자 한다. 또한 보안적 관점에서 발생할 수 있는 프론트 엔드 위협에 대해 파악할 수 있는 좋은 밑거름이 되길 기대해본다. 우선적으로 진행한 카카오 톡 디자인 클론은 노마드 아카데미의 강의를 수강하며 진행하였고 결과는 아래의 링크에서 확인할 수 있다. 소스코드 저장소 구현된 웹 페이지