recv WAITALL 옵션에 대한 문제...

nakorori의 이미지

기본적인 흐름은

클라이언트가 서버에 받고싶은 데이터 주소와 데이터 길이를 전달해주면

서버는 그 데이터 주소에서 데이터 길이를 클라이언트에게 전송해 줍니다.

MSG_WAITALL 옵션을 쓰지 않는것이 좋은것 같아.

처음에 recvAll 이라는 함수를 만들어 사용했습니다.

int recvAll(int sock, char* data, int size){
    int n = 0;
    for(;;){
         if( (n=recv(sock,data,size,MSG_NOSIGNAL))<size){
            if(n<0){
                  perror("recv failed ");
                  return -1;
            }else if(n == 0){
                  break;
            }
            data += n;
            size -= n;
         }
         if(size == n)
            break;
    }
}

1번 방법
if( recvAll(sock,(char*)data,len) < 0){
    fprintf(stderr,"recv error\n");
    shutdown(sock,SHUT_RDWR);
    close(sock);
}

2번 방법
if( recv(sock,(char*)data,len,MSG_WAITALL) < 0){
    perror("recv error\n");
    shutdown(sock,SHUT_RDWR);
    close(sock);
}

이후 재접속

1번 방법을 사용할 경우에 가끔씩 잘못된 데이터를 가져옵니다.
같이 일하는 동생은 recv 버퍼에 올바른 데이터가 아닌경우에도 recv가 읽어오는 경우가 있다고 하던데...
도데체 return 값을 믿을수가 없어지니...;;

2번방법을 사용할 경우에는 잘못된 데이터를 받아온적이 없습니다.

제가 구현한 recvAll이 잘못된 걸까요??

훗의 이미지

이런 질문하기전에 검색부터좀 하시죠?

쯧쯧

kslee80의 이미지

방식 자체는 맞습니다만, 로직상에 허점이 하나 있네요.

recv() 함수의 첫 실행 결과가 받고자 하는 데이터의 절반만을 받아왔을때,
size -= n; 라인의 결과로 인해 size == n 이 되고
바로 아래에 있는 if 문에 의해서 함수가 종료되는 허점이 있네요.