Probleme bizard sur les pointeurs en C !

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : Probleme bizard sur les pointeurs en C !

samuelp

Bonjour,

j'ai un probleme bizard en C. J'execute un programme executable en C, compilé donc. Je lui passe 3 paramètres.
Si je depasse un certain nombre de caractere sur le 1 parametre, ça me genere un segfault.
Le plus bizard, c'est qu'en voulant debuguer, j'ai :
printf("TOTO \n" );
ça passe
printf("TOTO" );
segfault !

Voila la fonction :

Code :

int obtenirParametres(int nbParametres, char * tableauParametres[])
{
if (nbParametres < 7)
{
printf("Erreur, le nombre de paramètres doit être d'au moins de 6 \n" );
return (-1);
}
/* Récupération des paramètres du TraitCode128 */
printf("lecture des paramètres\n" );
while(--nbParametres)
{
*++tableauParametres;
switch( *(*tableauParametres+1) & 0xDF)
{
case 'C':
/* Indice de la colonne: -c numéro de la colonne dans le fichier d'index */
*++tableauParametres;
printf("LA \n" );
printf("TERMINE \n" );
printf("TEST3 %s \n",*tableauParametres);
tcIndice=malloc(sizeof *tableauParametres);
printf("OUF \n" );
sprintf(tcIndice,"%s",*tableauParametres);
break;
case 'I':
/* fichier d'index en entrée : -i */
*++tableauParametres;
tcIndex_entree=malloc(sizeof *tableauParametres);
sprintf(tcIndex_entree,"%s",*tableauParametres);
printf("TEST %s \n",tcIndex_entree);
break;
case 'O':
/* fichier d'index en sortie: -o */
*++tableauParametres ;
printf("TEST1 %s \n",*tableauParametres);
tcIndex_sortie=malloc(sizeof *tableauParametres);
printf("TEST1c \n" );
sprintf(tcIndex_sortie,"%s",*tableauParametres);
printf("TEST1 \n" );
break;
default :
/* paramètre inconnu*/
printf("Le paramètre %s n'est pas connu ! \n",*tableauParametres);
return (-1);
break;
}
nbParametres--;
}
return 1 ;
}

nbparametres c'est le argc et tableauParametres le argv de la fonction main.

Quelqu'un aurait une idée ?

Publicité

Taz

bisounours-codeur

*(*tableauParametres+1) & 0xDF)

c'est quoi ça ? mettre en majuscule ? bordel ce que c'est crade. toupper() et on en parle plus

tcIndice=malloc(sizeof *tableauParametres);
bonjour, vous venez d'allouez 4octets ...

tu veux pas utiliser un i comme tout le monde, parce que là sinon, c'est crade, tu incrémentes, tu décréments, tu dereferences, etc ... tu t'y perds

samuelp

Taz a écrit :

En fait c'est du code que j'ai repris, du code "pro" comme ils disent.
c'est quoi le "i" comme tout le monde ?
Sinon, vois tu le probleme ? Des que me parametre 1 fait disons moinds de 10 caracteres, ça passe. Des qu'il depasse les 20 caracteres, segfault

Taz

bisounours-codeur

ben for(i=0; i<nbparam; ++i) { }

le problème ? ben c'est le sizeof te retourn invariablement sizeof(char*)

Taz

bisounours-codeur

tcIndice=malloc(sizeof *tableauParametres);
printf("OUF \n" );
sprintf(tcIndice,"%s",*tableauParametres);

t'es sous linux sam ?

tcIndice = strdup(*tableauParametres);

fera bien mieux le boulot (strdup étant malloc + strcpy)

darkoli

Le Petit Dinosaure Bleu

samuelp a écrit :

...

Code :

...
case 'C':
/* Indice de la colonne: -c numéro de la colonne dans le fichier d'index */
*++tableauParametres;
printf("LA \n" );
printf("TERMINE \n" );
printf("TEST3 %s \n",*tableauParametres);
tcIndice=malloc(sizeof *tableauParametres);
printf("OUF \n" );
sprintf(tcIndice,"%s",*tableauParametres);
break;
...

Il faut aussi vérifier que le paramètre *++tableauParametres existe réellement.

Taz

bisounours-codeur

*++tableauParametres;

moi pas comprendre cette instruction avant le switch

philou_a7

\_o< coin ! >o_/

Taz a écrit :

*++tableauParametres;

moi pas comprendre cette instruction avant le switch

je parie que c'est pour sauter le '-' dans les options...
par exemple, pour le "-C" il regarde "C"

t'as pas getopt ou un equivalent , là ?

fli

Et moi je ne comprends pas non plus pourquoi il y a :
while(--nbParametres)
et à la fin du while nbParametres--;

si nbParametres est impair c'est badaboum

philou_a7

\_o< coin ! >o_/

fli a écrit :

Et moi je ne comprends pas non plus pourquoi il y a :
while(--nbParametres)
et à la fin du while nbParametres--;

si nbParametres est impair c'est badaboum

même ordre d'idée : le nbParametres-- doit etre là pour sauter la valeur du parametre que l'on vient de trouver...

franchement, en ré-ecrivant la fonction tu gagnerais du temps...

Publicité

Taz

bisounours-codeur

philou_a7 a écrit :

je parie que c'est pour sauter le '-' dans les options...
par exemple, pour le "-C" il regarde "C"

t'as pas getopt ou un equivalent , là ?

sauf que ça n'est pas la bonne expression

fli

philou_a7 a écrit :

je parie que c'est pour sauter le '-' dans les options...
par exemple, pour le "-C" il regarde "C"

t'as pas getopt ou un equivalent , là ?

pour sauter le moins c'est dans le switch : *(*tableauParametres+1)

darkoli

Le Petit Dinosaure Bleu

Taz a écrit :

*++tableauParametres;

moi pas comprendre cette instruction avant le switch

Le nom du programme ?
Ben non en fait, c'est dans le switch.
Ben si c'est ça, en fait la premère fois ça saute le nom du programme et ensuite la valeur qui vient d'être prise en compte.

Message édité par darkoli le 19-04-2004 à 17:51:59

samuelp

pour le toupper, ça me fait un warning :
tableauParametres[i]=toupper(tableauParametres[i]);

Je sais pourquoi, mais comment dire au systeme que je m'attend à un caractere ???

philou_a7

\_o< coin ! >o_/

Taz a écrit :

sauf que ça n'est pas la bonne expression

j'avoue que j'ai pas detaillé le code, c'etait la premiere chose qui me venait a l'esprit

darkoli

Le Petit Dinosaure Bleu

samuelp a écrit :

pour le toupper, ça me fait un warning :
tableauParametres[i]=toupper(tableauParametres[i]);

Je sais pourquoi, mais comment dire au systeme que je m'attend à un caractere ???

Le paramètre de la fonction doit être un caractère et pas une chaîne.

TOUPPER(3) Manuel du programmeur Linux TOUPPER(3)

NOM
toupper, tolower - Conversion de lettres en majuscules ou minuscules.

SYNOPSIS
#include <ctype.h>

int toupper (int c);
int tolower (int c);

DESCRIPTION
toupper() convertit la lettre c en majuscule si c'est possible.

tolower() convertit la lettre c en minuscule si c'est possible.

VALEUR RENVOYÉE
La valeur renvoyée est celle de la lettre convertie, ou bien c si la conversion n'etait pas possible.

Si c n'est ni un caractère non-signé, ni EOF, le comportement de ces fonctions est imprévisible.

CONFORMITÉ
ANSI C, BSD 4.3

BOGUES
La définition de "majuscule" et "minuscule" dépend de la localisation. Par exemple la localisation "C" par défaut ne
connait rien concernant les accents, et n'effectue alors aucune conversion.

Dans certaines langues, des lettres minuscules n'ont pas d'équivalent majuscule.

VOIR AUSSI
isalpha(3), setlocale(3), locale(7)

TRADUCTION
Christophe Blaess, 1996-2003.

LDP 21 juillet 2003 TOUPPER(3)

Au pire tu peux toujours écrire :

switch (tableauParametres[i][1])
{
case 'c' :
case 'C' :
...
break;
...
}

Mais bon si le paramètre est vide c'est problematique.

Message édité par darkoli le 19-04-2004 à 22:23:42

Taz

bisounours-codeur

jagstang

Pa Capona ಠ_ಠ

Code :

int obtenirParametres(int nbParametres, char * tableauParametres[])
{
if (nbParametres < 7)
{
printf("Erreur, le nombre de paramètres doit être d'au moins de 6 \n" );
return (-1);
}

mmm... Je lis pas plus loin

Message édité par jagstang le 20-04-2004 à 01:55:49

darkoli

Le Petit Dinosaure Bleu

JagStang a écrit :

int obtenirParametres(int nbParametres, char * tableauParametres[])
{
if (nbParametres < 7)
{
printf("Erreur, le nombre de paramètres doit être d'au moins de 6 \n" );
return (-1);
}
mmm... Je lis pas plus loin

Non c'est bon puisque le nom du programme n'est pas un paramètre même si c'est le premier élément de la liste.

tableauParametres[0]="$chemin$nom_du_programme"
tableauParametres[1]=$paramètre_#1
...

Message édité par darkoli le 20-04-2004 à 09:11:25

christophe_d13

L'efficacité à tout prix.

malloc ( sizeof * xxxx )... Mais bien sur...

Un malloc alloue généralement non pas la taille demandée mais la taille aligné sur 8, 16 voire 32 octets...
Donc ton truc doit marcher avec 15 caractères (16 en comptant le 0 de fin) et pas 16...

Tu croies encore que sizeof renvoi la longueur d'une chaine ??? Retourne voir tes bases...
sizeof est une commande préprocesseur ! Cela veut dire que c'est à la compilation que la valeur est générée et pas pendant l'exécution.

Un code pro... code pourri !
Les post et pré incrémentation c'est pour faire genre je suis trop fort... Dans le genre, je fais ça aussi... a = i?(0): (-2);
Et puis on peut faire ça aussi (DWORD)(*((float *)a)[1])+23.1F)
On peut faire tout ce qu'on veut en C... même des plantages fastoches... *((dword *)(a-a)) = 0;

Message édité par christophe_d13 le 21-04-2004 à 01:20:50

Taz

bisounours-codeur

christophe_d13 a écrit :

malloc ( sizeof * xxxx )... Mais bien sur...

déjà dit

christophe_d13 a écrit :

sizeof est une commande préprocesseur ! Cela veut dire que c'est à la compilation que la valeur est générée et pas pendant l'exécution.

pas en C99

christophe_d13

L'efficacité à tout prix.

Taz> Qu'est ce que j'en sais que c'est en C99 (je ne connaissais même pas son existance...merci Taz et Google, je mourrait moins bête). Mais il a précisé en C.

J'ai déjà vu tellement de programmeur faire un sizeof à la place d'un strlen que ça m'énerve toujours plus à chaque fois.

Désolé, sans rancune.

Message édité par christophe_d13 le 21-04-2004 à 14:34:17

Publicité

FORUM HardWare.fr

Programmation

Probleme bizard sur les pointeurs en C !

Sujets relatifs
Probleme avec les dieses !	[C++] Compilateur C++ .NET 2003 / SDL > Problème à la compilation
[CSS] Probleme d'affichage de taille de police sur IE5	Problème de calcul de dates
Problème ACCESS et Recorsource	Tableau de pointeurs sur fonctons.
[eclipse] probleme de visualisation de page JSP	Problème de page
Probleme de transfert de chaîne de caractere vers BDD.HELP!!!!	[WEB FLASH ] Probléme plubication site flash
Plus de sujets relatifs à : Probleme bizard sur les pointeurs en C !

Page générée en 0.097 secondes