Discussion :

[peijnoob]

Bonjour à tous,

J'ai un petit souci de chaines qui me fait lamentablement planter mon prog.

C'est un peu en rapport avec un sujet que j'ai posté il y a peu de temps, mais le problème n'est pas tout à fait le même.

Voici le bout de code qui me pose problème :

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
 
    if (strcmp(callBackMessage,"\0"))
    {
        tempBuff = (char *)malloc(sizeof(callBackMessage) + 3);
        if (tempBuff == NULL) MessageBox (0, "Pb malloc !!", "Erreur", MB_ICONINFORMATION);
        else{
        MessageBox (0, callBackMessage, "Test", MB_ICONINFORMATION);
        sprintf(tempBuff, "1K%s", callBackMessage);
        strcpy(lpszReceiveBuff, tempBuff);
        buffSize = strlen(tempBuff);
        free(tempBuff);
        return buffSize;
        }
    }
    return 0;

La 2e MessageBox, c'est pour vérifier le contenu de callBackMessage. Et il est bon.

Le problème se situe au moment du sprintf. L'appli de test, avec laquelle je fais appel à la fonction de la DLL plante lamentablement, sans même un message d'erreur.

Pourtant, l'allocation mémoire est OK, callBackMessage est OK. Je vois vraiment pas pourquoi ça veux pas marcher.

Au niveau déclarations :

Code :

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

1
2
3
4
5
 
char * callBackMessage;
[...]
char * tempBuff;
int buffSize;

callBackMessage est une chaine initialisée par une fonction callBack d'une API.

Si quelqu'un voit d'où vient le problème Je galère vraiment là, je suis loin d'être un expert en C.

[homeostasie]

Remarque sur la dimension de l'espace alloué:

Code :

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

tempBuff = (char *)malloc(sizeof(callBackMessage) + 3);

Le transtypage en char* est inutile et le sizeof retourne la taille du pointeur...alors que dans ton cas il faut retourner la taille d'un char.

Code :

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

tempBuff = malloc(sizeof(*callBackMessage) + 3);

[Franck.H]

Salut !


Ton problème doit surement se situer dans l'allocation:

Code :

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

1
2
 
tempBuff = (char *)malloc(sizeof(callBackMessage) + 3);

En fait, tu utilise sizeof pour la taille de la chaîne alors qu'il retourne la taille de l'objet ... il faut que tu détermine la taille de la chaîne avec strlen !
Attention toutes fois à strlen, elle te retourne la taille de la chaîne mais sans le zéro de fin de chaîne !

Après tout dépend de ce que tu veux faire, juste avoir la taille d'un char ou allouer d'après la taille de la chaîne contenue dans callBaclMessage ?


PS: Le cast n'est pas nécessaire en C dans ce cas de figure !

[peijnoob]

Oki, le problème vient sûrement de l'allocation mémoire alors, oui

Mon but est de créer un buffer pour copier callBackMessage en y ajoutant quelques caractères au début (et à la fin aussi d'ailleurs ^^)

Vais essayer en changeant la ligne de malloc, merci

[peijnoob]

Yes, après application de vos conseils, ça marche nickel, merci

PS: Le cast n'est pas nécessaire en C dans ce cas de figure !

Need explications sur ce point Dans quels cas le cast est-il nécessaire ?? (désolé, je suis pas encore expert, je lis en ce moment les cours présents sur le site, mais j'ai pas encore tout assimilé)

Merci encore

[granquet]

Need explications sur ce point Dans quels cas le cast est-il nécessaire ??

quand tu programme en C++.
en C les conversions (void *) -> (type *) se font implicitement.

http://emmanuel-delahaye.developpez....tes.htm#malloc

[peijnoob]

Bonjour à tous

J'ai encore un petit souci et je n'ai pas voulu faire un nouveau post.

Dans une autre fonction, je dois écrire une chaine à une adresse qui m'est passée en paramètre.

Voici le prototype de ma fonction :

Code :

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

1
2
 
DLLIMPORT int __stdcall getChaine6 (int channel, BYTE bufferAdr);

Ici, le paramètre est de type BYTE donc. Il représente l'adresse mémoire du buffer dans lequel je dois écrire. Me demandez pas pourquoi il n'est pas de type char *, je n'en sais moi même rien (je dois réécrire une dll dont je n'ai pas le code).

La chaine est de taille fixe et fait 6 caractères.

Mon code :

Code :

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

1
2
3
4
5
6
7
8
 
DLLIMPORT int __stdcall getChaine6 (int channel, BYTE bufferAdr)
{
 
    strcpy((char *)bufferAdr, "AZERTY");
 
    return 1;
}

Ma démarche a été la suivante : si bufferAdr est l'adresse de ma zone mémoire, il doit être possible de l'interpréter comme un pointeur sur cette zone mémoire. Un char * est un pointeur aussi.. Donc, je me suis dit que je pouvais le transtyper.

Je n'ai pas besoin d'allouer le buffer, il existe déjà (il est créé par l'application qui appelle la dll).

Mais pareil, lamentable crash de l'application, apparemment une fuite mémoire..

J'ai regardé plusieurs cours sur les pointeurs mais je n'arrive décidément pas à bien assimiler le concept.

Merci d'avance pour vos réponses

[Emmanuel Delahaye]

Code :

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

1
2
 
DLLIMPORT int __stdcall getChaine6 (int channel, BYTE bufferAdr);

Ici, le paramètre est de type BYTE donc. Il représente l'adresse mémoire du buffer dans lequel je dois écrire.

Ceci est extrèmement suspect. Tant que je n'aurais pas vu la définition de BYTE, telle que la voit ton compilateur je n'irais pas plus loin.

[peijnoob]

Je code avec Code::Blocks, le compilateur est celui par défaut, GCC donc.

La définition du BYTE, je ne sais même pas où je peux la trouver ^^

Après, qu'est-ce que tu trouves de suspect ?? Enfin, ici, j'invente rien, la définition de la fonction est celle là, et je peux pas faire autrement.. Si je le pouvais, je le ferais, je t'assures

[Médinoc]

+1 Emmanuel.

Quant à la définition DLLIMPORT, je me doute bien de ce qu'il y a dedans, mais c'est un mauvais usage des __declspec(dllimport)/__declspec(dllexport)...

PS: Tu es absolument sûr que le type est bien BYTE, et non pas BYTE*, PBYTE ou LPBYTE ?
Car ce qui est certain, c'est qu'on ne peut pas mettre une adresse dans un BYTE:

Code :

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

1
2
//WinDef.h du 14/04/2005, ligne 153
typedef unsigned char BYTE;

[Emmanuel Delahaye]

Je code avec Code::Blocks, le compilateur est celui par défaut, GCC donc.

La définition du BYTE, je ne sais même pas où je peux la trouver ^^

Après, qu'est-ce que tu trouves de suspect ?? Enfin, ici, j'invente rien, la définition de la fonction est celle là, et je peux pas faire autrement.. Si je le pouvais, je le ferais, je t'assures

BYTE n'est pas un type standard du C.

Certaines implémentations le définissent comme unsigned char, ce qui ne convient évidemment pas pour une adresse.

Je penche pour BYTE *.

[peijnoob]

Oups Médinoc, je n'avais pas vu ta réponse.

En effet, en changeant mon paramètre en BYTE *, ça a l'air de fonctionner..

En fait, je vais même carrément pourvoir le changer en LPSTR... En effet, je viens de me rendre compte d'un truc.. Les définitions présentes dans la doc de la dll que je dois remplacer sont complètement fausses !!!

En regardant de plus près le header fourni avec la DLL, j'ai les bonnes définitions (ça va un petit peu m'aider xD)

Mea culpa, désolé de vous avoir fait perdre de votre temps

[Emmanuel Delahaye]

Les définitions présentes dans la doc de la dll que je dois remplacer sont complètement fausses !!!

En regardant de plus près le header fourni avec la DLL, j'ai les bonnes définitions (ça va un petit peu m'aider xD)

Une fois de plus, le viel adage est vérifié :

"The only true documentation is the code itself"