Discussion distribuée en broadcast

Problème de réception

L'auteur de ce sujet a trouvé une solution à son problème.
Auteur du sujet

Bonjour à tous,

Je travail actuellement sur un petit tchat en C, sans serveur. En utilisant le broadcast pour communiquer avec d'autres clients, si il y en a.
L'objectif est assez simple de base. J'ai réussi à communiquer des messages entre deux ordinateurs sans problèmes.

Je ne communique pas uniquement du texte, mais une structure, dans l'objectif de transmettre différents types de messages par la suite.
Mais lorsque j'envoie un message trop long, au bout d'un certain nombre de caractère je dois avoir des données binaires qui subsiste car à l'affichage j'ai des symboles spéciaux. ("aaaaa4V�aaaaa" au lieu d'une chaîne de 'a' uniquement) Second problème beaucoup plus embêtant, peut-être lié au premier : lorsque je veux connaître l'adresse IP de l'émetteur, je ne reçois plus le message, malgré que j'arrive bien à voir la bonne adresse IP de l'émetteur.

Voici d'abord les parties majeurs du codes :

Les structures :

 1
 2
 3
 4
 5
 6
 7
 8
 9
10
11
12
13
14
15
16
17
18
typedef enum msg_type
{
    MT_INVAL = 0,
    MT_HELLO = 1,
    MT_MSG   = 2,
    MT_NICK  = 3,
    MT_COLOR = 4,
    MT_MAX,
} msg_type_t;

#define BUF_SIZE 1024
typedef struct msg
{
    msg_type_t  type;
    uint16_t    len;
    char        buf[BUF_SIZE];
} msg_t;
#define MSG_SIZE sizeof(msg_t)

Fonctions d'envoies (dans un thread indépendant)

 1
 2
 3
 4
 5
 6
 7
 8
 9
10
11
12
13
14
15
16
17
18
19
20
21
22
void send_msg(msg_t *data, size_t data_len)
{
  printf("Send : %d : %d : %s\n", data->type, data->len, data->buf);
    if(sendto(bcast_sock, data, data_len, 0, (struct sockaddr *) &bcast_addr_in, sizeof(bcast_addr_in)) != MSG_SIZE)
      printf("Erreur %d lors de l'envoie du message\n", errno);
}

void enter_loop(void)
{
    while (1)
    {
        char str[BUF_SIZE];

        fgets(str, BUF_SIZE, stdin);

        msg_t * msg = get_buf(MT_MSG);
      msg->len = BUF_SIZE;
      strncpy(msg->buf, str, BUF_SIZE);
        send_msg(msg, MSG_SIZE);
        //free_buf(msg);
    }
}

Fonction de réception

 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
void *receive_thread(void *arg)
{
  while (1)
    {
        struct sockaddr_in sender_addr;
        unsigned int addrlen = sizeof(sender_addr);
        user_t *sender = NULL;
        void *data = malloc(MSG_SIZE);
        msg_t *msg = get_buf(MT_INVAL);

      //if((recvfrom(bcast_sock, &data, MSG_SIZE, 0, (struct sockaddr *) &sender_addr, &addrlen)) != MSG_SIZE)
      if((recvfrom(bcast_sock, &data, MSG_SIZE, 0, NULL, 0)) != MSG_SIZE)
          printf("Erreur de reception %d\n", errno);
      else
      {
          printf("data : %d : %d : \n", data, (&data+sizeof(msg_type_t)));
          msg = (msg_t*) &data;
          //printf("taille : %d\n", sizeof(*msg));
          //printf("%c", msg->buf[msg->len]);
          printf("%d : %d : %s\n", msg->type, msg->len, msg->buf);

          sender = lookup_user(&sender_addr);

          //process_received_msg(sender, (msg_t*) &data);

          /*printf("%s\n", inet_ntoa(sender_addr.sin_addr));*/
      }
    }
    return NULL;
}

Lors de la réception, si j'utilise :

1
if((recvfrom(bcast_sock, &data, MSG_SIZE, 0, NULL, 0)) != MSG_SIZE)

Je reçois le message sans soucis.

Mais si j'utilise :

1
if((recvfrom(bcast_sock, &data, MSG_SIZE, 0, (struct sockaddr *) &sender_addr, &addrlen)) != MSG_SIZE)

Je ne reçois plus le message, en revanche je reçois les informations de l'émetteur correctement.

Je me suis demandé si le fait que mes ordinateurs soient pour l'un sous 32 bits et l'autres sous 64 bits influerait la taille de la structure à cause de certains types. C'est peut-être le cas car j'obtiens des décalages légèrement différents pour les caractères spéciaux. Mais j'ai aussi essayer en compilant en 32 bits, et de plus, j'ai le problème aussi simplement en local.

Voici le code complet pour mieux comprendre :

  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
 50
 51
 52
 53
 54
 55
 56
 57
 58
 59
 60
 61
 62
 63
 64
 65
 66
 67
 68
 69
 70
 71
 72
 73
 74
 75
 76
 77
 78
 79
 80
 81
 82
 83
 84
 85
 86
 87
 88
 89
 90
 91
 92
 93
 94
 95
 96
 97
 98
 99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <arpa/inet.h>
#include <sys/socket.h>
#include <sys/types.h>
#include <errno.h>
#include <string.h>
#include <stdbool.h>
#include <pthread.h>
#include <netdb.h>

//#define BCAST_ADDR "192.168.10.255" // ou #define BCAST_ADDR (192<<24+168<<16+3<<8+255) puis htons(BCAST_ADDR)
//#define BCAST_PORT 1234

#define BCAST_ADDR "192.168.1.255"    // ou #define BCAST_ADDR (192<<24+168<<16+3<<8+255) puis htons(BCAST_ADDR)
#define BCAST_PORT 5000


typedef enum msg_type
{
    MT_INVAL = 0,
    MT_HELLO = 1,
    MT_MSG   = 2,
    MT_NICK  = 3,
    MT_COLOR = 4,
    MT_MAX,
} msg_type_t;

#define BUF_SIZE 1024
typedef struct msg
{
    msg_type_t  type;
    uint16_t    len;
    char        buf[BUF_SIZE];
} msg_t;
#define MSG_SIZE sizeof(msg_t)


#define NB_USER 100
#define NICK_DEFAULT "Guest"
#define COLOR_DEFAULT (colors[0].name)
#define NICK_LEN 256
#define NODE_INFO_LEN 256
typedef struct user
{
    struct sockaddr_in      sa;
    char                    nick[NICK_LEN];
    char                    node_info[NODE_INFO_LEN];
    const char             *color;
} user_t;

user_t users[NB_USER];
int user_online = 0;



/***** GLOBAL VARIABLES *****/

int bcast_sock = -1;
struct sockaddr bcast_addr;
struct sockaddr_in bcast_addr_in;




/***** PROTOTYPE DE FONCTIONS *****/

int init_bcast(struct sockaddr *bcast_addr);
msg_t *get_buf(msg_type_t type);
void free_buf(msg_t *buf);
user_t *add_user(struct sockaddr_in *sa);
void get_node_info(user_t *user, struct sockaddr_in *si);
void del_user(user_t *user);
user_t *lookup_user(struct sockaddr_in *sa);
void show_users();
void *receive_thread(void *arg);
void process_received_msg(user_t *user, msg_t *msg);
void send_msg(msg_t *data, size_t data_len);
void enter_loop(void);




/***** MAIN *****/

int main (int argc, char **argv)
{
  printf("Bienvenue, vous pouvez discutez avec les personnes connectées sur le réseau !\n");
  
  pthread_t receive_th;
  int rc;

  bcast_sock = init_bcast(&bcast_addr);

    rc = pthread_create(&receive_th, NULL, receive_thread, NULL);
    if (rc < 0)
      printf("Cannot create receive thread\n");

    enter_loop();

  
  return 0;
}




/***** FONCTIONS *****/

int init_bcast(struct sockaddr *bcast_addr)
{
  int sockfd;
  struct sockaddr_in broadcastAddr;
  int broadcastPermission;

  if ((sockfd = socket(PF_INET, SOCK_DGRAM, IPPROTO_UDP)) < 0)
        perror("socket() failed");

    broadcastPermission = 1;
    if (setsockopt(sockfd, SOL_SOCKET, SO_BROADCAST, (void *) &broadcastPermission, sizeof(broadcastPermission)) < 0)
        perror("setsockopt() failed");

    memset(&broadcastAddr, 0, sizeof(broadcastAddr));
    broadcastAddr.sin_family = AF_INET;
    broadcastAddr.sin_addr.s_addr = inet_addr(BCAST_ADDR);
    broadcastAddr.sin_port = htons(BCAST_PORT);

    if (bind(sockfd, (struct sockaddr *) &broadcastAddr, sizeof(broadcastAddr)) < 0)
        perror("bind() failed");

    bcast_addr = (struct sockaddr *) &broadcastAddr;
    bcast_addr_in = broadcastAddr;
  return sockfd;
}

msg_t *get_buf(msg_type_t type)
{
    msg_t *msg = malloc(MSG_SIZE);
  msg->type = type;

    return msg;
}

void free_buf(msg_t *buf)
{
    free(buf);
}

user_t *add_user(struct sockaddr_in *sa)
{
  user_t *user = NULL;
  
  user = malloc(sizeof(user_t));
  if (user != NULL)
  {
      user->sa = *sa;
      strncpy(user->nick, NICK_DEFAULT, NICK_LEN);
      get_node_info(user, sa);
      //user.color = COLOR_DEFAULT;
  
      users[user_online] = *user;
      user_online++;
  }
  else
      printf("ERREUR : Allocation utilisateur échouée\n");
  
  return user;
}

void get_node_info(user_t *user, struct sockaddr_in *sa)
{
  int port = sa->sin_port;
  char *ip = inet_ntoa(sa->sin_addr);
  sprintf(user->node_info, "%d", port);
  strcat(user->node_info, ":");
  strcat(user->node_info, ip);
}

void del_user(user_t *user)
{
  int i;
  int supprimer = 0;
  for(i=0 ; i < user_online-1 ; i++)
  {
      if (supprimer == 0 && memcmp(&(&users[i])->sa, &user->sa, sizeof(struct sockaddr_in)) == 0)
          supprimer = 1;
      if (supprimer == 1)     
          users[i] = users[i+1];
  }
  user_t empty;
  users[user_online-1] = empty;
  user_online--;
}

user_t *lookup_user(struct sockaddr_in *sa)
{
  user_t *user = NULL;
  int i;
    for(i=0 ; i < user_online-1 ; i++)
  {
      if (memcmp(&(&users[i])->sa, &sa, sizeof(struct sockaddr_in)) == 0)
      {
          user = &(user[i]);
          return user;
      }
  }
    return user;
}

void show_users()
{
  int i;
    for(i=0 ; i < user_online-1 ; i++)
  {
      printf("User %s\n", users[i].nick);
      printf("\tUsing color %s\n", users[i].color);
      printf("\tConnected with %s\n", users[i].node_info);
  }
}

void *receive_thread(void *arg)
{
  while (1)
    {
        struct sockaddr_in sender_addr;
        unsigned int addrlen = sizeof(sender_addr);
        user_t *sender = NULL;
        void *data = malloc(MSG_SIZE);
        msg_t *msg = get_buf(MT_INVAL);

      //if((recvfrom(bcast_sock, &data, MSG_SIZE, 0, (struct sockaddr *) &sender_addr, &addrlen)) != MSG_SIZE)
      if((recvfrom(bcast_sock, &data, MSG_SIZE, 0, NULL, 0)) != MSG_SIZE)
          printf("Erreur de reception %d\n", errno);
      else
      {
          printf("data : %d : %d : \n", data, (&data+sizeof(msg_type_t)));
          msg = (msg_t*) &data;
          //printf("taille : %d\n", sizeof(*msg));
          //printf("%c", msg->buf[msg->len]);
          printf("%d : %d : %s\n", msg->type, msg->len, msg->buf);

          sender = lookup_user(&sender_addr);

          //process_received_msg(sender, (msg_t*) &data);

          /*printf("%s\n", inet_ntoa(sender_addr.sin_addr));*/
      }


        //sender = lookup_user(&sender_addr);
        /*if (user != NULL)
        {
            printf("user y\n");
        }
        else
          add_user(&sender_addr);*/

        /* À décommenter */
        //process_received_msg(user, msg);
    }
    return NULL;
}

void process_received_msg(user_t *user, msg_t *msg)
{
    switch(msg->type)
    {
        case MT_HELLO:
            {
                printf("%d\n", msg->type);
            }
            break;

        case MT_NICK:
            {
                printf("%d\n", msg->type);
            }
            break;

        case MT_MSG:
            {
                printf("Receive : %d : %d : %s\n", msg->type, msg->len, msg->buf);
            }
            break;

        case MT_COLOR:
            {
                printf("%d\n", msg->type);
            }
            break;

        default:
            perror("Invalid message type");
            break;
    }
}

void send_msg(msg_t *data, size_t data_len)
{
  printf("Send : %d : %d : %s\n", data->type, data->len, data->buf);
    if(sendto(bcast_sock, data, data_len, 0, (struct sockaddr *) &bcast_addr_in, sizeof(bcast_addr_in)) != MSG_SIZE)
      printf("Erreur %d lors de l'envoie du message\n", errno);
}

void enter_loop(void)
{
    while (1)
    {
        char str[BUF_SIZE];

        fgets(str, BUF_SIZE, stdin);

        msg_t * msg = get_buf(MT_MSG);
      msg->len = BUF_SIZE;
      strncpy(msg->buf, str, BUF_SIZE);
        send_msg(msg, MSG_SIZE);
        //free_buf(msg);
    }
}

Pour compiler simplement avec gcc :

1
gcc main.c -o discussion -lpthread

J'ai commenté / décommenté pas mal d'endroit pour essayer plein de chose. Cela fait 2 jours que je recherche en vain. Impossible de trouver pourquoi j'ai un problème de réception du message de manière intacte, et pourquoi je n'arrive pas à recevoir le message lorsque je veux aussi récupérer les informations de l'émetteur.
Si il y a des zesteux pour m'éclairer, je les en remercie d'avance de leur courage :)

+0 -0

Cette réponse a aidé l'auteur du sujet

Je ne sais pas si c'est la cause de ton problème, mais tu ne passes pas le bon pointeur comme deuxième argument de la fonction recvfrom. La fonction attend l'addresse du buffer où écrire les données, mais tu lui passes l'addresse d'un pointeur sur ce buffer.

L'appel devrait se lire comme suit:

1
if((recvfrom(bcast_sock, data, ...
+1 -0
Auteur du sujet

En effet, j'ai mal lu le man et pas fait attention à cela sur les exemples que j'ai lu.
C'est bien l'origine de mes deux erreurs. Je reçois maintenant très bien le message et les informations de l'émetteur en même temps.

Merci beaucoup pour ta relecture :)

+0 -0
Vous devez être connecté pour pouvoir poster un message.
Connexion

Pas encore inscrit ?

Créez un compte en une minute pour profiter pleinement de toutes les fonctionnalités de Zeste de Savoir. Ici, tout est gratuit et sans publicité.
Créer un compte