Discussion :

[Médinoc]

Quel est le type de nb_send ?

[Creuvard]

La j'ai juste remis la boucle dans laquelle ca plante, les définitions sont sur la page précédente (en bas de la page)

Sinon pour nb_send

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
size_t nb_send;

[Médinoc]

size_t est un type non-signé.
Edit: Ça m'apprendra à me fier aux fonctions Win32...

[Emmanuel Delahaye]

La j'ai juste remis la boucle dans laquelle ca plante, les définitions sont sur la page précédente (en bas de la page)

Sinon pour nb_send

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
size_t nb_send;

Ben voilà. T'as compris la différence ?

La valeur retournée par send() est d'abord un entier signé (officiellement, selon POSIX.1, qui est une norme ISO, un ssize_t, mais non standard C, ou peut être C99...).

Ca permet de savoir si il y a erreur ou non.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
 
   ssize_t err = send(...);
   if (err != SOCKET_ERROR)
   {

Après le test, et seulement après, on peut considérer la valeur comme non signée. Pas avant, sinon, on ne voit jamais le -1.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
      size_t n = (size_t) err;

J'ai donné précédemment une version un peu moins paranoïaque et probablement plus portable...

[Emmanuel Delahaye]

size_t est un type non-signé, et ne sert à rien puisque send() retourne un int.

Officiellement (POSIX.1), un ssize_t...

[Creuvard]

Je viens de passer mon size_t en ssize_t et j'ai plus de souci avec GCC (avec les options -Wall -Wextra -O2 donc c'est déja une chose de gagné, je vous en remercie

Voici donc ma fonction telle qu'elle est a l'instant t (avec les modifs apportées depuis le début du post)

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
Edit: Voir post ci-aprés

Mon code me semble suivre les conseils que vous m'avez donné, pourtant j'ai toujours le même problème, j'ai donc surement ratté quelque chose, mais je vois pas quoi.

[Emmanuel Delahaye]

Mon code me semble suivre les conseils que vous m'avez donné, pourtant j'ai toujours le même problème, j'ai donc surement ratté quelque chose, mais je vois pas quoi.

Malgré mon insistance, il y a toujours un 'return (NULL);' inadéquate que ton compilateur devrait signaler...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
main.c: In function `envoie_de_fichier':
main.c:18: warning: `return' with a value, in function returning void

Si il ne le fait pas, c'est qu'il est mal réglé :

http://emmanuel-delahaye.developpez....tm#cfg_compilo

Je déconseille de combiner affectation et test dans un if. Ca alourdit inutilement.

[Creuvard]

Malgré mon insistance, il y a toujours un 'return (NULL);' inadéquate que ton compilateur devrait signaler...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
main.c: In function `envoie_de_fichier':
main.c:18: warning: `return' with a value, in function returning void

Arf c'est une erreur de copier-coller de ma part. désolé.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
 
#define BUFSIZE 512
#define SOCKET_ERROR -1 
void
envoie_de_fichier(int socket,char *nom, int debut, int fin)
{
        int reste;
        int nb_lu;
        ssize_t nb_send;
        char buffer[BUFSIZE];
        FILE *fichier_src;
        reste = fin - debut;
        fichier_src = fopen (nom, "rb");
        if ( fichier_src == NULL)
        {
                perror("fopen() ");
                return ;
        }
        fseek(fichier_src, debut, SEEK_SET);
        while ( reste > BUFSIZE)
        {
             nb_lu = fread( buffer, 1, sizeof buffer, fichier_src);
             if (nb_lu >= 0 )
             { 
                  nb_send = send(socket, buffer, nb_lu, 0);
                  if( (nb_send != SOCKET_ERROR)
                         reste = reste - nb_lu;
                  else
                  {
                         perror("send() ");
                         fclose(fichier_src);
                         return ;
                 }
              }
              else
              {
                     perror("fread()");
               }
        }
        nb_lu = fread( buffer, sizeof(char), reste, fichier_src);
        nb_send = send(socket, buffer, nb_lu, 0);
        if(nb_send == SOCKET_ERROR)
        {
                perror("write() ");
                fclose(fichier_src);
                return ;
        }
        fclose(fichier_src);
}

[Creuvard]

Salut à tous.

Voila je viens de remettre le nez dans ma fonction.
Il semblerait que (selon moi, donc cela n'est point sur dutout) que le problème vienne du signal SIGPIPE renvoyé par la fonction "socket()"

Un signal SIGPIPE est envoyé au processus tentant d'écrire sur une socket inutilisable, forçant les programmes ne gérant pas ce signal à se terminer.

J'ai trouvé un "man setsockopt()" dans lequel on peut passer à cette fonction la fonction " SO_NOSIGPIPE "

SO_NOSIGPIPE is an option that prevents SIGPIPE from being raised when a
write fails on a socket to which there is no reader; instead the write to
the socket returns with the error EPIPE when there is no reader.

Sauf que avec cette option j'ai une erreur de compilation.

int
create_tcp_server (int port, int nb_max_clients)
{
...
..
int socket_id;
int optval=1;
setsockopt(socket_id, SOL_SOCKET, SO_NOSIGPIPE,(char *)&optval, sizeof (optval));
...
..
.
return(socket_id);
}

Le message du compilateur

create_tcp_server.c: In function ‘create_tcp_server’:
create_tcp_server.c:34: erreur: ‘SO_NOSIGPIPE’ undeclared (first use in this function)
create_tcp_server.c:34: erreur: (Each undeclared identifier is reported only once
create_tcp_server.c:34: erreur: for each function it appears in.)

J'ai oublié quelque chose ? Ou il faut que je cherche ailleurs ?

[hegros]

A tout hasard dans quel header se trouve SO_NOSIGPIPE ?

[Creuvard]

A tout hasard dans quel header se trouve SO_NOSIGPIPE ?

A priori dans:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
 
#include <sys/types.h>
#include <sys/socket.h>

cf ce lien => http://www.hmug.org/man/2/setsockopt.php

Ils sont bien dans la fonction qui crée le socket.

[Creuvard]

Je m'y suis remis ce matin et j'i finalement trouvé deux solutions.

Rajouter le flag MSG_NOSIGNAL à la fonction "send()"

Et puis l'utilisation de la commande "signal(SIPIPE, SIG_IGN)"

La seconde méthode est peut être une méthode moin correcte (Bien que j'en sache rien) mais elle a le mérite de marcher sur les déconnexions impromptue des client sur les connexions via la lib OpenSSL (SSL_write)