使用std :: bind进行函数回调会导致分段错误

Question

我有一个类（让我们称之为“Common”），用于创建计时器，并在计时器到期时从另一个类调用成员函数（让我们称之为“A”）。 一般的想法是这样的： - A从Common调用一个函数，传递一个指向成员函数的指针，该函数将在计时器到期时执行 - Common将指针存储在带有计时器id键的映射中的函数中 - 当计时器时命中，搜索映射以查找匹配的键，然后调用相关的函数。

这是A的代码：

void A::setTimer()
{ 
   std::function<void(void)> handler = std::bind(&A::onEventFunc, this);
   pCommon->scheduleTimer(&handler);
}

onEventFunc是A类的公共函数，它在计时器到期时执行某些操作。

以下是Common的代码：

static void timerHandler(int sig, siginfo_t *si, void uc)
{
   map<timer_t*, std::function<void>(void)>*)::iterator iter = 
                                globalTimerMap.find((timer_t*)si->si_value.sival_ptr);
   if (iter != globalTimerMap.end)
   {
      std::function<void>(void)>* handler = iter->second;
      (*handler)();
   }
}

Common::schedulerTimer(std::function<void>(void)* handler)
{
   // uses Linux timer_create and timer_settime
   // code for these excluded EXCEPT the relevant lines
   sa.sa_sigaction = timerHandler;

   // bunch of other code here for timer_create and timer_settime

   // insert into map
   globalTimerMap.insert(pair(&timerid, handler);
}

这一切都编译并运行得很好UNTIL它命中（* handler）（）; 给出分离错误的线。

我是一名新兴的Linux程序员，我从C ++学到的任何知识都来自于在职培训。 这让我相信我正在做一些非常愚蠢/愚蠢的事情，但我只是不知道它是什么。 我想做什么甚至应该工作？

Answer 1

以下行：

std::function<void(void)> handler = std::bind(&A::onEventFunc, this);

这会在堆栈上创建一个局部变量（ setTimer成员函数的本地变量）。 这意味着你的地图中的指针都指向无效的内存。

我会改变你的地图来存储实际的std::function对象而不是指针。 如果您坚持在地图中存储指针，则需要动态分配内存以使内存保持有效：

std::function<void(void)>* handler = new std::function<void()>(std::bind(&A::onEventFunc, this));

但是，您需要delete内存以避免内存泄漏，因此另一种方法是在映射中使用std::unique_ptr来处理它。