strcpy et strcpy_s

bonjour, j'ai ramene mon projet sous visual studio 2005 pro (de dev-cpp) sous dev-cpp j utilisait strcpy..aucun probleme.. alors que la premiere fois que j'ai compile sous vs, un warning aparait comme quoi strcpy n est pas tres sur et que je doi utiliser strycpy_s je le remplace avec strycpy_s (qui prend un argument de plus, celui de la taille maximale de la chaine de destination) j ecrit alors: strcpy_s(lang[i],sizeof(lang[i]),temp_buffer); et bam ca plante lors de cette..(pas d erreur de compilation, ca plante quand j execute l exe) une suggestion peut etre? doi-je juste laisser strcpy et ignorer le warning ?

Salut,
Tu peux bien sûr ignorer le warning. Personnellement, je préfère utiliser la fonction API lstrcpy(). Pour un nombre déterminé de caractères il y'a lstrcpyn().

j'ai eu le même problème, vérifie bien la taille maximale de la chaine de destination, pense au zero terminal.

louis14

c est a dire, je doi mettre strcpy_s(lang[i],sizeof(lang[i]+1),temp_buffer); ?

non surtout pas  mais -1 pour laisser de la place au zéro terminal. j'avais une exception de ce type et en applicant cette démarche je n'en ai plus.


louis14

la taille de lang doit être choisie de façon à ne pas avoir de troncature.

louis14

-1 je le met dans le sizeof ou apres? (sizeof(lang[i])-1) ou (sizeof(lang[i]-1) ?

ceci est mon code: char* lang[50]; for(i=1;i<51;i++) { wsprintf(temp_str,"%ld",i); length=GetPrivateProfileString("hydrid_tool_language",temp_str,"",temp_buffer,MAX_PATH,lngfile); lang[i]=NULL; lang[i]=malloc(sizeof(char)*(length+1)); if(lang[i]==NULL) { error(9,hwnd_loading); } strcpy(lang[i],temp_buffer); } Pouvait vous me dire svp comment je vai ecrire strcpy_s qui remplacera strcpy..merci

wsprintf, c'est pas pour du UNICODE ça ?

De plus, peut-être est-ce voulu mais, tu commence ton tableau à 1. Normalement c'est à 0 donc: for(i=0;i<51;++i)

char *lang[50] <- c'est un tableau de pointeurs.
Si tu fais sizeof(lang[i]), tu obtiendra à tout les coups 4, car c'est la taille d'un pointeur.

Moi, à ta place, je choisirais d'ignorer l'avertissement. Si tout est bien codé, il n'y a aucune raison d'utiliser strcpy_s (en plus, c'est plus lourd qu'un strcpy).

Il existe un emplacement dans les propriétés de ton projet pour désactiver les avertissements. Suffis d'entrer le numéro de l'avertissement pour qu'il n'apparaise plus. Je ne peux pas te dire exactement où car je suis sous Linux actuellement mais je crois que c'est dans l'onglet compilateur.

C++ (@++)

Oups pardons. Petite erreur:

for(i=0;i<50;++i)
D'ailleur, c'est applicable aussi si tu commence à 1 sinon, tu te trouve à lire en dehors du buffer:
for(i=1;i<50;++i)

C++ (@++)