Discussion :

[ikuzar]

Bonjour,
mon programme fait un segmentation fault quand il execute l'instruction suivante :
memset(policy, 0, sizeof(srtp_policy_t));

Voici le message de seg fault :

Code :

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

1
2
3
4
Program received signal SIGSEGV, Segmentation fault.
memset () at ../sysdeps/x86_64/memset.S:331
331	../sysdeps/x86_64/memset.S: Aucun fichier ou dossier de ce type.
	in ../sysdeps/x86_64/memset.S

Voici le contexte:

Code :

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

1
2
3
4
5
6
srtp_policy_t  *policy;
  policy = si->u.rtp.info->policy;
 
  memset(policy, 0, sizeof(srtp_policy_t));
  if(computeSessKey(si, policy, key_len) == -1)
    return -1;

et voici srtp_policy_t :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
typedef struct srtp_policy_t {
  ssrc_t        ssrc;        /**< The SSRC value of stream, or the 
                              *   flags SSRC_ANY_INBOUND or 
                              *   SSRC_ANY_OUTBOUND if key sharing
                              *   is used for this policy element.
                              */
  crypto_policy_t rtp;         /**< SRTP crypto policy.                  */
  crypto_policy_t rtcp;        /**< SRTCP crypto policy.                 */
  unsigned char *key;          /**< Pointer to the SRTP master key for
                                *    this stream.                        */
  struct srtp_policy_t *next;  /**< Pointer to next stream policy.       */
} srtp_policy_t;

Apparemment c'est sizeof(srtp_policy_t qu'il n'aime pas ...

Comment je devrais indiquer à memset qu'il faut prendre la taille de srtp_policy_t ?

Merci d'avance pour votre aide.

[jeroman]

Salut

Code :

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

1
2
3
policy = si->u.rtp.info->policy;
  
  memset(policy, 0, sizeof(srtp_policy_t));

Si memset plante, c'est que tu écris dans une zone interdite en mémoire : policy pointe probablement n'importe où. Un printf lèvera le voile.

Par contre, je ne comprends pas ce que tu essayes de faire exactement... Si tu veux mettre à 0 tous les champs d'une structure, je pense qu'il est préférable de le faire champs par champs. D'autant qu'il y a des pointeurs parmi les membres de la structure et NULL ne vaut pas forcément 0 (même si c'est le cas la plupart du temps).

[ikuzar]

Effectivement, policy pointerais vers une zone mémoire qui a été desallouée ...
Pour memset(policy, 0, sizeof(srtp_policy_t));
Je me suis inspiré de l'exemple de la doc ... c'est comme cela qu'il initialise policy ...
Je verrai plus tard si ca pourrait etre la source d'un quelconque problème

[Thierry Chappuis]

Une bonne pratique pour trouver rapidement ce type d'erreur est d'affecter la NULL à tout pointeur pointant sur une zone mémoire qu'on vient de désallouer. NULL a en effet l'avantage d'être un adresse invalide testable.

Thierry