memcpy qui ne veut pas memcopier

**nexoo** · 27/02/2007, 14h43

Bonjour à tous !

Tout d'abord, merci pour votre aide future !

Voilà mon soucis j'utilise la fonction memcpy :

Code :

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

1
2
3
4
5
6
7
 
#define   ADD_BINARY_ST         0x43800
length_binary_st = addEndST - addBeginST;
for( i=0; i<length_binary_st; i=i+sizeof(T_UINT32))
 {
  memcpy((T_CHAR*)ADD_BINARY_ST+i*sizeof(T_UINT32), (T_CHAR*)  addBeginST+i*sizeof(T_UINT32), sizeof(T_UINT32));
 }

avec addBeginST égal à 0xFF663800

Y a t-il quelque chose de mauvais la dedans ?

**Nemehy** · 27/02/2007, 14h57

Je trouve que tu te compliques beaucoup la vie. J'aurais écris plutôt un truc du genre :

Code :

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

1
2
3
4
 
#define   ADD_BINARY_ST         0x43800
length_binary_st = addEndST - addBeginST;
memcpy( (T_CHAR*)ADD_BINARY_ST, (T_CHAR*)  addBeginST, length_binary_st);

Je ne vois pas pourquoi tu essayes de copier petit bout par petit bout un bloc d'octets, alors que la fonction memcpy sert à copier un bloc d'octets.
Pour moi length_binary_st correspond au nombre d'octets que tu veux copier.

**nexoo** · 27/02/2007, 15h04

J'ai déjà essayé comme ça "basiquement" mais rien de concluant.

J'essaye un peu tout ce qui me passe par la tête dont cette méthode en me disant que ça marcherait mieux si cela se fait pas à pas...Mais non...

Je ne comprends pas et c'est pour cela que j'ai posté, je ne vois pas où est le probléme...

Je pensais à un truc, est ce que le fait de mettre mon adresse "source" pour la copie dans un "unsigned long" pose probléme ?

Autrement dit, pour le moment, j'ai ça :

T_UINT32 addBeginST = 0xFF663800;

alors que memcpy a besoin d'une zone mémoire...

**stephl** · 27/02/2007, 15h19

A première vue, je ne vois rien de faux, même si l'utilisation de memcpy() est un peu étrange. Comment voyez-vous que cela ne fonctionne pas? Quel est le but?

**nexoo** · 27/02/2007, 15h27

Pour expliquer en gros, il faut que je charge des données d'une flashprom vers de la ram.
j'ai un emulateur qui me permet de voir en "watchant" l'adresse de destination que le move ne se fait pas...

Je ne comprends pas...Je vais me renseigner si il n'y aurait pas des autorisations d'ecriture lecture sinon je ne vois vraiment pas le probleme...

**stephl** · 27/02/2007, 15h36

Si le code proposé par Nemehy ne fonctionne pas, je ne vois pas non plus d'où peut provenir le problème.

**nexoo** · 27/02/2007, 15h37

Merci en tout cas pour votre aide !

**stephl** · 27/02/2007, 15h45

Qu'est-ce que le type T_UINT32? unsigned int (32 bits)?

**nexoo** · 27/02/2007, 15h55

Cela :

typedef unsigned long T_UINT32;

Et donc ui int de 32bits

**stephl** · 27/02/2007, 16h03

A supposer que l'adresse de destination soit ADD_BINARY_ST et que celle de source soit addBeginST, alors, à mon avis, l'appel correct de memcpy() devrait être:

Code :

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

1
2
length_binary_st = addEndST - addBeginST;
memcpy(ADD_BINARY_ST,addBeginST,length_binary_st*sizeof addBeginST);

**nexoo** · 27/02/2007, 16h39

En fait, je pense avoir une idée.

Vu que mon adresse "source" pour la fonction memcpy est dans une variable de type T_UINT32 "addBeginST", je pense que je prends comme adresse "source", non pas comme je le voudrais le contenu de la variable, mais l'adresse de la variable.

**Nemehy** · 27/02/2007, 16h40

Envoyé par stephl

A supposer que l'adresse de destination soit ADD_BINARY_ST et que celle de source soit addBeginST, alors, à mon avis, l'appel correct de memcpy() devrait être:

Code :

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

1
2
length_binary_st = addEndST - addBeginST;
memcpy(ADD_BINARY_ST,addBeginST,length_binary_st*sizeof addBeginST);

Petite remarque, ce ne serait pas plutôt la taille de ce que pointe addBeginST et non pas la taille du pointeur ? du genre :

Code :

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

1
2
3
4
TRUC * addBeginST;
[...]
length_binary_st = addEndST - addBeginST;
memcpy(ADD_BINARY_ST,addBeginST,length_binary_st*sizeof(TRUC));

---
EDIT (dsl, nexoo j'avais pas vu que tu avais posté) :
a priori, je ne vois pas pourquoi tu prendrais l'adresse de la variable si tu ne mets pas de & devant celle-ci.
Mets nous ton code en entier, ça sera peut-être plus simple...

**stephl** · 27/02/2007, 16h44

Envoyé par Nemehy

Petite remarque, ce ne serait pas plutôt la taille de ce que pointe addBeginST et non pas la taille du pointeur ? du genre :

D'après le type de addBeginST, non. Il semble que ce soit un entier dont la valeur est l'adresse source.

**nexoo** · 27/02/2007, 17h00

Voici donc pour resumer où j'en suis :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
#define ADD_BINARY_ST 0x43800
 
void fonction(unsigned long addBeginST, unsigned long addEndST)
{
...
length_binary_st = addEndST - addBeginST;
memcpy((T_CHAR*)ADD_BINARY_ST, (T_CHAR*)addBeginST, length_binary_st);
...
}

Avec addBeginST contenant 0xFF663800, l'adresse à laquelle je souhaite faire le "move"...

**stephl** · 27/02/2007, 17h03

Je pense que le troisième paramètre de memcpy() n'est pas bon: cf post #10.

**Nemehy** · 27/02/2007, 17h08

En fait ce qui me parait le plus bizarre, c'est qu'il n'y ait même pas quelques octets qui aient été copiés...

Sinon je ne pense pas qu'il faille multiplié par quoique ce soit :
Avec T*p1 et T*p2, on a normalement équivalence :
p2-p1 <=> ( (unsigned long)p1 - (unsigned long) p2 ) / sizeof( T )

Ici addBeginST est de type unsigned long, donc il manque le type T dans l'équation, d'où une autre idée à tester :

Code :

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

1
2
3
4
5
6
7
8
9
10
#define ADD_BINARY_ST 0x43800
 void fonction(unsigned long addBeginST, unsigned long addEndST)
{
T_CHAR * p1 = addBeginST;
T_CHAR * p2 = addEndST;
...
length_binary_st = p2-p1;
memcpy((T_CHAR*)ADD_BINARY_ST, p1, length_binary_st);
...
}

**stephl** · 27/02/2007, 17h17

Oui, excusez-moi, vous avez raison. Il ne faut pas multiplier par la taille de unsigned long; en effet, la taille est ici calculée comme une différence d'entiers, pas de pointeurs... Mille excuses

**nexoo** · 27/02/2007, 17h19

Pas de problémes Stephl

Sinon j'vais tester ton idée nemehy...

**nexoo** · 28/02/2007, 10h15

Bon ça ne marche toujours pas...

Me suis renseigné, normalement pas de problémes d'accés donc je vais faire ça à la main avec deux pointeurs que je vais faire avancer...

Encore merci à vous deux pour votre aide !

**stephl** · 28/02/2007, 10h30

Bizarre...

Peut-être pouvez-vous essayer ainsi alors:

Code :

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

1
2
3
4
5
6
char *dest=(char *) ADD_BINARY_ST;
char *src=(char *) addBeginST;
unsigned long length_binary_st=addEndST-addBeginST;
unsigned long i;
 
for (i=0;i<length_binary_st;++i) dest[i]=src[i]; /*ou *dest++=*src++*/