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]);

[DaZumba]

Code :

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

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

malloc(strlen(argv[1]) + 1);

[OlivierDotre]

Bonjour et merci pour vos réponses,

je ne connaissais pas strdup

J'ai donc testé mais il me renvoi ceci comme erreur

Code :

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

1
2
3
src/main.c: In function «main":
src/main.c:49: attention : implicit declaration of function «strdup"
src/main.c:49: attention : incompatible implicit declaration of built-in function «strdup"

Hors j'inclus bien la string.h

Qu'est ce qui m'échappe?

Merci

Olivier

[nicolas.sitbon]

as-tu tenté de compiler avec l'option -std=gnu99 ?
De plus, un petit conseil comme ça, utilise sunstudio(12) ça te permettra de développer avec un compilateur c99 full compliance ce qui n'est pas négligeable quand même autant en profiter.
Cordialement.

[Emmanuel Delahaye]

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]);

Et le 0 final ?

[Emmanuel Delahaye]

je ne connaissais pas strdup

J'ai donc testé mais il me renvoi ceci comme erreur

Code :

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

1
2
3
src/main.c: In function «main":
src/main.c:49: attention : implicit declaration of function «strdup"
src/main.c:49: attention : incompatible implicit declaration of built-in function «strdup"

Hors j'inclus bien la string.h

Qu'est ce qui m'échappe?

Déjà, il faut inclure <string.h>. Ensuite, comme ce n'est pas une fonction standard du C, mais POSIX.1, il faut sans doute activer le mode POSIX. Pour cela, consulter la doc de ton compilateur.

Par exemple, avec gcc/Linux, on ajoute :

Code :

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

-D__POSIX_C_SOURCE=200112L

(2 underscores)

sur la ligne de commande du compilateur.

Enfin, si ton implémentation du C ne connait pas strdup(), il faut l'implémenter toi même, et ce n'est pas difficile.

Voici la définition officielle (POSIX.1) :

http://www.opengroup.org/onlinepubs/...ns/strdup.html

[OlivierDotre]

Ce qui est bizarre c'est que j'utilise strdup dans un autre fichier (contenant plein de fonctions d'aide pour mon fichier principal) et que là il fonctionne. Sinon j'avais bien inclut string.h