Discussion :

[BestFF]

Salut,

Aujourd'hui en faisant un petit cout, j'ai eu un segmentation fault que j'ai beaucoup de mal à comprendre.

En gros ce que j'ai :

Code :

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

1
2
3
4
5
6
7
8
9
10
float load_avg[3];
float shiftfloat = (float)(1<<SI_LOAD_SHIFT);
struct sysinfo  the_info;
sysinfo(&the_info);
 
load_avg[0] = ((float)the_info.loads[0]) / shiftfloat;
load_avg[1] = ((float)the_info.loads[1]) / shiftfloat;
load_avg[2] = ((float)the_info.loads[2]) / shiftfloat;
 
cout << "load_1:" << load_avg[0] << "\nload_5:" <<  load_avg[1] << "\nload_15:" <<  load_avg[2] << "\n";

Ce cout produit une Segmentation Fault. Je ne comprends pas du tout pourquoi mais si je ne passe pas par un tableau, le problème ne se pose pas :

Code :

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

1
2
3
4
5
6
7
float a, b, c;
 
a = ((float)the_info.loads[0]) / shiftfloat;
b = ((float)the_info.loads[1]) / shiftfloat;
c = ((float)the_info.loads[2]) / shiftfloat;
 
cout << "load_1:" << a << "\nload_5:" <<  b << "\nload_15:" <<  c << "\n";

Note : Ce problème ne se pose pas sur toutes les machines, mais s'il arrive, c'est en permanence.

Bref, je voulais savoir si quelqu'un avait une explication concernant ceci ? ^^

Merci.

[LittleWhite]

Vérifie que sys_info ne te renvoie pas une erreur ... on ne sait jamais, car s'il renvoie une erreur il se peut qu'il ne mette pas de valeurs dans le tableau. ( et donc ne pas l'alloué ... )

En fait je ne sais pas trop d'ou cela peut venir. Peut être une division par zéro , mais cela ne peut pas être à cause que cela renverrai une erreur illegal operation ...

[loufoque]

Si tu veux qu'on puisse tester et te dire d'où ça vient, fournis un code minimal qui compile, qu'on peut simplement copier et envoyer à notre compilateur, et qui a le problème.

Sinon c'est simple, tu utilises valgrind et tu verras clairement où est le problème.

[Noxen]

C'est en lisant quel indice du tableau que la segfault se produit ?

[BestFF]

Vérifie que sys_info ne te renvoie pas une erreur ... on ne sait jamais, car s'il renvoie une erreur il se peut qu'il ne mette pas de valeurs dans le tableau. ( et donc ne pas l'alloué ... )

Si sysinfo renvoyait une erreur, le fait de passer par des variables intermédiaires devrait toujours entraîner une segmentation fault, non ?
De plus, c'est le cout qui produit la segmentation fault, pas le calcul des valeurs du tableau load_avg.

Pour le code minimal :

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
#include <sys/sysinfo.h>
#include <iostream>
 
using namespace std;
 
int main()
{
    float load_avg[3];
    float shiftfloat = (float)(1<<SI_LOAD_SHIFT);
    struct sysinfo  the_info;
    sysinfo(&the_info)
 
    load_avg[0] = ((float)the_info.loads[0]) / shiftfloat;
    load_avg[1] = ((float)the_info.loads[1]) / shiftfloat;
    load_avg[2] = ((float)the_info.loads[2]) / shiftfloat;
 
    cout << "load_1:" << load_avg[0] << "\nload_5:" <<  load_avg[1] << "\nload_15:" <<  load_avg[2] << "\n";
 
    return 0;
}

En ré-exécutant, le code, je n'arrive plus à reproduire le problème donc je ne peux pas voir pour valgrind. Mais j'avais essayé avec gdb, mais il ne savait pas ce qui déclenchait la segmentation fault.

Sinon, j'avais essayé de séparer les 3 cout, et l'erreur se produisait dés le premier.

[koala01]

Si c'est au niveau du cout que l'erreur survient, c'est que tu corromps la mémoire avant d'y arriver, donc, vraissemblablement dans la seule partie que tu ne nous a pas donnée: la fonction sysinfo(sysinfo*) (un constructeur quelconque ), étant donné que rien dans le code présenter ne semble la justifier, du moins si on considère comme juste la valeur de SI_LOAD_SHIFT (que l'on ne peut pas controler ici)

Il ne serait pas particulièrement surprenant que l'erreur vienne d'un pointeur mal initialisé

Mais, là, sans le code de cette fonction, il nous est difficile d'être plus précis

Au fait... En C++, le fait de définir une structure, une classe, une énumération ou une union déclare automatiquement le type concerné...

Il n'est donc pas nécessaire de repréciser le mot clé struct lorsque tu souhaite déclarer une variable du type souhaité

La ligne de code

Code :

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

     struct sysinfo  the_info;

peut donc avantageusement être remplacée par

Code :

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

    sysinfo  the_info;

[loufoque]

En ré-exécutant, le code, je n'arrive plus à reproduire le problème donc je ne peux pas voir pour valgrind.

L'intérêt de valgrind, c'est que même si le problème ne génère pas d'erreur, tu le vois...
Donc ce que tu dis n'a pas de sens.