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