Erreur de segmentation sur une concaténation

Version imprimable

Bonjour,

Pour mon interface GTK, j'ai besoin de générer un nom de fichier du genre "aammjjliste.txt" avec aa : numéro de l'année en cours, mm : numéro du mois en cours, jj : numéro du jour en cours.
Tout se passe bien lors de la création de la chaîne aammjj, mais dès que j'essaie d'ajouter la partie "liste.txt", j'ai une SegFault.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 #include <stdio.h> #include <time.h> int main(void) { time_t Timbre_Date; struct tm * Date_Jour; int Annee; int Mois; int Jour; int Heure; char *Date_Compactee; char *Nom_Fichier; Timbre_Date = time(NULL); Date_Jour = localtime(&Timbre_Date); Annee = (Date_Jour-> tm_year-100); Mois = (Date_Jour -> tm_mon); Jour = (Date_Jour -> tm_mday); Heure = (Date_Jour -> tm_hour); /* Affiche la date et l'heure courante (format francais) */ printf("Annee : %02u\n Mois : %u\n Jour : %u\n Heure : %u\n", Annee, Mois, Jour, Heure); sprintf(Date_Compactee, "%02u%02u%02u%02u", Annee, Mois, Jour, Heure); printf("Date compactée : %s", Date_Compactee); sprintf(Nom_Fichier, "%sListe.txt", Date_Compactee); return 0; }
Pourriez-vous m'indiquer où je fais l'erreur ?
Merci beaucoup

13/08/2009, 13h05
Médinoc

Le pointeur Date_Compactee ne pointe sur rien du tout.
Tu ne peux pas faire un sprintf() sur du vide comme ça, du moins pas en C standard.
13/08/2009, 13h12
Mimidoux

Bonjour.

je pense qu'il faut allouée de la mémoire avec malloc()

si tu alloues pas la mémoire il veut remplir le texte dans date_compact qui est un pointeur non initialisé. sprintf() écrit alors dans un endroit ou il n'a surement pas le droit d'écrire. => segfault.

Par contre, en utilisant les bonnes fonctions, on doit pouvoir faire ce que tu veux.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
/* Version avec va_list */
char * alloc_vsprintf(char const *format, va_list args)
{
	size_t longueur;
	char* ret = NULL;
 
	/* 1. Calcul de la taille à allouer */
	#ifdef _MSC_VER
	/* Version Microsoft: snprintf() n'existe pas */
	longueur = _vscprintf(format, args);
	#else
	/* Version utilisant snprintf() */
	longueur = vsnprintf(NULL, 0, format, args);
	#endif
 
	/* 2. Allocation de mémoire */
	ret = malloc((longueur+1) * sizeof *ret);
	if(ret != NULL)
	{
		/* 3. Écriture */
		vsprintf(ret, format, args);
	}
	return ret;
}
 
/*Version avec arguments*/
char * alloc_sprintf(char const *format, ...)
{
	char * retVal;
	va_list args;
	va_start(args, format);
	retVal = alloc_vsprintf(format, args);
	va_end(args);
	return retVal;
}

Ne pas oublier de faire un free() sur le pointeur retourné quand tu n'en as plus besoin...

Citation:
Envoyé par Médinoc
Code:

1 2 3 4 5 6 7 8 9 /* 1. Calcul de la taille à allouer */ #ifdef _MSC_VER /* Version Microsoft: snprintf() n'existe pas */ longueur = _vscprintf(format, args); #else /* Version utilisant snprintf() */ longueur = vsnprintf(NULL, 0, format, args); #endif
En fait, parce que je sens déjà la question venir, la CRT (MS) contient la fonction _snprintf et semblables mais elle c'est juste un sprintf avec un argument buf_size en plus. Ce n'est pas une vraie implémentation de snprintf. C'est pourquoi il faut utiliser _vscprintf(format, args) à la place de vsnprintf(NULL, 0, format, args).

13/08/2009, 17h50
cypher.sephiroth
Bon...
Et sinon, une simple modification du genre :
Code:

1 2 char Date_Compactee [6]; char Nom_Fichier [20];
ça peut suffire ?
13/08/2009, 18h05
Médinoc

Déjà, c'est huit caractères et non six, mais oui, si les données d'entrées sont vérifiées et ne dépassent jamais la taille limite, ça passe.

Mais je trouve qu'allouer la bonne longueur à coup sûr est plus sécurisant.
13/08/2009, 18h59
cypher.sephiroth

Citation:

Envoyé par Médinoc

Déjà, c'est huit caractères et non six, mais oui, si les données d'entrées sont vérifiées et ne dépassent jamais la taille limite, ça passe.

Ben c'est sûr que le jour où les mois feront plus de 99 jours et les années plus de 99 mois, ça pourra être problématique.
13/08/2009, 20h25
gl

Citation:

Envoyé par Médinoc

Déjà, c'est huit caractères et non six

9 caractères avec le 0 terminal.
14/08/2009, 11h20
Médinoc

Citation:

Envoyé par cypher.sephiroth

Ben c'est sûr que le jour où les mois feront plus de 99 jours et les années plus de 99 mois, ça pourra être problématique.

Pour être franc, je n'avais pas vu que l'entrée venait de localtime() :oops:. Je pensais à un fichier ou une saisie utilisateur...
16/08/2009, 13h58
cypher.sephiroth

Citation:

Envoyé par Médinoc

Pour être franc, je n'avais pas vu que l'entrée venait de localtime() :oops:. Je pensais à un fichier ou une saisie utilisateur...

Y'a pas de mal, il m'arrive aussi de lire en diagonale, parfois.
La version avec les va_list me semble intéressante, je l'analyse demain.

Bonne journée !

J'ai trouvé une traduction pour mon problème :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
 
char *concat (int Nombre_Args, ...)
{
    va_list Liste_A_Traiter;
    int Compteur_Args;
    size_t Taille_Chaine;
    char *Resultat_Concatene;
 
    va_start(Liste_A_Traiter, Nombre_Args);
    for (Taille_Chaine = Compteur_Args = 0; Compteur_Args < Nombre_Args; Compteur_Args++)
            Taille_Chaine += strlen(va_arg(Liste_A_Traiter, char *));
    va_end(Liste_A_Traiter);
 
    Resultat_Concatene = (char *) malloc(Taille_Chaine + 1);
    if (NULL != Resultat_Concatene)
    {
        va_start(Liste_A_Traiter, Nombre_Args);
        strcpy(Resultat_Concatene, "");
        for (Compteur_Args = 0; Compteur_Args < Nombre_Args; Compteur_Args++)
            strcat(Resultat_Concatene, va_arg(Liste_A_Traiter, char *));
        va_end(Liste_A_Traiter);
    }
 
    return Resultat_Concatene;
}

17/08/2009, 11h05
Médinoc

Essaie d'en faire une version qui prend directement une va_list en paramètre.
Pour la parcourir deux fois, tu devrais pouvoir faire une copie de ladite va_list...
18/08/2009, 14h48
cypher.sephiroth

Citation:

Envoyé par Médinoc

Essaie d'en faire une version qui prend directement une va_list en paramètre.

C'est-à-dire ? Là, je ne vois pas grand-chose.

Citation:

Envoyé par Médinoc

Pour la parcourir deux fois, tu devrais pouvoir faire une copie de ladite va_list...

Est-ce que ça ne serait pas trop gênant au niveau consommation des ressources ?
18/08/2009, 18h42
Médinoc

Une va_list généralement est un simple pointeur, donc au niveau copie ça ne bouffe rien.