Discussion :

[gyver76370]

Hello

Je dois reprendre un programme d'un client en C.
Ayant une expérience très moyenne en C, je fait un appel a l'aide pour un Problème qui n'apparait que en mode Debug.

Le programme marche quand je build mais pas j'exécute en Debug , En release ca marche


Message d'erreur

Code :

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

1
2
API_Utils.c lin63,col5 thread id 3120
Dereference of pointer to freed memory

Voici le bout de code

Déclaration des Variable et appel de la fonction

Code :

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

1
2
3
4
5
6
Declaration de ma variable 
    unsigned int       vl_uMask[0]={0};  
 
 Appel de la fonction ci dessus
 
Util_GetMaskFromSig (vl_uMask);

Fonction

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
int Util_GetMaskFromSig (unsigned int *mask){
 
    unsigned int mask1,mask2,mask3,mask4,mask5,mask6;
    unsigned int vl_mask = 0;
 
    sscanf(vg_szDeviceHSDIORadiatedSup,    "%i",&mask1);
    sscanf(vg_szDeviceHSDIOModulation,     "%i",&mask2);
    sscanf(vg_szDeviceHSDIOPhaseModulation,"%i",&mask3);
    sscanf(vg_szDeviceHSDIOMskControl,     "%i",&mask4);
    sscanf(vg_szDeviceHSDIOTestSup,        "%i",&mask5);
    sscanf(vg_szDeviceHSDIOTest,           "%i",&mask6);
 
 
 
    vl_mask=pow(2,mask1)+pow(2,mask2)+pow(2,mask3)+pow(2,mask4)+pow(2,mask5)+pow(2,mask6);       
 
    *mask=vl_mask;   <===== ERREUR ICI
 
return 0;
}

Merci de votre aide

[gyver76370]

Bon solution trouvé entre temps

La variable suivante
unsigned int vl_uMask[0]={0};

il faut enlever le {0}

voila

Bonne journée

[WhiteCrow]

Hello,
définir un tableau de taille nulle c'est jamais bon →

Code :

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

unsigned int       vl_uMask[0]={0};

Vu le code, un simple unsigned int suffirait, ou alors un tableau de taille 1 (mais c'est moyen propre).

edit: ta solution n'est pas bonne, enfin àmha. Si tu passe un profiler sur ton code il va s'afforler.

[Médinoc]

De plus, ton calcul de vl_mask n'est pas très robuste: Si jamais deux masques fournis ont la même valeur, tu vas avoir des résultats inattendus. Tu devrais, selon si tu es cansé accepter ce cas ou non, remplacer les + par des |, ou faire des contrôles plus complexe pour afficher un message d'erreur et refuser l'entrée.
(Au passage, vu que tu bosses avec des entiers, 1<<x remplacerait avantageusement pow(2, x) ici)

[Sve@r]

Bonjour

Bon solution trouvé entre temps

La variable suivante
unsigned int vl_uMask[0]={0};

il faut enlever le {0}

Un réflexe à avoir: si enlever une instruction licite règle un souci, c'est que

1. en réalité le souci n'est pas réglé (il semble juste réglé)
2. le souci est bien plus profond que ce qu'il parait car il est très probablement un comportement indéterminé (en fait non, c'est réellement un comportement indéterminé)

Mettre ={0} pour initialiser un tableau est parfaitement autorisé. Si le mettre fait crasher ton truc et l'enlever ne crashe plus, c'est que tu es tombé direct dans le cas ci-dessus.

voila

Tu l'as dit.