Discussion :

[OlivierDotre]

Bonjour, je dois faire un programme qui tourne sous solaris,

Je l'ai fait sur mon pc, tout va bien et il fonctionne, mais quand je veux le migrer sur la solaris, mais lors de l'exécution,il me fait des erreurs de segmentation, alors que sous linux pas.

Quelle différence y a t'il?

Merci

Olivier

[crocodilex]

Sans le code, nous ne pouvons rien dire....

[gl]

Bonjour, je dois faire un programme qui tourne sous solaris,

Je l'ai fait sur mon pc, tout va bien et il fonctionne, mais quand je veux le migrer sur la solaris, mais lors de l'exécution,il me fait des erreurs de segmentation, alors que sous linux pas.

Quelle différence y a t'il?

Tu as un comportement indéterminé dans ton code (je pencherais pour un débordement de buffer) qui passe, pour l'instant, inaperçu sous Linux et qui, coup de chance, plante sous Solaris.

[OlivierDotre]

Désolé pour le retard,

Mais voici le code en question

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
 
int main (int argc, char *argv[]){
 
 
	print_terminal("Main","Starting...");
	/* Check Arguments */
	if (argc != 3) 					error_exit("2 argument required for cracking");
	the_crack = malloc(sizeof(argv[1]));
	the_crack = argv[1];
	the_salt = malloc(sizeof(char *));
	the_salt = get_salt(the_crack);
	printf("The crack is %s\n",the_crack);
	//printf("The longueur of the word %i\n",the_length);
	printf("The salt is %s\n",the_salt);
	cracking();
	return 1;
}

Il met l'erreur avant d'afficher "The crack is ..." Je ne sais pas vraiment ce qu'il faut changer. J'appelle main avec comme paramètre un code crypter dont il faut retrouver la valeur.

Merci

Olivier

[gl]

Outre le fait que l'on ne sait pas ce que font les fonctions get_salt() et crack(), voici deux-trois remarques :

* Aucune verification sur le resultat de malloc, en cas d'erreur d'allocation ca va mal se passer.
* Il manque la liberation des zones memoires allouees.
* sizeof(argv[1]) ne fournit pas la taille de la chaine mais la taille d'un pointeur.
*

Code :

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

1
2
the_crack = malloc(sizeof(argv[1]));
the_crack = argv[1];

Ici tu modifies le pointeur apres allocation. La zone allouee n'est plus reference ==> fuite memoire.

[OlivierDotre]

Ok, merci, je vais tacher de résoudre tout ça.

Je suis débutant en C, faut bien commencer un jour:d

[pascal.barbier]

Je pense que le problème est effectivement dans :

the_crack = malloc(sizeof(argv[1]));
the_crack = argv[1];

Je suppose que le but est de recopier la chaine reçue en argument de la ligne de commande dans la variable the_crack.

Dans ce cas le plus simple est

Code :

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

the_crack = strdup(argv[1]);

Sinon si on veut utiliser malloc, il faut écrire :

Code :

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

1
2
the_crack = malloc(strlen(argv[1]));
strcpy(the_crack, argv[1]);