구조체를 char[128] temp; 로 선언된 배열에 복사할때
현재 소켓을 이용한 파일 전송 프로그램을 짜다가
5시간째 구조체를 소켓으로 보내는거에 대해서 삽질하다가 하도하도 안되서 질문 올립니다 ㅠㅠ
선언된 구조체는
struct ftp{
int type;
char prefix[BUF];
char data[BUF];
};
struct ftp fn, *fnp;
이렇게 선언되어 있구요,
문제가 뭐냐면
현재 prefix 배열에는 "filename_"이라는 문자열이,
data배열에는 scanf()를 이용하여 문자열을 입력받았습니다.
type에는 그냥 =1 해서 숫자 하나 집어넣었구요
memset(temp, 0x00, BUF);
size = sizeof(struct ftp)-sizeof(fn.prefix)-sizeof(fn.data)+strlen(fn.prefix)+strlen(fn.data);
memcpy(temp, (void *)&fn,size);
우선 memset으로 temp 배열(char 형 배열입니다, BUF는 걍 128) 초기화 해주고
size를 계산한후 원하는 크기 만큼만 temp 배열에 복사를 했습니다
그담에 값을 확인해볼려고
fnp = (struct ftp*)temp;
해봤드니 fnp->prefix에는 "filename_"이라는 문자열이 제대로 들어가있는데
type 이랑 data에는 아무것도 안들어가 있는겁니다;;
제가 무슨 한과정을 빼먹은건가요 아니면 단순한 오류인걸까요;;
알수가 없네요 ㅠㅠ 이것때메 5시간을 허비 했습니다
해결책보다는 왜 그런지가 더 궁금해지네요 ㅠㅠ
size = sizeof(struct
size = sizeof(struct ftp)-sizeof(fn.prefix)-sizeof(fn.data)+strlen(fn.prefix)+strlen(fn.data);
이게 몇이 나오나요? 제가 생각했을 땐 sizeof(ftp)보다는 적을거 같습니다.
그리고 이게 128보다(BUF)보다 작다면 모를까..
128을 넘어서면 memcpy(temp, (void *)&fn,size); 코드는 잘못된 코드입니다.
암튼 temp는 128바이트이기 때문에 결과가 어떻게 나오든 오류가 있는 코드입니다.
이것을 수정 후에 다시 확인해보세요
제 생각에는 memcpy(temp, (void *)&fn,size); 코드가 버퍼 오버플로가 나서
원래 fn.type을 0으로 덮어 쓴게 아닌가 하네요..
변수를 fn, temp순으로 선언했고 스택은 아래로 자란다면(리눅스도 그럴겁니다.. 아마) 충분히 가능성이 있습니다.
-------------------------------------------------------------------------------
It's better to appear stupid and ask question than to be silent and remain stupid.
-------------------------------------------------------------------------------
It's better to appear stupid and ask question than to be silent and remain stupid.
이상해요.
좀 이상한 코드군요.
원하시는게 불분명한 듯 합니다.
분명 원하시는게
//memcpy(temp, &fn, sizeof(struct ftp));
*(struct ftp*)temp = fn;
이거 아니면,
*(int*)(temp+offsetof(struct ftp, type)) = fn.type;
strcpy(temp+offsetof(struct ftp, prefix), fn.prefix);
strcpy(temp+offsetof(struct ftp, prefix)+strlen(fn.prefix)+1, fn.data);
둘 중 하나겠군요.
댓글 달기