Discussion :

[cypher.sephiroth]

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 :

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
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

[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.

[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.

[Médinoc]

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

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
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...

[Melem]

Code :

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

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).

[cypher.sephiroth]

Bon...
Et sinon, une simple modification du genre :

Code :

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

1
2
char Date_Compactee [6];
char Nom_Fichier [20];

ça peut suffire ?

[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.