[C] System.AccessViolationException Résolu/Fermé

Question

Bonjour, bonsoir,  

De ce que j'ai vu ici : https://stackoverflow.com/questions/4269938/an-unhandled-exception-of-type-system-access-violation-exception-occured-in-h  

Je merdouille avec les tableaux...  

Pourtant je ne fais rien de mal me semble-t-il !  


char * db_registry [size_reg_key][2] = {{"HKEY_LOCAL_MACHINE", "SOFTWARE\Microsoft\Windows\CurrentVersion\Run"},  
          {"HKEY_LOCAL_MACHINE", "SOFTWARE\Microsoft"}};  

fprintf(rapport, "

::::: Registry :::::

");  
    
 HKEY key;  

 printf(db_registry[0][0]);  
 printf(db_registry[0][1]);  

 printf("

");  

 printf(db_registry[1][0]);  
 printf(db_registry[1][1]);  
    
 system("pause");  

// ici, j'obtiens bien ce que j'ai dans mon tableau, mais dès que je sors de la pause, boum, le message de violation.  

 for (i = 0 ; i < size_reg_key ; i++)  
 {  
 char * full_key = strcat(db_registry[i][0], db_registry[i][1]);  

  if (RegOpenKeyExA((HKEY) db_registry[i][0], db_registry[i][1], 0, KEY_READ, &key) == ERROR_SUCCESS)  
  {  
   fprintf(rapport, full_key);  
  }  
 }  


Donc deux questions :

1 - Comment se fait-il qu'un tableau commence à 0 ??  

2 - Qu'est-ce qui provoque cette erreur, car j'ai le même soucis dans un autre morceau de code...  

Merci à vous !  


Edit:: Capture: http://puu.sh/1bnwY

Configuration: Linux : Debian, soyez libre !   
   
&#9834;&#9834; Slowin' Cleaner development, Windows Optimizer &#9835;

fiddy · Answer

1 - Comment se fait-il qu'un tableau commence à 0 ?? 
Ben, ça toujours été comme ça. Et c'est logique :
toto[0] signifie toto+0, on ne perd pas de case mémoire.

printf(db_registry[0][0]);
Ecriture très dangereuse. Mets plutôt : printf("%s",db_registry[0][0]);
 
char * full_key = strcat(db_registry[i][0], db_registry[i][1]);
Aïe.
db_registry[i][j] est de type char* (c'est-à-dire pointeur sur une zone en lecture seule). Autrement dis, tu cherches à écriture dans une lecture en lecture seule. Et là, ça passe pas.
Utilise plutôt un malloc au lieu du char*. Et alloue une zone suffisamment grande.

Utilisateur anonyme · Answer

Coucou Fiddy,

 
printf(db_registry[0][0]);
Ecriture très dangereuse. Mets plutôt : printf("%s",db_registry[0][0]); 


Pourquoi ?
 

Ben, ça toujours été comme ça. Et c'est logique :
toto[0] signifie toto+0, on ne perd pas de case mémoire. 


toto[0] ne vaut pas le nombre de lignes ?


db_registry[i][j] est de type char* (c'est-à-dire pointeur sur une zone en lecture seule)


Peux-tu m'expliquer cela ?

char * = C'est un pointeur (zone en mémoire) vers une chaîne de caractères, elle n'est pas modifiable ?

Et je le place comment mon malloc ?

Qu'est-ce que je dois mettre en mémoire ?

Pas de sizeof() pour le malloc ?

Merci de ta réponse :-)

Utilisateur anonyme · Answer

 Avec ce genre de code, un pirate pourrait s'amuser à lire des zones en mémoire. Par exemple si db_registry[0][0] contient des %s ou autre.

Puis-je avoir un peu plus d'explication là dessus ? Il lira le contenu de mon: db_registry ?

</code> malloc(sizeof(db_registry));</code>

ça ça ne marche pas ? J'aimerais un truc du genre pour ne pas avoir à déclarer un buffer trop grand ou trop petit, ce n'est pas "propre". :-)

Et je le place comment mon malloc ?
Par exemple, char * db_registry [size_reg_key][2]=malloc(255); ou alors db_registry [size_reg_key][2][255] =

Désolé rien compris,

Merci de ton aide,

Utilisateur anonyme · Answer

Huuummmm, d'accord !

J'ai compris, le pointeur et la mémoire :-)

Cependant, est-ce là même chose ici avec DWORD ?


#define size_files 2

char * db_files [size_files] = {"C:\force_delete.txt", 
								"C:\ComboFix.txt"};

DWORD Attributes;

for (i = 0 ; i < size_files ; i++)
{
	if (_access(db_files[i], 0) != -1)
	{
		hFile = CreateFileA((LPCSTR) db_files[i], GENERIC_READ, FILE_SHARE_READ, NULL, OPEN_EXISTING, FILE_ATTRIBUTE_NORMAL, NULL);

		if (GetFileTime(hFile, &ftCreate, &ftAccess, &ftWrite))
		{
			// Convert the created time to local time. set +1

				FileTimeToSystemTime(&ftCreate, &stUTC);
						SystemTimeToTzSpecificLocalTime(NULL, &stUTC, &stLocal);

				Attributes = GetFileAttributesA(db_files[i]);

			fprintf(rapport, "0x%x - [%02d/%02d/%d %02d:%02d] - %s
", Attributes, stLocal.wDay, stLocal.wMonth, stLocal.wYear, stLocal.wHour, stLocal.wMinute, db_files[i]);
		}
		else
		{
			fprintf(rapport, "[error: %d] : %s
", GetLastError(), db_files[i]);
		}

		CloseHandle(hFile);   
	}
}

Utilisateur anonyme · Answer

Bonjour,

J'ai préféré te donner tout le code, il n'y a pas grand chose:

https://pjjoint.malekal.com/files.php?read=20121006_z14u8k12n14g9

Sachant aussi que j'ai un autre sujet en cours:

 https://forums.commentcamarche.net/forum/affich-26195289-c-handle

Pour pouvoir créer un handle sur un dossier :-)

Deux sujets car deux problèmes différents,

Merci ;-)

Utilisateur anonyme · Answer

D'accord merci pour ces réponses, je progresse petit à petit,

Pour sptrinf() je pensais que ça affichait le résultat à l'écran un peu comme printf(), mais visiblement non...

Je suppose qu'avec malloc, on peut attribuer juste ce qu'il faut de mémoire ?

Car ici,


char home[255];
	strcpy(home, getenv("HOMEDRIVE"));
	const char * report_path = strcat(home,"\SHZ_LookUp.txt");

 
255 me semble élevé, et d'un autre côté je ne veux pas incrémenter manuellement le bazar jusqu'à la bonne valeur :-)

Aussi, j'ai remarqué qu'il y a erreur lorsque je tente d'afficher une chaîne de caractère mal formatée, par exemple en voulant afficher du DWORD avec %s au lieu de %x...

(si ça peut aider quelqu'un)

Merci, ;-)

Utilisateur anonyme · Answer

D'accord, je vais assimiler tout cela, merci !!

Avant de partir:

	char * home;
	(ligne 43) home = malloc(strlen (getenv("HOMEDRIVE")) + strlen("\SHZ_LookUp.txt") +1);
	strcpy(home, getenv("HOMEDRIVE"));

	const char * report_path = strcat(home, "\SHZ_LookUp.txt"); 

Me renvoie:  

1>Database.cpp(43): error C2440: 'initialisation' : impossible de convertir de 'void *' en 'char *'
1>          La conversion de 'void*' en pointeur vers non 'void' nécessite un cast explicite
 

Et je n'arrive pas à le caster comme il le faut...

Utilisateur anonyme · Answer

Oups, y'a pas mal de choses à modifier et ce n'est pas le top pour moi qui risque de tout péter (j'aime bien trifouiller les réglages).

Je pense laisser ainsi, du toute manière je vais devoir aller vers le C++ un jour prochain alors... :-)

 Merci pour votre aide, j'ai beaucoup appris !
 
A bientôt, ...