Crash à l'exécution sur delete[]

Bonjour,

voilà, j'ai un ptit souci à l'exécution de mon appli...

Voilà où ça se passe :

Code:

---

1 2 3 4 5

CHAR* maChaine = new CHAR[nbChar]; memset(maChaine, 0, strl(maChaine)); maChaine =...; //... delete[] maChaine;

---

et donc, c'est sur le 'delete[]' que ça plante
et j'ai une erreur du genre :
Debug Assertion Failed!

Program : monAppli.exe
File : dbgdel.cpp
Line 52:

Expression : _BLOCK_TYPE_IS_VALID(pHead->nBlockUse)

Est-ce que quelqu'un pourrait me dire comment ça se fait?
qu'est-ce que je fais mal?

Merci

Code:

---

memset(maChaine, 0, strl(maChaine));

---

strl ne fait pas ce que tu penses, ca cherche le premier charactere 0 de la chaine. Or, tu appelles ca sur une chaine non initialisée
utilise nbChar a la place

Euh oups!
faute de frappe...
j'utilise 'strlen'...
ça devrait être bon, non?

non, c'est comme j'ai ecrit
strlen "calcule" la longueur d'une chaine en fonction de son contenu
elle ne donne pas la longueur du buffer

Bonjour,

pour le memset, il faut utiliser le même nombre utilisé pour l'allocation, et pas strlen(). Dans ton cas, c'est nbChar:

Code:

---

1 2 3 4 5

CHAR* maChaine = new CHAR[nbChar]; memset(maChaine, 0, nbChar); maChaine =...; //... delete[] maChaine;

---

Car dans ton exemple, strlen(machaine) renvoie une valeur supérieure à nbChar, et du coup le bloc est invalide.

Par contre, je ne sais pas pourquoi.

Mais ça, c'est du C. En C++, on utilisera un vector, ou directement une string s'il s'agit de char, ainsi, tous les problèmes de ce genre disparaissent.

Citation:

---

Envoyé par r0d

Car dans ton exemple, strlen(machaine) renvoie une valeur supérieure à nbChar, et du coup le bloc est invalide.

Par contre, je ne sais pas pourquoi.

---

Tout simplement, la mémoire n'étant pas initialisée, le premier caractère nul peut être (très loin) après nbChar.

Citation:

---

Envoyé par r0d

Mais ça, c'est du C. En C++, on utilisera un vector, ou directement une string s'il s'agit de char, ainsi, tous les problèmes de ce genre disparaissent.

---

++

ok,
bah j'ai essayé en mettant nbChar
bah, ça marche pas mieux :(

gggrrrr
le pire, c'est qu'au départ, j'avais fait sans le memset et ça marchait
mais là, j'sais pas c'que j'ai fait mais ça marche plus:cry:...

d'autres pistes??


En ce qui concerne cette utilisation de tableau de char
ouais, je sais, c'est un peu obsolète
mais j'dois m'interfacer avec une librairie en C...
en fait, j'ai une chaîne de type BSTR que je dois mettre dans un tableau d'octets...
c'est pour ça que je la passe entre deux dans un tableau de char...
mais y'a pitêtre d'autres solutions,
j'suis preneuse;)

c_str permet toujours de récupérer une chaîne à la 'C' :

Code:

---

1 2 3

std::wstring str(L"coucou"); BSTR bstr = SysAllocString(str.c_str()); SysFreeString(bstr);

---

A tout hasard, à quoi ressemble la suite de

Code:

---

maChaine =...;

---

Alors, pour donner plus de détails, j'ai ça :

Code:

---

1 2 3 4 5 6 7 8 9 10

octet *data; data = new octet[nbChar]; CComVariant DataValue;   CComBSTR S(DataValue.bstrValue); CHAR* maChaine = new CHAR[nbChar]; memset(maChaine, 0, nbChar); maChaine = W2A(S); memcpy(&data, maChaine, nbChar); delete[] maChaine;

---

Bon, c'est clair, que c'est un joyeux bordel:oops:...
j'suis trop une buse en C/C++...

= sur des pointeurs copie l'adresse et non la valeur !
Donc ce devrait être une copie et non une affectation.

Code:

---

1 2 3 4 5 6 7

octet *data; data = new octet[nbChar]; CComVariant DataValue;   CComBSTR S(DataValue.bstrValue); strncat(data, W2A(S), nbChar-1);

---

Ou carrément :

Code:

---

1 2 3 4 5 6

std::string data; CComVariant DataValue; /* [...] */ CComBSTR S(DataValue.bstrValue); data = W2A(S);

---

au passage un article sur les memset

http://blog.emmanueldeloget.com/inde...89-c-et-memset

Ah ouais,
c'est clair que c'est vachement plus simple:roll:
merci 3DArchi

et en fait, j'crois que ça venait d'une variable mal initialisée avant:aie:
ppfff, des fois pour voir d'où ça vient, c'est sioux:?...