programme pour recupérer des mails

programme pour recupérer des mails - C - Programmation

Marsh Posté le 05-06-2007 à 18:52:13    

Bonjour, je voulais faire un programme qui me permettrais de récupérer tous mes mails et de les stocker dans un dossier.
Le problème c'est qu'il plante avec "certain mail" sans que je puisse savoir pourquoi...
exemple :
j'ai 100 mails dans ma boites il récupère puis plante au 10eme.
je recommence cette fois en commencer par le 20eme et je peux récupérer tous le reste.
si je supprime le mail qui me plante je récupère tous sans problème...
 
procédé :
- je récupère la liste pour avoir le nombre de mails
- je fais une boucle pour récupérer les mails et les stocks dans des fichiers séparer
 
NB: il ouvre un fichier connection.txt ou il y a les infos d'un compte mail d'un server (sans SSL ou autre) :
smtp.fournisseur.com
compte
pass
port
 
donc j'aimerais savoir si le problème vous arrive aussi, et si vous avez des conseils. =)
 
http://dl.free.fr/pqTkKEKO/main.txt

Code :
  1. #ifdef WIN32
  3. #include <winsock2.h>
  4. #pragma comment(lib, "ws2_32.lib" )
  6. #elif defined (linux)
  8. #include <sys/types.h>
  9. #include <sys/socket.h>
  10. #include <netinet/in.h>
  11. #include <arpa/inet.h>
  12. #include <unistd.h> /* close */
  13. #include <netdb.h> /* gethostbyname */
  14. #define INVALID_SOCKET -1
  15. #define SOCKET_ERROR -1
  16. #define closesocket(s) close(s)
  17. typedef int SOCKET;
  18. typedef struct sockaddr_in SOCKADDR_IN;
  19. typedef struct sockaddr SOCKADDR;
  20. typedef struct in_addr IN_ADDR;
  22. #else
  24. #error not defined for this platform
  26. #endif
  28. #define USER        "USER"
  29. #define PASS  "PASS"
  30. #define PASSWORD "2t2k9q"
  31. #define LIST  "LIST"
  32. #define RETR        "RETR"
  33. #define END_DATA    "."
  34. #define QUIT        "QUIT"
  35. #define PORT        110
  36. #define BUF_SIZE    1024
  37. #define RECV_SIZE   32768
  38. #define CRLF        "\r\n"
  40. #include <stdio.h>
  41. #include <stdlib.h>
  43. int _server_response(SOCKET sock, char *buffer);
  44. int send_server(SOCKET sock, char *command);
  45. int _pop_retrieve_list();
  47. int main()
  48. {
  49. int a=0;
  51. a = _pop_retrieve_list();
  53. if(a==0)
  54.  printf("failed\n" );
  55. else
  56.  printf("succeeded\n" );
  59. }
  62. int _server_response(SOCKET sock, char *buffer)
  63. {
  65. if(recv(sock, buffer, BUF_SIZE, 0) < 0)
  66. {
  67.         perror("recv()" );
  68.  return 0;
  69.     }
  70.  strtok(buffer, " \r\n" );
  72. if (strcmp(buffer,"+OK" ) != 0)
  73. {
  74.  printf("Connexion failed\n" );
  75.  return 0;
  76. }
  81. return 1;
  82. }
  84. int send_server(SOCKET sock, char *command)
  85. {
  86.     if(send(sock, command, strlen(command), 0) < 0)
  87. {
  88.  perror("send()" );
  89.  return 0;
  90. }
  91. return 1;
  92. }
  95. int _pop_retrieve_list()
  96. {
  97. SOCKET sock;
  98. SOCKADDR_IN sin;
  99.     FILE *fichier = NULL;
  100. struct hostent *hostinfo;
  101. char command[BUF_SIZE];
  102. char hostname[128];
  103. char account[128];
  104. char pass[128];
  105. char port[5];
  106. long port_long;
  107. char list[RECV_SIZE];
  108. char text[128];
  109. char text_dump[128];
  110. char* buffer = (char*)malloc(BUF_SIZE);
  111. char* temp = (char*)malloc(BUF_SIZE);
  112. int check = 0;
  113. int nombre_mail=0;
  114. int i;
  117. #ifdef WIN32
  118.  WSADATA wsa;
  119.  int err = WSAStartup(MAKEWORD(2, 2), &wsa);
  120.  if(err < 0)
  121.  {
  122.   printf("WSAStartup failed !\n" );
  123.   return 0;
  124.  }
  125. #endif
  127. sock = socket(AF_INET, SOCK_STREAM, 0);
  128.     if (sock == INVALID_SOCKET)
  129.     {
  130.         perror("socket()" );
  131.  return 0;
  132.     }
  136. fichier = fopen("connection.txt", "r" );
  137. if (fichier == NULL)
  138. {
  139.  perror("fopen()" );
  140.  return 0;
  141. }
  142. fgets(hostname, 128, fichier);
  143. fgets(account, 128, fichier);
  144. fgets(pass, 128, fichier);
  145. fgets(port, 128, fichier);
  146. fclose(fichier);
  147. strtok(hostname, "\n" );
  148. strtok(account, "\n" );
  149. strtok(pass, "\n" );
  150. strtok(port, "\n" );
  151. port_long = strtol(port, NULL, NULL);
  153. hostinfo = gethostbyname(hostname);
  154.     if (hostinfo == NULL)
  155.     {
  156.         printf(stderr, "Unknown host %s.\n", BUF_SIZE);
  157.  return 0;
  158.     }
  161. sin.sin_addr = *(IN_ADDR *) hostinfo->h_addr;
  162.     sin.sin_port = htons(port_long);
  163.     sin.sin_family = PF_INET;
  164. if(connect(sock,(SOCKADDR *) &sin, sizeof(SOCKADDR)) == SOCKET_ERROR)
  165.     {
  166.         perror("connect()" );
  167.  return 0;
  168.     }
  170. //to check the reception
  171. check = _server_response(sock, buffer);
  172.  if(check == 0)
  173.   return 0;
  175. // send user account
  176. sprintf(command, "USER %s%s", account, CRLF);
  177. check = send_server(sock, command);
  178.  if(check == 0)
  179.   return 0;
  180. check = _server_response(sock, buffer);
  181.  if(check == 0)
  182.   return 0;
  184. // send user pass
  185. sprintf(command, "PASS %s%s", pass, CRLF);
  186. check = send_server(sock, command);
  187.  if(check == 0)
  188.   return 0;
  189. check = _server_response(sock, buffer);
  190.  if(check == 0)
  191.   return 0;
  193. // send LIST command
  194. sprintf(command, "LIST%s",  CRLF);
  195. check = send_server(sock, command);
  196.  if(check == 0)
  197.   return 0;
  200. // retrieve LIST response
  201. sprintf(text, "%s@%s/list.txt", account, hostname);
  202. fichier = fopen(text, "w+" );
  203. if(fichier==NULL)// to create a new list file in a folder account@provider
  204. {
  205.  sprintf(text, "mkdir %s@%s", account, hostname);
  206.  system(text);
  207.  sprintf(text, "%s@%s/list.txt", account, hostname);
  208.  fichier = fopen(text, "w+" );
  209. }
  210. fclose(fichier);
  211. fichier = fopen(text, "a" );
  212. do
  213. {
  214.  memset (list,0,RECV_SIZE);
  215.  if(recv(sock, list, RECV_SIZE, 0) < 0)
  216.  {
  217.   perror("recv()" );
  218.   return 0;
  219.  }
  220.   fprintf(fichier, list);
  221. }while(strstr(list, "\r\n.\r\n" )==0);
  222. fclose(fichier);
  224. // retrive the number of mail from the mail list
  225. fichier = fopen(text, "r" );
  226. if(fichier == NULL)
  227. {
  228.  printf("error" );
  229. }
  230. else
  231. {
  232.  fgets(text_dump, 128, fichier);
  233.  do
  234.  {
  235.   nombre_mail++;
  236.  }while(fgets(text_dump, 128, fichier)!=NULL);
  237.  fclose(fichier);
  238.  nombre_mail = nombre_mail-1;
  239. }
  243. for( i=1 ; i<nombre_mail ; i++)
  244. {
  245.  // send RETR command
  246.  sprintf(command, "RETR %ld%s", i, CRLF);
  247.  check = send_server(sock, command);
  248.   if(check == 0)
  249.    return 0;
  252.  // retrieve RETR response
  253.  sprintf(text, "%s@%s/mail %3ld.txt", account, hostname, i);
  254.  fichier = fopen(text, "w+" );
  255.  fclose(fichier);
  256.  fichier = fopen(text, "a" );
  257.  strcpy(list, "" );
  258.  printf(list);
  259.  do
  260.  {
  261.     memset (list,0,16384);
  262.   if(recv(sock, list, 16384, 0) < 0)
  263.   {
  264.    perror("recv()" );
  265.    return 0;
  266.   }
  267.    fprintf(fichier, list);
  268.  }while(strstr(list, "\r\n.\r\n" )==0);
  270.   fclose(fichier);
  272.   printf("%ld/%ld\n", i, nombre_mail-1);
  273. }
  277. return 1;
  280. }


Reply

Marsh Posté le 05-06-2007 à 18:52:13   

Reply

Marsh Posté le 05-06-2007 à 22:53:17    

bon âpres moult tentative je comprend toujours pas pourquoi ca plante, mais apparemment c'est fprintf qui fait planté...
je l'ai remplacé par fputs et ça à l'air de mieux fonctionner.
faudrait qu'on m'explique pourquoi quand même, parce que je ne vois vraiment pas pourquoi =/
ensuite je pourrais remettre un code un peu plus propre si jamais ca interesse...

Reply

Marsh Posté le 06-06-2007 à 09:00:16    

Ancien code :

Code :
  1. fprintf(fichier, list);


Ce code a un problème.
fprintf() a besoin (comme printf, sprintf etc...) d'au moins 3 paramètres. Le plantage aléatoire vient du fait que si "list" contient
la chaîne "%s" par exemple, alors le programme (fonction frprintf()) va chercher le pointeur associé sur la pile, ce qui donne vraisemblablement un pointeur invalide d'où plantage.
 
Il faut donc que tu précises explicitement le deuxième paramètre "%s", comme cela ton fprintf() maintenant est solide et ne plantera plus aléatoirement selon les données reçues.
 
Nouveau code :

Code :
  1. fprintf(fichier, "%s", list);

Reply

Marsh Posté le 06-06-2007 à 11:59:03    

oki c'etait bien ca merci =)

Reply

Marsh Posté le 06-06-2007 à 12:39:27    

ha oui un autre probleme lorsqu'il recupere des mails, il arrive parfois que ca reste figée...
vous avez une idée de comment évitez cela ?

Message cité 1 fois
Reply

Marsh Posté le 06-06-2007 à 16:06:39    

minokitaro a écrit :

ha oui un autre probleme lorsqu'il recupere des mails, il arrive parfois que ca reste figée...
vous avez une idée de comment évitez cela ?


 
Ton problème vient de ceci:

Code :
  1. while(strstr(list, "\r\n.\r\n" )


 
Selon le buffering et la réception (problème aléatoire), tu peux recevoir cette séquence de fin en plusieurs paquets
comme "xxx\r\n." suivi de "\r\n", ou "xxx\r\n" suivi de ".\r\n" etc.. ou autres combinaisons (xxx= exemples de caractères de longueur variable de fin de l'email reçu).
Ce qui fait que pour le serveur il a tout envoyé et tu as tout reçu mais tu n'as pas détecté la condition de fin pour finir
le protocole POP3.
 
A mieux détecter toutes les conditions de fin possible, ou voir des exemples de codes existants (chercher Google pour code source de client POP3).


Message édité par dnlilas le 06-06-2007 à 18:03:52
Reply

Marsh Posté le 06-06-2007 à 17:39:28    

oki

Reply

Sujets relatifs:

Leave a Replay

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