problème de programmation Socket en C

Question

de la pièce ci-dessous du code, pourquoi je reçois lecture Prise pour réponse

int Read(int sock, char *p, int size) 
{ 
    int remain, read=0; 

    remain = size; 
    while (remain > 0) { 

     if ((read = recv(sock, p, remain, 0)) < 0) { 

      /* Error */ 
      return(read); 
     } else if (read == 0 || *p == 0x0a) { 

      /* EOF */ 
      break; 
     } 
     remain -= read; 
     p += read; 
    } 

    return(size - remain); 
} 



while (!done) 
{ 
    printf("***Reading Socket for response***"); 
    rsplen= Read(myVsHandle.sock,(char *)encXMLResponse,MAX_RSP_LEN); 
    if (rsplen < 0) 
    { 
     printf("Internal Communication Error"); 
     return -1; 
    } 
    else if (rsplen >0) 
     printf("Revieved response"); 
     done++; 
     return 0; 
    else if (rsplen == 0) 
    { 
     printf("Reading socket"); 
    } 

Answer 1

Vous attendez MAX_RSP_LEN octets à lire - y at-il beaucoup d'octets à lire? Peut-être que votre processus est bloqué dans un blocage read().

En outre, selon le type de socket vous recv() ING à partir, il n'y a aucune garantie sur la quantité de données que vous allez lire, donc en particulier la recherche d'une valeur 0x0a peut ne pas fonctionner.

Answer 2

Ce:

if ((read = recv(sock, p, remain, 0)) < 0) { 

devrait-elle être

if ((read = recv(sock, p, remain, 0)) > 0) { // Greater then 0, because recv returns the number of bytes received if successful, if it fails -1. 

Answer 3

Votre problème pourrait être que vous ne terminez pas votre sortie avec un saut de ligne. Essayez de terminer vos sorties avec une nouvelle ligne (\n). stdout est mis en mémoire tampon, de sorte que vous ne voyiez peut-être rien pendant longtemps si vous ne produisez pas de retour à la ligne.

Une autre possibilité est que vous ne reveniez pas de Read() sauf si vous avez lu le nombre d'octets spécifié. En fonction de la valeur de MAX_RSP_LEN et de la quantité de données disponibles, Read() peut attendre indéfiniment.

De même, votre test: *p == 0x0a semble suspect. Que testez-vous ici?

Modifier: Il y a un autre "bug":

else if (rsplen >0) 
    printf("Revieved response"); 
    done++; 
    return 0; 
else... 

Il vous manque des accolades. Dans le formulaire actuel, le code ne doit pas compiler. S'il vous plaît poster le code réel.

Answer 4

vous manque accolades autour du:

else if(rsplen > 0) 
    ... statements 

Il devrait être:

... 
}else if (rsplen >0){ 
    printf("Revieved response"); 
    done++; 
    return 0; 
} ...