Ошибка сегментации при изменении значения атрибута

Question

Я работаю в классе C++ 11, который будет получать через I2C значение температурного датчика в малиновом Pi. Он будет опросить значение до тех пор, пока оно не будет остановлено. Он выполняет опрос в отдельном потоке, так что он не останавливает поток приложения. Проблема заключается в том, что в строке 64 этого файла: https://github.com/OpenStratos/server/blob/feature/temperature/temperature/Temperature.cpp#L64

void Temperature::read_temperature() 
{ 
    while (this->reading) 
    { 
     #ifndef OS_TESTING 
      int value = wiringPiI2CRead(this->filehandle); 
     #else 
      int value = 16000; 
     #endif 

     float voltage = value * 5/32768; // 2^15 
     float temp = r_to_c(TEMP_R * (TEMP_VIN/voltage - 1)); 
     this->temperature = temp; // Gives segmentation fault 

     this_thread::sleep_for(chrono::milliseconds(50)); 
    } 
} 

это дает ошибку сегментации. Любопытство заключается в том, что это не всегда происходит. После компиляции, запуск двоичного файла много раз примерно в 75% случаев будет сбой.

Это файл, который invoques код: https://github.com/OpenStratos/server/blob/feature/temperature/testing/temperature_test.cpp

Temperature temp(20); 
temp.start_reading(); 
AssertThat(temp.is_reading(), Equals(true)); 

// this_thread::sleep_for(chrono::milliseconds(100)); if uncommented less segmentation faults 

temp.stop_reading(); 
AssertThat(temp.is_reading(), Equals(false)); 

Что могло случиться? Как это можно исправить?

Answer 1

Вы должны ждать Temperature::read_temperature(), чтобы бросить курить, так что вам нужно:

bool reading; 
volatile bool stopped; // volatile required to make the compiler re-read 
         // the value everytime we expect it to. 
// 
bool is_stopped(){ return stopped; } 

и

void Temperature::start_reading() 
{ 
    if (!reading) 
    { 
     stopped = false; 
     reading = true; 
     // etc 

и

void Temperature::read_temperature() 
{ 
    while (this->reading) 
    { 
    // etc 
    } 
    stopped=true; 
} 

и

temp.stop_reading(); 
while(!temp.is_stopped(); 
AssertThat(temp.is_reading(), Equals(false));