System.AccessViolationException avec c_str

Bonjour,

J'essaie de coder une application pour utiliser une caméra GigE Basler Scout. Pour cela j'utilise visual studio 2008 et le SDK fourni par le fabriquant. L'application est basée sur les windows form en C++/CLI.

J'ai un problème lorsque j'essaie de récupérer l'adresse IP de la caméra. Le SDK prévoit une méthode GetIPAddress pour cela. Celle-ci me retourne un objet de type Pylon::String_t.

Ce type permet de faire appel à des fonctions utilisées par les std::string comme c_str. Dans le but d'afficher le résultat dans un label, je cherche à passer le résultat en System::String via :

Code:

---

System::String ^sIP = gcnew System::String(pBgdi.GetIpAddress().c_str());

---

La compilation se passe sans problème mais j'ai une erreur à l'exécution. Je me retrouve avec :

Citation:

---

Une exception non gérée du type 'System.AccessViolationException' s'est produite dans TestPylonNET.exe

Informations supplémentaires*: Tentative de lecture ou d'écriture de mémoire protégée. Cela indique souvent qu'une autre mémoire est endommagée.

---

alors que sIP prend bien la valeur voulue (adresse en XXX.XXX.XXX.XXX)

Vous êtes sûr que "pBgdi.GetIpAddress().c_str()" contient un caractère fin de chaîne correcte ?

Comment puis-je être sûr que le caractère en fin de chaine est correct ? Car si j'ai bien compris, le dernier caractère de chaine doit être '\0', mais je n'arrive pas à trouver comment l'obtenir.

bonjour,
quand tu dis les std::string,pour les chaines standat en c++ il y a 2 cas du au fourbi qui distingue Ansi(1 octet) et Unicode(widechar=2 octets).
en utilisant le namespace std et avec la meme chaine on a :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

#include <string> using namespace std; void main() { //Ansi String string szAnsiString; szAnsiString = "c:\\Program Files\\Microsoft Visual Studio .Net";   //Wide Char String wstring wszWCharString; wszWCharString = L"c:\\Program Files\\Microsoft Visual Studio .Net"; }

---

Le "string^ c++/cli " lui c'est de l'unicode.
Il faudrait caster en widecar peut etre avant de convertir en "string^" de c++/cli.
bon code...

Moi, je préfère aller à la source.

Dans le débuggeur de VS (menu Debug de VS), il y a des vues "memory".

Sélectionnez en une et mettez l'adresse retournée par "pBgdi.GetIpAddress().c_str()". Vous verrez la zone mémoire contenant la chaîne, vous verrez comment elle est encodée et si le caractère de fin de chaîne est bien présent.

Les chaînes sont en ANSI. J'ai essayé le cast en wide char, mais je me retrouve avec la même erreur. En utilisant la vue memory pour le résultat de GetIPAddress().c_str(), j'obtiens :

Code:

---

0x04C97F78 0d f0 ad ba 31 39 32 2e 31 36 38 2e 31 2e 35 00 0d f0 ad ba .ð*º192.168.1.5..ð*º

---

Il y a bien la valeur '00' dans la mémoire allouée à cette chaine, si je ne me trompe pas dans mon interprétation.

Ce qui est relativement bizarre, c'est que lorsque j'utilise d'autres fonctions de la même classe, me retournant des objet de même type (Pylon::String_t), la conversion en System::String ^ fonctionne; comme par exemple :

Code:

---

String ^sVendor = gcnew System::String(pBgdi.GetVendorName().c_str());

---

En me servant de la vue mémoire pour cette chaîne classique pBgdi.GetVendorName().c_str(), j'obtiens :

Code:

---

0x05F10FF0 00 00 00 00 cd cd cd cd 42 61 73 6c 65 72 00 cd cd cd cd cd ....ÍÍÍÍBasler.ÍÍÍÍÍ

---

De plus, lorsque je passe en release, tout fonctionne... L'erreur obtenue en debug est une erreur première chance. Je n'arrive pas à trouver à quoi cela correspond et comment passer outre...

J'ai fini par trouvé un moyen pour faire fonctionner le code en DEBUG. Tout simplement :

Code:

---

1 2 3 4 5 6 7

try { sIP = gcnew System::String(pBgdi.GetIpAddress().c_str()); } catch(Exception ^) { }

---

Le DEBUG montre que vous faites des bêtises mais vous lui clouez le bec.
Ne vous étonnez pas si la machine se venge. :aie:

Vous tripatouillez au mauvais endroit en mémoire, vérifiez votre code.

Je me doute bien que ce n'est pas la meilleure façon de gérer le problème. Mais je ne suis jamais arrivé à le résoudre.

Il y a certainement une règle du C++/CLI qui m'échappe. Si vous pouviez m'indiquer laquelle, je vous en serai très reconnaissant.

AccessViolationException ne vient pas du code managé mais du code natif, qui ne change pas avec C++/CLI.

Utilisez les techniques de débuggages standards, C++/CLI ne change pas la manière de faire.

Activez l'arrêt de votre programme via la configuration du débuggeur de VS, lors du lancement de cette exception "AccessViolationException".

"menu de VS" -> "Debug" -> "Exceptions" -> "Native ..." -> "cochez dans le ligne correspondant AccessViolationException et la colonne "throw".

Le débuggeur vous montrera le code qui génère cette exception.

Dans mon Visual, les AccessViolation Exception sont dans les Win32 Exception. J'ai déjà effectué cette partie du Debug (mais ne suis pas aller beaucoup plus loin faute de connaissances dans le domaine :( ); Toujours est-il que l'exception est levée à chaque manipulation de chaine données réseau de la caméra ( à savoir : Adresse IP, Default Gateway, Masque sous réseau et adresse Mac). Le problème ne se présente pas avec d'autres données de même type (Pylon::String_t), lui-même dérivant de std::string.

Code:

---

1 2 3 4

try { sMacro = gcnew System::String(pBgdi.GetMacAddress().c_str()); }

---

Le pointeur d'instructions lors de la levée d'instruction pointe sur l'accolade de fermeture. Les adresses donnée dans la fenêtre sortie sont les suivantes :

Citation:

---

Exception de première chance à 0x007ee138 dans Scout_MultiSpectrale_Beta_v0.01.exe*: 0xC0000005: Violation d'accès lors de la lecture de l'emplacement 0x43303329.

---

Lorsque je vais voir ces deux adresses (0x007ee138 et 0x43303329), dans la vue mémoire, elles sont vides ( ce qui est peut être normal ...:?).

Bref, je ne vois pas trop où chercher...

Avez-vous vérifié dans votre cas, que vous n'étiez pas dans la situation décrite par la troisième phrase de la section "Notes" de la documentation MSDN du constructeur d'une System::String ?

http://msdn.microsoft.com/fr-fr/library/6y4za026.aspx

c_str retourne une chaîne terminée par 0, il n'y a pas le choix.

Pour transformer des chaînes entre natif et managé, le plus simple je trouve est d'utiliser marshal_as :

http://msdn.microsoft.com/en-us/library/bb384865.aspx

Code:

---

1 2 3 4 5 6

#include <marshal_cppstd.h>   using namespace msclr::interop;     String^ sMacro = marshal_as<String^>( pBgdi.GetMacAddress() );

---

Merci Loïc pour cette réponse concise.
Mais je crois que justement, si la chaîne est foireuse, c'est le "c_str()" qui plantera.

Avec le nouveau code, je pense que "marshal_as<String^>" plantera (et heureusement, si la chaîne est foireuse).

Donc commencez par vérifier la qualité de la chaîne retournée par "pBgdi.GetMacAddress()". J'aurais jamais confiance dans une API qui donne une adresse MAC sous forme de chaîne de caractère, pour moi c'est un entier 48 bits, voir un tableau d'octet, mais pas une string. :aie:

Voilà ce que j'obtiens dans la vue mémoire à l'adresse où est écrite ma chaine adresse MAC :

Code:

---

0x04D63078 0d f0 ad ba 30 30 33 30 35 33 30 43 36 35 43 35 00 f0 ad ba .ð*º0030530C65C5.ð*º

---

Si je ne me trompe pas dans la lecture, l'adresse MAC de ma caméra (0030530C65C5) est bien suivi de la valeur 00.

Cependant tout les symptômes d'une chaîne mal codée que vous décrivez Bacelar sont présents (le marshal_as m'a retourné l'exception).

Autre fait notable, lorsque je code :

Code:

---

1 2	Pylon::String_t Mac = pBgdi.GetMacAddress(); sMacro = gcnew System::String(Mac.c_str());

---

Le tout passe mais une exception AccessViolation est levée à la sortie de la fonction.:arf:

Bingo. 8-)

MSDN indique qu'il faut fournir un pointeur sur un tableau de caractères Unicode
http://msdn.microsoft.com/en-us/library/6y4za026.aspx

Or, votre dump mémoire montre une chaîne de caractère terminé par un simple '\0' (correcte pour une chaîne ASCII) et non le double caractère Unicode '\0\0'.

Nous sommes en présence d'une chaîne ASCII et non du chaîne UNICODE.

Il vous reste donc à choisir une solution qui permette de régler ce problème.
- Utilisez une API type "GetMacAddress" en version Unicode
- Convertir la chaîne ASCII en chaîne Unicode
- Utilisez les constructeurs de string ASCII friendly comme http://msdn.microsoft.com/en-us/library/ezh7k8d5.aspx ou encore mieux http://msdn.microsoft.com/en-us/library/9d876whe.aspx
- etc...

Merci pour cette réponse mais il y a encore des choses qui m'échappent. L'API ne fournissant pas de méthode retournant une chaine Unicode, j'ai choisit de faire appel à un constructeur prenant en compte l'encodage d'origine :

Code:

---

1 2 3 4

System::Text::Encoding ^ascii = System::Text::Encoding::ASCII; size_t size = pBgdi.GetMacAddress().size(); System::String ^Mac = gcnew System::String(pBgdi.GetMacAddress().c_str(), 0, size, ascii); label_MacAdress->Text = "Mac Address : " + Mac;

---

Seulement, à l'appel du constructeur de System::String comme décrit http://msdn.microsoft.com/en-us/library/9d876whe.aspx, j'obtiens toujours la même exception.

Autre fait étrange, si je souhaite récupérer une autre donnée, comme le nom du vendeur de la caméra, j'obtiens en dump:

Code:

---

0x060705A0 00 00 00 00 cd cd cd cd 42 61 73 6c 65 72 00 cd cd cd cd cd ....ÍÍÍÍBasler.ÍÍÍÍÍ

---

La chaine se termine donc encore par un '\0'. Mais il n'y a pas de problème si je code :

Code:

---

sVendor = gcnew System::String(pBgdi.GetVendorName().c_str())

---

Donc encore une fois : :arf:

Citation:

---

Code:

---

size_t size = pBgdi.GetMacAddress().size();

---

---

Size est égale à combien ?
Oui, je n'est aucune confiance dans "GetMacAddress()". :aie:

Citation:

---

This constructor processes characters from value starting at startIndex and ending at (startIndex + length - 1).

---

Citation:

---

Code:

---

System::String ^Mac = gcnew System::String(pBgdi.GetMacAddress().c_str(), 0, size, ascii);

---

---

Il vous manquerait pas un "size+1" à la place de "size" ? :aie:

Citation:

---

La chaine se termine donc encore par un '\0'.

---

Vous ne plantez pas parce que vous avec qu'un '\0' mais parce qu'il n'y a pas de '\0\0' avant la fin de la page mémoire allouée au programme par l'OS.

Effectivement, j'avais oublié le size + 1. Mais cela n'a pas résolu l'exception. Après contact avec le constructeur de la caméra et développeur API, il s'avère que cette méthode retournant ne peux être utilisée que lorsque l'objet caméra n'a pas été instancié, point non précisé dans la doc fournit ou, tout du moins, que je n'ai pas vu( et ce n'est pas faute de l'avoir parcouru...). Une fois le device instancié, il convient d'utiliser :

Code:

---

1 2 3

int64 Mac =pCamera->GevMACAddress(); int64 IP = pCamera->GevCurrentIPAddress();

---

Code retournant un entier, ce qui est bien plus pratique pour l'interprétation et affichage.

Citation:

---

Oui, je n'estai aucune confiance dans "GetMacAddress()".

---

8O Dur Dur l'orthographe tard le soir. :cry:



Je l'avais bien dit que cette API ne sentait pas bon du tout. :aie::aie::aie:
;)