Broken Code (window31)

넥슨같은 좋은 회사도 일정에 쫒기나요? ㅎㅎ

저희 팀 같은 경우는 일주일에 한두번 꼴로 코드 리뷰를 합니다. 처음에는 아주 죽을 맛이었는데, 리뷰를 할때마다 버그를 잡아내는 선배들을 보니 정신을 바짝 차려야겠다는 생각이 들더라구요. 뛰어난 팀장이 외세의 압력에 잘 버텨주는 편이라 릴리즈 직전에는 무조건 코드 리뷰를 1번 이상 합니다. ㅎㅎ

문제는 그래도 버그가 나온다는거... ㅠㅠ)-b
좋은 글 잘 보고 갑니다. ^^

쓰레기 코드 만드는 일은 참으로 곤욕이더군요.

오히려 정상 코드보다 더 어려운 것이, 정말 순수하게 아무일도 안하고, 복잡하면서, 버그도 없으며, 소요되는 시간 조차 거의 없어야 한다는 것 ㅠㅠ

또한, 복잡하기 위해서 반드시 디스어셈블리된 코드로 일일히 확인해야 한다는것...

코드리뷰는 예전에 회사에서 워낙 진기한 버그들이 많은 프로젝트들을 수정하느라 몇번 신청해본적(코드 리뷰를 하고 싶으니 시간을 좀 할애해주세요) 있습니다.

다소 떨떠름한 반응으로, 잘된다는 보장도 없는데 왜하냐 라는 느낌이더라구요. 그 이후로 아예 다음부터는 코드 리뷰라고 이야기하지 않고 그냥 디버깅하는 시간을 달라고 이야기를 합니다. 아니면 처음부터 다른 일정을 조금씩 늘려서 짜거나요.

오히려 아예 일정에도 없으니 부담도 없고 좋더라구요. 덜덜...

쓰레기코드 잘 만드는 사람.. 환영!!

쓰레기 코드를 만들 때는 일반적으로 잘 알려진 수법 말고 좀 어려운 instruction을 사용해야 혼동하기 쉬울것 같네요. (제 생각입니다)

아래와 같이 되어 있으면 리버서는 pushad~popad가 Dummy Code 영역임을 눈치채기가 쉽지만...
pushad
xor eax, eax
inc ebx
popad

아래와 같이 약간 꼬아놔야할것 같네요...
push esi
push ebx
lea esi, dword ptr ss:[ebx]
clc
xor [esp], esi ; [esp]=0
jc $+1200h
add [esp], 12345678h
pop esi
pop esi

비밀댓글입니다

ㅋㅋ pushad 자체를 본순간 스크롤바를 내려보게 되죠,
popad가 있는지 보기 위해 가운데에 뭐가 있는진 보통 그다음에
보지 않나요?ㅎㅎ 저만 그러는가...ㅠ0ㅠ

그러니까 아까 심심해서 만든 코드인데,

#include<stdio.h>
#include<memory.h>
#include<malloc.h>
#include<conio.h>
#include<windows.h>
#define DEEP 1000
char bs[12]={ 0xb8,0x00,0x00,0x00,0x00,//mov eax, ?
0xbb,0x00,0x00,0x00,0x00,//mov ebx, ?
0xff,0xe3};//jmp ?
char tprocs[]={0x90,0x90,0x60,0x8b,0xd8,0x83,0xe3,0x00,0x33,0xdb,0x33,0xc0,0x60,0x61,0x8b,0xc2,0x8b,0xd1,0x8b,0xd9,0x2b,0xd3,0x61,0xff,0xe0};
/*void _declspec(naked) tproc(){ _asm{ pushad; mov ebx,eax; and ebx,0x00000000; xor ebx,ebx; xor eax,eax; pushad; popad; mov eax,edx; mov edx,ecx; mov ebx,ecx; sub edx,ebx; popad; jmp eax; }}*/
int _stdcall tp(char *s){
printf(s);
return (int)s;
}
int makecode(void *func,char *arguments,int argubytes){
int to=(int)func;
char rs[DEEP][12];
char *tproc=(char*)malloc(sizeof(tprocs));
int i,j;
int rsp,tpp;
int old=0;
memcpy(tproc,tprocs,sizeof(tprocs));
tpp=(int)tproc;
srand(GetTickCount());
for(i=1;i<DEEP;i++){
if(i%((rand()%10)+1)==0){
for(j=0;j<12;j++){
rs[old][j]=bs[j];
}
rsp=((int)rs[i]);
memcpy(rs[old]+1,&rsp,4);
memcpy(rs[old]+1+5,&tpp,4);
old=i;
}else{
for(j=0;j<12;j++){
rs[i][j]=rand()%0x100;
}
}
}
for(i=0;i<12;i++){
rs[old][i]=bs[i];
}
memcpy(rs[old]+1+5,&tpp,4);
tpp=(int)to;
memcpy(rs[old]+1,&tpp,4);
rsp=((int)(&rs[0][0]));
unsigned int temp;
for(i=0;i<argubytes;i+=4){
temp=0;
temp=(temp<<8) | (arguments[i+3]&0xff);
temp=(temp<<8) | (arguments[i+2]&0xff);
temp=(temp<<8) | (arguments[i+1]&0xff);
temp=(temp<<8) | (arguments[i+0]&0xff);
_asm{
push temp;
}
}
int retval;
_asm{
call rsp;
mov retval,eax;
}
free(tproc);
return retval;
}
void main(){
while(1){
printf("%c",makecode(getch,0,0));
}
}

이런 코드 말이겠죠? 사실 정리하면
#include<stdio.h>
#include<conio.h>
void main(){
while(1){
printf("%c",getch());
}
}
이라는...

원래는 패커 만들다가 귀찮아서 버렸다는...