Problème de CRLF et de strcpy

**Otacon** · 16/04/2007, 17h18

Bonjour à tous,

Je développe en ce moment un programme qui accède à des fichiers de configuration grace à une fonction que j'ai trouvée et modifiée un peu et je rencontre un problème :

lorsque le caractère de fin de ligne est un simple LF (\n), pas de problèmes, tout ce passe bien
lorsque le caractère est un CR (\r), la fonction ne reconnait pas la fin de ligne et considère le fichier comme vide
lorsque c'est un CRLF (\r\n) la fonction rajoute un caractère NUL (\0) pour chaque ligne présente dans le fichier et ce à la fin.

Le code de cette fonction est :

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
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
int conf_set_str(const char file[], const char name[], const char value[])
{
    int done = 0, len = 0, found = 0;
    unsigned int i = 0;
    char *buf = NULL, *p = NULL;
    FILE *conf_file = NULL;
 
    if(!file || !name)
    {
        return -1;
    }
 
    if((conf_file = fopen(file, "r")) == NULL)
    {
        return -2;
    }
 
 
    fseek(conf_file, 0L, SEEK_END);
    len = ftell(conf_file);
    fseek(conf_file, 0L, SEEK_SET);
 
    if((p = buf = malloc(len)) == NULL)
    {
        return -3;
    }
 
 
 
    fread(buf, len, 1, conf_file);
 
    while(*p)
    {
        if(*p == '#')
        {
            while(*p != '\n')
            {
                p++, done++;
            }
 
            p++, done++;
            continue;
        }
        else if(*p == name[0])
        {
            for(found = 1, i = 0 ; i < strlen(name) ; i++, p++, done++)
            {
                if(*p != name[i])
                {
                    found = 0;
                    break;
                }
            }
 
            if(found)
            {
                while(isspace(*p))
                {
                    p++, done++;
                }
 
                done++;
 
                if(*p++ != '=')
                {
                    if(buf)
                    {
                        free(buf), buf = NULL;
                    }
 
                    return -4;
                }
 
                while(isspace(*p))
                {
                    p++, done++;
                }
 
                if(*p == '"')
                {
                    *p = '\0';
 
                    freopen(file, "w", conf_file);
 
                    if(fwrite(buf, done, 1, conf_file) != 1)
                    {
                        if(buf)
                        {
                            free(buf), buf = NULL;
                        }
 
                        return -5;
                    }
 
                    *p = '"';
                    fprintf(conf_file, "\"%s\"", value);
 
                    do
                    {
                        p++, done++;
                    }
                    while(*p != '"');
 
                    p++;
 
                    if(len - done > 0)
                    {    
                        if(fwrite( p, len - done, 1, conf_file ) != 1)
                        {
                            if(buf)
                            {
                                free(buf), buf = NULL;
                            }
 
                            return -5;
                        }
                    }
                }
                else
                {
                    if(buf)
                    {
                        free(buf), buf = NULL;
                    }
 
                    return -6;
                }
 
                fclose(conf_file);
 
                if(buf)
                {
                    free(buf), buf = NULL;
                }
 
                return 1;
            }
            else
            {
                while(*p != '\n' && *p != '\0')
                {
                    p++, done++;
                }
            }
        }
 
        while(*p != '\n')
        {
            p++, done++;
        }
 
        p++, done++;
    }
 
    fwrite(p, len - done, 1, conf_file);
    fclose(conf_file);
 
    if(buf)
    {
        free(buf), buf = NULL;
    }
 
    /* Create the parameter if it doesn't exist */
 
    if((conf_file = fopen(file, "a")) == NULL)
    {
        return -2;
    }
 
    fprintf( conf_file, "%s = \"%s\"\n", name, value );
    fclose(conf_file);
 
    return 1;
}

Je l'appelle comme cela :

Code :

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

1
2
3
4
5
6
7
 
int error = 0;
if((error = conf_set_str("nom_du_fichier.conf", "Name", "valeur")) != 1)
{
    fprintf(stderr, "Error while updating name : %i\n", error);
    return -2;
}

Le fichier de config. est sous cette forme :

Code :

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

1
2
# Nom 
Name = "Nom"

S'il y a des CRLF on aura :

Code :

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

1
2
# Nom 
Name = "Nom"\0

(une seule ligne avec le caractère de fin)

-----------------------

J'ai aussi un autre problème concernant une structure cette fois :
Celle-ci est définie comme cela :

Code :

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

1
2
3
4
5
6
typedef struct pet_t
{
    int health_points;
    char *name;
    (d'autres choses sans importance ...)
} pet_t;

J'ai une fonction pour l'initialiser :

Code :

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

1
2
3
4
5
6
void init_pet(pet_t *pet)
{
    pet->health_points = 0;
    pet->name = NULL;
    ...
}

Je fais mes appels comme cela :

Code :

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

1
2
pet_t pet;
init_pet(&pet);

Le problème a lieu lorsque j'essaie de me mettre une valeur à name. Si je fais pet.name = "Le nom"; je me retrouve avec une erreur du compilo : "warning: assignment discards qualifiers from pointer target type". J'ai donc essayé avec strcpy(pet.name, "Le nom"); mais je me retrouve avec un magnifique SIGSEGV et un monumental plantage.

Je vous remercie par avance de l'aide que vous pourrez m'apporter,

Cordialement
Otacon

**Médinoc** · 16/04/2007, 17h24

Tu dois allouer une place pour le nom. Soit un buffer de taille fixe dans ta structure, soit une zone mémoire allouée dynamiquement avec malloc() ou strdup()...

**souviron34** · 16/04/2007, 17h29

quant à la fonction pour trouver les mots-clés et leurs valeurs, quelle horreur .....

Pour un fichier aussi simple.. si c'est pas malheureux de rendre un code simple aussi complexe...

**Otacon** · 16/04/2007, 19h02

Tout d'abord merci pour vos réponses

!

Envoyé par Médinoc

Tu dois allouer une place pour le nom. Soit un buffer de taille fixe dans ta structure, soit une zone mémoire allouée dynamiquement avec malloc() ou strdup()...

D'accord merci, il me semblait bien que j'oubliais quelque chose

Envoyé par souviron34

quant à la fonction pour trouver les mots-clés et leurs valeurs, quelle horreur .....

Pour un fichier aussi simple.. si c'est pas malheureux de rendre un code simple aussi complexe...

Désolé mais comme je l'ai dit la fonction n'est pas de moi j'ai juste viré quelques trucs bizarres mais sinon je ne l'ai pas codé

Envoyé par Emmanuel Delahaye

Il y a 3 façons de traiter ce problème, qui n'est pas un problème de C...

1 - Traitement en amont. Convertir manuellement (ou automatiquement, script...) les fichiers textes au format natifs du système avec les outils traditionnels dos2unix, unix2dos etc. Le programme n'a plus affaire qu'à des fichiers textes 'normaux' (donc traités en mode texte).

2 - Variante du 1 : pré-traitement dans l'application : reconnaitre les terminaisons et convertir le fichier en conséquence en appelant l'outil de conversion qui va bien avec system()...

3 - Traitement dans le programme en mode binaire. On fait l'hypothèse que le fin de ligne est une des 4 formes possibles :

* CR
* LF
* CR/LF
* LF/CR

après avoir reconnu la première forme, on se base sur celle-ci pour déterminer la fin de ligne dans le reste du programme.

Tu veux donc dire que
1) soit je passe par des programmes style sed, unixtodos ... pour faire cela
2) soit je l'ouvre en mode binaire avec rb (ou wb) et que je recherche la première forme du caractère de fin en considérant qu'ils sont tous pareils ?

EDIT : j'ai essayé en ouvrant le fichier en binaire et ça marche la première fois sans problème. Après ça me rajoute ça : un $, un SOH, un ACK, un NUL, un NUL ...

En tout cas merci encore de me depanner

Cordialement
Otacon

**souviron34** · 16/04/2007, 23h52

Je pense plus sérieusement que tu ferais mieux de réfléchor et dd'exposer clairement ton problème, ce serait nettement plus simple..

La fonction que tu as récupérée est complexe et "geek-like" pour un problème simple..

Que veux-tu faire exactement ?

Tu veux lire le fichier de config ?

En tirer les valeurs ? Eventuellement les remodifier et le réécrire c'est ça ?

Les valeurs sont toujours du texte ?

**Otacon** · 17/04/2007, 01h13

Envoyé par souviron34

Je pense plus sérieusement que tu ferais mieux de réfléchor et dd'exposer clairement ton problème, ce serait nettement plus simple..

La fonction que tu as récupérée est complexe et "geek-like" pour un problème simple..

Que veux-tu faire exactement ?

Tu veux lire le fichier de config ?

En tirer les valeurs ? Eventuellement les remodifier et le réécrire c'est ça ?

Les valeurs sont toujours du texte ?

En fait, j'aimerais simplement pouvoir lire un fichier et en tirer des valeurs et les modifier/créer comme tu l'as dit.
Les valeurs seraient soit du texte, soit des valeurs entière (signées) soit des booléens.

Le fichier aurait cette forme par exemple :

Code :

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

1
2
3
4
5
6
7
8
9
# Commentaire
# chaine de caractère entre ""
Nom = "Le nom"
 
# valeur entière signée
Age = 14
 
# booléen
Vivant = TRUE

Bien sûr, il se peu qu'il y ai plus de lignes ^^

Il me faudrait donc : une fonction pour obtenir la valeur associé à un élément et une autre pour mettre à jour/créer des valeurs.

Merci encore une fois par avance

Cordialement
Otacon

**femtosa** · 17/04/2007, 08h46

Envoyé par Otacon

Il me faudrait donc : une fonction pour obtenir la valeur associé à un élément et une autre pour mettre à jour/créer des valeurs.

Et alors, tu y a réfléchi ...

Commence par réfléchir à la façon dont tu vas t'y prendre, fais un algorithme et code-le ! Quelqu'un t'aidera dans une de ces étapes, mais personne les fera pour toi ... !

Si t'as besoin d'aide hésite pas, mais pose une question précise et non un "voilà ce dont j'ai besoin"

**Emmanuel Delahaye** · 16/04/2007, 18h31

Envoyé par Otacon

Je développe en ce moment un programme qui accède à des fichiers de configuration grace à une fonction que j'ai trouvée et modifiée un peu et je rencontre un problème :

lorsque le caractère de fin de ligne est un simple LF (\n), pas de problèmes, tout ce passe bien
lorsque le caractère est un CR (\r), la fonction ne reconnait pas la fin de ligne et considère le fichier comme vide
lorsque c'est un CRLF (\r\n) la fonction rajoute un caractère NUL (\0) pour chaque ligne présente dans le fichier et ce à la fin.

Il y a 3 façons de traiter ce problème, qui n'est pas un problème de C...

1 - Traitement en amont. Convertir manuellement (ou automatiquement, script...) les fichiers textes au format natifs du système avec les outils traditionnels dos2unix, unix2dos etc. Le programme n'a plus affaire qu'à des fichiers textes 'normaux' (donc traités en mode texte).

2 - Variante du 1 : pré-traitement dans l'application : reconnaitre les terminaisons et convertir le fichier en conséquence en appelant l'outil de conversion qui va bien avec system()...

3 - Traitement dans le programme en mode binaire. On fait l'hypothèse que le fin de ligne est une des 4 formes possibles :

CR
LF
CR/LF
LF/CR

après avoir reconnu la première forme, on se base sur celle-ci pour déterminer la fin de ligne dans le reste du programme.

Problème de CRLF et de strcpy

C

Vue hybride

Discussions similaires

Partager

Partager