Message d'erreur en Debug mais pas en Release

Version imprimable

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:

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:

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:

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

18/12/2020, 10h54
gyver76370

Bon solution trouvé entre temps

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

il faut enlever le {0}

voila

Bonne journée
18/12/2020, 10h55
WhiteCrow

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

Code:

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.
18/12/2020, 12h24
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)
19/12/2020, 14h35
Sve@r
Bonjour

Citation:

Envoyé par gyver76370

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.

Citation:

Envoyé par gyver76370

voila

Tu l'as dit.