Erreur dans code client-serveur

Erreur dans code client-serveur - C - Programmation

Marsh Posté le 13-10-2005 à 09:01:04    

Bonjour je doit faire un petit programme de transmission de fichier en c par socket sous linux, et j'ai des erreur de compilation et je n'en trouve pas la source, pourriez vous m'aider.
 
Merci
 
 

Code :
  1. #include <stdio.h>
  2. #include <sys/types.h>
  3. #include <sys/socket.h>
  4. #include <netinet/in.h>
  5. #include <netdb.h>
  6. #include <fcntl.h>
  8. #define PORT 2806
  10. char choix()
  11. {
  12. char car;
  13. car = getchar();
  14. while(getchar() != '\n');
  15. return(car);
  16. }
  18. main(argc,argv)
  19. int argc;
  20. char *argv[];
  21. {
  22. struct hostent *host;
  23. struct sockaddr_in nom;
  24. char buff;
  25. int sd,acc;
  26. int fd;
  27. int lg;
  29. char nom_serveur[15];
  30. char car;
  31. car = ' ';
  34. /*obtent°socket*/
  36. sd=socket(AF_INET,SOCK_STREAM,0);
  38. if(sd == -1)
  39.  {
  40.  printf("appel a socket refuse\n" );
  41.  exit(0);
  42.  }
  44. bzero((char*)&nom, sizeof(nom));
  45. nom.sin_port=PORT;
  46. nom.sin_addr.s_addr = INADDR_ANY;
  47. nom.sin_family=AF_INET;
  50. /*BIND*/
  51. if(bind(sd,&nom,sizeof(nom)) ==-1)
  52.  {
  53.  printf("erreur sur BIND \n" );
  54.  exit(2);
  55.  }
  58. /*LISTEN*/
  59. listen(sd,1);
  61. /*msg*/
  62. if((gethostname(nom_serveur,sizeof(nom_serveur))) ==-1)
  63.  {
  64.  perror("1-HOST LOCAL INCONNU\n" );
  65.  exit(2);
  66.  }
  68. printf("serveur sur : %s \n", nom_serveur);
  71. /*accept*/
  73. lg = sizeof(nom);
  74. acc = accept(sd,&nom,&lg);
  75. if(acc<0)
  76.  {
  77.  printf("appel a accept refuse \n" );
  78.  exit(0);
  79.  }
  81. /*lecture du msg*/
  82. fd = fopen("/home/timat/tmp/reponse.txt","w" );
  83. /*while(!feof(fd))
  84. {*/
  85. recv(acc,&buff,sizeof(buff),0);
  86. fwrite(&buff,sizeof(buff),1,fd);
  87. /*}*/
  89. printf("fichier reçu" );
  91. fclose(fd);
  92. close(acc);
  93. }

Reply

Marsh Posté le 13-10-2005 à 09:01:04   

Reply

Marsh Posté le 13-10-2005 à 09:46:22    

en fin de compte il y a des warning et erreur dans mon client aussi:
 
 

Code :
  1. #include <stdio.h>
  2. #include <sys/types.h>
  3. #include <sys/socket.h>
  4. #include <netinet/in.h>
  5. #include <netdb.h>
  6. #include <fcntl.h>
  8. #define PORT 2806
  10. char choix()
  11. {
  12. char car;
  13. car = getchar();
  14. while(getchar() != '\n');
  15. return(car);
  16. }
  18. main(argc,argv)
  19. int argc;
  20. char *argv[];
  21. {
  22. struct hostent *host;
  23. struct sockaddr_in nom;
  24. char buff;
  25. int sd;
  26. int fd;
  27. char car;
  29. /*Vérif nb paramètre*/
  30. if(argc!=3){
  31. perror("Utilisation: cleint sun_x message\n" );
  32. exit(3);
  33. }
  35. /*obtent°socket*/
  36. if((sd=socket(AF_INET,SOCK_STREAM,0)) == -1)
  37. {
  38. printf("appel a socket refuse\n" );
  39. exit(0);
  40. }
  43. /*recherche adresse internet du serveur*/
  44. if((host = gethostbyname(argv[1])) == NULL)
  45. {
  46. printf("site serveur non trouve\n" );
  47. exit(1);
  48. }
  50. printf("le nom retourne de gethostbyname = %s\n",host->h_name);
  51. /*printf("%s\n\n",host->h_aliases);*/
  53. /*preparation socket distant*/
  55. bzero((char*)&nom, sizeof(nom));
  56. bcopy(host->h_addr, &nom.sin_addr, host->h_length);
  57. nom.sin_port=PORT;
  58. nom.sin_family=AF_INET;
  60. /*connection serveur*/
  61. if((connect(sd,&nom,sizeof(nom))) ==-1)
  62. {
  63. printf("appel a connect refuse\n" );
  64. exit(0);
  65. }
  67. /*envoie msg*/
  69. fd = fopen(argv[2],"r" );
  70. while(!feof(fd))
  71. {
  72. fread(&buff,sizeof(buff),1,fd);
  73. send(sd,&buff,sizeof(car),0);
  74. /*printf("%s",&car);*/
  75. }
  77. printf("fichier envoyé\n" );
  78. fclose(fd);
  79. close(nom);
  80. }

Reply

Marsh Posté le 13-10-2005 à 10:35:56    

Ouais bon, tu aurais pu nous recopier tes erreurs, hein, ça aurait été plus sympa que de nous laisser compiler nous-même. :o
 
Ensuite, ça :

Code :
  1. main(argc,argv)
  2. int argc;
  3. char *argv[];
  4. {


ben c'est ignoble, on ne déclare plus une fonction comme ça depuis... hmm... 15 ans ?
 
Bref, remplace par :

int main(int argc, char **argv)


 
 
Quant aux erreurs elles-mêmes :

test.c:19: warning: return-type defaults to `int'
test.c: In function `main':
test.c:44: warning: implicit declaration of function `bzero'
test.c:51: warning: passing arg 2 of `bind' from incompatible pointer type
test.c:62: warning: implicit declaration of function `gethostname'
test.c:74: warning: passing arg 2 of `accept' from incompatible pointer type
test.c:82: warning: assignment makes integer from pointer without a cast
test.c:86: warning: passing arg 4 of `fwrite' makes pointer from integer without a cast
test.c:91: warning: passing arg 1 of `fclose' makes pointer from integer without a cast
test.c:92: warning: implicit declaration of function `close'
test.c:22: warning: unused variable `host'
test.c:19: warning: unused parameter `argc'
test.c:20: warning: unused parameter `argv'
test.c:93: warning: control reaches end of non-void function


 
19 : main() doit explicitement retourner un int
44 : bzero() est défini dans strings.h, manque le #include
51 : bind() attend un struct sockaddr *, pas un struct sockaddr_in *
62 : gethostname() est défini dans unistd.h, il manque le...
 
... euuuh. Dis, t'aurais pas codé au hasard, des fois ?
La majeure partie des tes erreurs c'est à cause de typage incorrect ou de fonctions dont tu as oublié le #include.
 
Vu que tu utilises des fonctions POSIX j'imagine que tu travailles sur un unix-like, alors utilise "man" sur les fonctions que tu ne connais pas.

Message cité 1 fois
Reply

Marsh Posté le 13-10-2005 à 10:42:14    

En plus de tes problèmes de warning dus aux include manquants, tu auras un gros soucis avec le "while (!feof(fd))" parce que cette instruction ne fait pas ce que tu crois.
feof ne sert pas à détecter la fin d'un fichier dasn une lecture, elle sert juste à te confirmer si, une fois la lecture terminée, cette lecture a été finie à cause de la fin du fichier.
 
Pour faire une boucle de lecture sur un fichier, on utilise juste le fait que la fonction de lecture elle-même renvoie une valeur particulière lorsque la fin de fichier est atteinte !!!
 
Exemples:
while ((carac=fgetc(...)) != EOF)
while (fgets(...) != NULL)
while (fread(...) > 0)
 
Remarque perso: Je conseillerais plutôt les fonctions "memset" et "memcpy" (standard) plutôt que "bzero" et "bcopy"
 
Ici un cours sur les sockets sous Unix : http://fr.lang.free.fr/cours/SocketCsyst_v1.0.pdf
 

Elmoricq a écrit :


Bref, remplace par :

int main(int argc, char **argv)



 
Moi j'écrirais plutôt

int main(int argc, char *argv[])


 


Message édité par Sve@r le 13-10-2005 à 11:00:38
Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed