通过指向struct

时间:2016-05-31 13:45:34

标签: c++ pointers struct valgrind

结构定义如下:

struct section_{
    int start;
    ...
};

由于我不会进入的原因,我需要将指向结构的指针传递给接受void*的函数。该函数如下所示:

void* my_fun(void* sec){
    section_* section = (section_*)sec;
    int start = section->start; // <---- valgrind complains here
    ...
}

我有std::vector<section_*>,我需要在此向量的每个元素上调用my_fun。我是这样做的:

std::vector<section_*> sections = get_sections();
for (int i = 0; i < sections.size(); ++i){
    my_fun((void*)sections[i]);
}

get_sections()函数类似于:

std::vector<section_*> get_sections(){
    std::vector<section_*> sections;
    section_ sec1;
    sec1.start = 0;
    ...
    sections.push_back(&sec1);
    return sections;
}

我已将问题跟踪到my_fun中说明

的行
int start = section->start;

错误说:

==3512== Invalid read of size 4
==3512==    at 0x41A970: my_fun(void*)
...
==3512==  Address 0xffeffa2a0 is on thread 1's stack
==3512==  14160 bytes below stack pointer

即使我收到无效的读取,我仍然可以访问my_fun内的结构成员,它们是正确的值。那是为什么?

我意识到这段代码很少,但实际的代码比我展示的要复杂得多,但我认为我正在展示所有相关的部分。我希望我提供的信息足够了。

1 个答案:

答案 0 :(得分:2)

如@BoPersson的评论中所述,您可以向向量添加局部变量:

std::vector<section_*> get_sections(){
    std::vector<section_*> sections;
    section_ sec1;    // <- Local, temporary variable 
    sec1.start = 0;
    ...
    sections.push_back(&sec1); // <- Address of local var
    return sections;
    // When this function ends, sec1 is no longer valid
}

您可以使用new创建sec1(稍后再delete)。或者将矢量类型更改为std::vector<section_> sections;