Discussion :

[melw67]

Bonjour à tous,

Dans le cadre d'un projet pour mes études, je dois créer ma propre commande tar. Pour ce faire, j'ai besoin de convertir un entier en date.

J'ai le nombre de seconde depuis le 1 janvier 1970 (timestamp) en char* et en octal que je converti ensuite en long long et en décimal. Je transforme ensuite ce nombre de type long long en date à l'aide de la fonction ctime mais je reçois à chaque fois une erreur de type "malloc() : memory corruption".

Ci-dessous se trouve le code, si quelqu'un a une idée sur la provenance de l'erreur ou une amélioration de mon code, je suis preneuse

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
 
//Convertir un octal en décimal (char* --> long long)
long long octalToLongDecimal(char* octalNumber)
{
  long long decimal = 0;
  int iterator = 0;
  long long octal = atoll(octalNumber);
 
  while(iterator < 12)
  {
    decimal = decimal + (octal % 10) * pow(8,iterator++);
    octal = octal/10;
  }
 
  return decimal;
}
 
char* getDate(char mtime[12])
{
  long long seconds = octalToLongDecimal(mtime);
  time_t time = (time_t)seconds;
  char *p = ctime(&time);
  return p;
}

Merci.

[kwariz]

Bonsoir,

Je vais supposer que tu es sous linux.
En général un message "memory corruption" si tu utilises la glibc signifie simplement soit que tu écris dans de la mémoire non allouée, que tu continues d'accéder à de la mémoire déjà libérée, de libérer de la mémoire déjà libérée ou de libérer de la mémoire qui n'a pas été mallocée.
Ton code et tes symptômes semblent indiquer que c'est ce dernier cas qui te concerne.
Je suppose (mais il faudrait voir un peu plus de code) que tu essayes de faire un free sur la chaîne renvoyée par ton getDate. Cette chaîne provient de ctime qui renvoie une chaîne statique = de la mémoire non allouée par malloc qui est réutilisée à chaque appel de ctime.

Ci-dessous se trouve le code, si quelqu'un a une idée sur la provenance de l'erreur ou une amélioration de mon code, je suis preneuse
[...]
Merci.

Comme amélioration, je te propose d'utiliser strtoll plutôt que atoll. Cela te permettra non seulement de repérer si la conversion échoue, mais aussi de spécifier une base pour la conversion. Utiliser pow n'est pas non plus recommandé, il vaut mieux pour des conversions de bases utiliser des multiplications cumulées (algorithme de horner).
Et finalement dupliquer la chaîne renvoyée par ctime.

[orfix]

Salut,

Code :

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

...je reçois à chaque fois une erreur de type "malloc() : memory corruption".

Dans ce cas l'erreur est ailleurs... il faut nous montrer le reste du code.

Sinon je te suggère de remplacer atoll par strtoll, cela t'évitera de faire la conversion octal -> decimal, en gros tu n'as plus besoin de octalToLongDecimal():

Code :

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

1
2
3
4
5
 
<...>
//long long seconds = octalToLongDecimal(mtime);
long long seconds = strtoll(mtime, NULL, 8);
<...>

Edit: La réponse de kwariz est plus complète

[melw67]

Bonjour,

Merci pour vos réponses ! Je vais remplacer atoll par strtoll, c'est en effet bien plus simple.

En ce qui concerne le reste du code, je n'utilise aucun malloc ou free, je fais juste un printf sur la valeur reçue... C'est pourquoi je ne comprends pas d'où provient l'erreur.

[kwariz]

Bonjour,

poste ton code en entier pour que nous puissions mieux t'aider.

[melw67]

Bonjour,

Mon problème est résolu, merci pour votre aide. Cela n'avait rien à voir avec la fonction "ctime" mais avec la fonction que j'appelais après où j'allouais mal la mémoire, merci à vous.