Discussion :

[Joccy]

Bonjour,

je dois remplir un pointeur (suite de datas) depuis des variables. Comment faire ?
j'ai essayé

Code :

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

1
2
3
4
5
6
 
mafonction(void *monptr)
{
  memcpy(monptr, madata, sizeof(u32));
  monptr += sizeof(u32);
}

mais je n'obtiens que des 0 ...

Merci de votre aide ...

[Médinoc]

monptr est n'est qu'une copie, donc le += n'a aucun effet.
Si tu veux le modifier, tu dois passer un pointeur vers lui.

[Joccy]

tu veux dire

Code :

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

1
2
3
4
5
6
 
mafonction(void **monptr)
{
  memcpy(monptr, madata, sizeof(u32);
  *monptr += sizeof(u32);
}

cela ne fonctionne pas mieux ... enfin, si j'ai une data (la derniere, fausse d'ailleurs) en premiere position et que des 0 derriere ...

[Médinoc]

Ben c'est normal: Si tu n'appelles qu'une fois la fonction, tu n'auras qu'une seule data, puisque tu copies juste la taille pour sizeof(u32)...

De plus, on ne peut normalement pas faire d'addition directement sur un void*. Tu dois passer par un char*, ou plus simple, un u32* :

Code C :

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

1
2
3
u32* p = *monptr;
p++;
*monptr = p;

[Joccy]

non, en fait je fais une boucle sur mes datas (depuis un fichier: autre post )
Bon, je vais voir ca

[homeostasie]

tu veux dire

Code :

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

1
2
3
4
5
6
 
mafonction(void **monptr)
{
  memcpy(monptr, madata, sizeof(u32);
  *monptr += sizeof(u32);
}

cela ne fonctionne pas mieux ... enfin, si j'ai une data (la derniere, fausse d'ailleurs) en premiere position et que des 0 derriere ...

Maintenant faut aussi faire:

Code :

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

memcpy(*monptr, madata, sizeof(u32));

Apparemment "madata" est une variable globale, ca ne me semble pas utile, pourquoi ne pas la passer en paramètre de ta fonction?

EDIT: Je me demande aussi le type de "madata", peux tu montrer sa déclaration?

[Joccy]

ben, la j'ai segmentation fault !!!

[souviron34]

euh..

Faudrait peut-etre preciser :

1. ce qu'est madata
2. ce qu'est ptr
3. comment la fonction est appelee

la on commencera a y voir plus clair....

[Joccy]

en fait ma fonction n'est pas tout a fait celle la:

Code :

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

1
2
3
4
5
6
7
8
9
10
 
ma fonction(void **monptr)
{
 /* recuperation des données depus un fichier */
while fgets(.....)
  {
    /* avec un sscanf() je recupere la data de la ligne u8 ou u16 ou u32 madata*/
    /* et je dois la mettre dans mon pointeur, les unes à la suite des autres *
  }
}

voici le code dans le cas par exemple d'une valeur u32 :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
u32 val32 = 0;
size_t ptrlength = 0;
 
    int ret = sscanf (line, "%10lx", &val32);
    if (ret == 1)
    {
	u32 *p = *out_ptrMsgContent;
	p += ptrlength;
	*out_ptrMsgContent = p;
	memcpy(out_ptrMsgContent, &val32, sizeof(u32));
	ptrlength += sizeof(u32);
    }

[Joccy]

Bon, mes valeurs sont correctes mais toujours en u32, pas de u8 ou de u16 ...

J'esaye ca mais j'ai un "bus error":

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
/* if u8 */
...
 
/* if u16 */
u16 *pcData = (u16*)out_ptrMsgContent;
sscanf(line, "%06hx", &val16);
*(pcData++) = val16;
ou_ptrMsgContent = (void*)pcData;
 
/* if u32 */
...

ca fonctionne pour les premiers u16 puis les u8 puis quand je retrouve des u16 j'ai l'erreur ...

Une idée SVP ?

[Joccy]

ca y est, j'ai trouvé une solution: je passe tout en u8 et ca marche.

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
16
17
18
19
20
21
22
 
myfonction(void *my_ptr)
{
...
u8 *pcData = (u8*)my_ptr;
...
/* if u8 */
...
 
/* if u16 */
...
 
/* if u32 */
sscanf(line, "%10lx",&val32);
*(pcData++) = (u8)((val32 >> 24) &0xFFFFFF);
*(pcData++) = (u8)((val32 >> 16) & 0xFFFFFF);
*(pcData++) = (u8)((val32 >> 8) & 0xFFFFFF);
*(pcData++) = (u8)(val32 & 0xFFFFFF);
 
...
 
my_ptr = (void*)pcData;