Erreur de segmentation (encore. . .)

Version imprimable

Bonjour à tous,

J'ai lu le post en dessous, mais je ne trouve pas mon bonheur dedans... J'ai fait tous les sites reprennant les infos sur les chaine de caractères et j'en peux plus, parce que j'ai l'impression de respecter tout ce qu'il faut... Mais ca me le fait toujours... :bug: Donc j'en viens à vous appeler au secours! :oops:

J'ai tenté d'appliquer les remarques que l'on m'avait conseillées dans les posts précédents. Mais rien n'y fait alors que je suis reparti à zéro.

En gros, mon but est de créer un nom de fichier dans le répertoire de l'application. Ce nom de fichier doit être aléatoire.

Je vous transmets mon code:

Récupération du nom de fichier : (OK)
Code:

1 2 3 4 5 6 7 8 9 10 11 12 char * filename; char * getFilename() { return filename; } void setFilename(char * s_Filename) { filename = s_Filename; }
Récupération du répertoire courant : (OK)
Code:

1 2 3 char s_Directory[100]; GetCurrentDirectory(sizeof(s_Directory),s_Directory);
Création nom de fichier : OK : Lorsque je définis moi-même les champs, ca fonctionne. Il me crée bien mon fichier dans c:/test/fichier21229.txt
Code:

1 2 3 4 5 6 7 8 9 10 11 12 srand(time(NULL)); const char * s_Vol= "c:"; const char * s_Dir = "test"; const char * s_Filename= "fichier"; const char * s_Ext= ".txt"; char * s_Path = malloc (sizeof (*s_Path) * 1024); sprintf(s_Path, "%s\\%s\\%s%d%s", s_Vol, s_Dir,s_Filename,rand(),s_Ext); setFilename(s_Path);
Création nom de fichier : (NOK) : Par contre, lorsque j'utilise mon s_Directory, il m'envoie péter... Alors que lorsque j'affiche mon s_Path, il est bien : c:/test/fichier21229.txt
Code:

1 2 3 4 5 6 7 8 9 10 11 12 srand(time(NULL)); const char * s_Filename= "fichier"; const char * s_Ext= ".txt"; char * s_Path = malloc (sizeof (*s_Path) * 1024); sprintf(s_Path, "%s\\%s%d%s",s_Directory,s_Filename,rand(),s_Ext); setFilename(s_Path); printf("%s",getFilename()); // Affiche c:/test/fichier21229.txt
J'espère que vous y verrez plus clair que moi.

Bonjour,

Déjà

Code:

1
2
3
4
5
6
7
8
9
10
11
12
char * getFilename() 
{
    /* attention, ici on renvoie un un pointeur modifiable, 
        est ce bien le comportement voulu ? */
    return filename;     
}
 
void setFilename(char * s_Filename)
{
    /* NON, une copie de chaine se fait avec str[n]cpy */    
    filename = s_Filename;    
}

Citation:

il m'envoie péter

Qui ça ?? (quel est le message d'erreur ?)
Chez moi, le code fonctionne :

Code:

1
2
3
4
>gcc -Wall -Wextra  08-10-22.c
 
>./a.exe
c:\test\fichier1784060792.txt

22/10/2008, 13h42
WaKaaN
Qu'entends-tu par :

Citation:

/* attention, ici on renvoie un un pointeur modifiable,
est ce bien le comportement voulu ? */

De plus, lorsque je fais :
Code:

1 2 3 4 void setFilename(char * s_Filename) { strcpy(filename,s_Filename); }
J'ai une erreur de mémoire alors que ca fonctionne avec le filename = s_Filename.

Sinon, les erreurs qu'il m'envoie sont des erreurs de mémoire ne pouvant être read... (Sur windows)
22/10/2008, 13h43
WaKaaN

Mais le but est que le fichier texte se crée dans le même répertoire que l'executable...
22/10/2008, 14h12
Invité(e)

Citation:

Envoyé par WaKaaN

J'ai une erreur de mémoire alors que ca fonctionne avec le filename = s_Filename.

Oui car pour appeler la fonction strcpy(dst, src), il faut qu'il y ai suffisament de place allouée dans dst.

Quand je dis

Citation:

attention, ici on renvoie un un pointeur modifiable,
est ce bien le comportement voulu ?

Ça veut dire que la fonction getFilename n'est pas sure.

Si je tape

Code:

getFilename()[0] = 'D'

je change le disque sans avoir demandé à modifer la variable filename.

Citation:

Sinon, les erreurs qu'il m'envoie sont des erreurs de mémoire ne pouvant être read... (Sur windows)

As tu fait une execution pas à pas pour voir exactement d'où cela venait ?

Citation:

Mais le but est que le fichier texte se crée dans le même répertoire que l'executable...

Pas besoin de s'embêter avec GetCurrentDirectory. Si on tape fopen("toto", "w"); le fichier toto sera créé dans le répertoire courant.
22/10/2008, 14h54
WaKaaN

Comment rendre ma fonction getFilename sure alors?

Citation:

Envoyé par WaKaaN

Comment rendre ma fonction getFilename sure alors?

Minimum syndical :

Code:

1
2
3
4
const char *getFilename()
{
    return filename;
}

Solution un peu plus évoluée :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
/**
 * accession à filename
 * @param[out] buffer: buffer où écrire le nom du fichier
 * @param[in] buf_size: taille du buffer, /!\ '\0' final INCLUS
 * @return 0 si OK
 */
int getFilename(char * buffer, size_t buf_size)
{
    int err = 0;
    size_t size = strlen(filename);
    if(buf_size < (size+1)) {
        /* le buffer de destination n'est pas assez grand */
        err = 1;
    } else {    
        /* buf_size - 1 pour être sur que buffer se termine par '\0', 
            cf man strncpy*/
        strncpy(buffer, filename, buf_size - 1);
    }
 
    return err;
}