commit e24042225faf54c26fe7326f76e754567ed22a61
author Ashlesh Gawande <agawande@memphis.edu> Mon May 21 18:30:24 2018 -0500
committer Ashlesh Gawande <agawande@memphis.edu> Thu May 24 16:05:29 2018 -0500
tree 9b5ce7d7247fc8406557faa3ddbb938900acdd7a
parent cc910cd0aa16d8c9a0733a27b4c27d8f9720b157

face: use post instead of dispatch to avoid re-entrancy issues

This fixes a segfault in satisfyPendingInterests when Face::put is
called in DataCallback. Other functions are also safeguarded
against similar situations.

refs: #4596

Change-Id: I8d1564b0d28d1ee7b80c50712d888f874804823a

src/face.cpp

diff --git a/src/face.cpp b/src/face.cpp
index fdb6b29..cf1e69d 100644
--- a/src/face.cpp
+++ b/src/face.cpp
@@ -1,6 +1,6 @@
 /* -*- Mode:C++; c-file-style:"gnu"; indent-tabs-mode:nil; -*- */
 /*
- * Copyright (c) 2013-2017 Regents of the University of California.
+ * Copyright (c) 2013-2018 Regents of the University of California.
  *
  * This file is part of ndn-cxx library (NDN C++ library with eXperimental eXtensions).
  *
@@ -183,7 +183,7 @@
   shared_ptr<Interest> interest2 = make_shared<Interest>(interest);
   interest2->getNonce();
 
-  IO_CAPTURE_WEAK_IMPL(dispatch) {
+  IO_CAPTURE_WEAK_IMPL(post) {
     impl->asyncExpressInterest(interest2, afterSatisfied, afterNacked, afterTimeout);
   } IO_CAPTURE_WEAK_IMPL_END
 
@@ -215,7 +215,7 @@
 void
 Face::put(Data data)
 {
-  IO_CAPTURE_WEAK_IMPL(dispatch) {
+  IO_CAPTURE_WEAK_IMPL(post) {
     impl->asyncPutData(data);
   } IO_CAPTURE_WEAK_IMPL_END
 }
@@ -223,7 +223,7 @@
 void
 Face::put(lp::Nack nack)
 {
-  IO_CAPTURE_WEAK_IMPL(dispatch) {
+  IO_CAPTURE_WEAK_IMPL(post) {
     impl->asyncPutNack(nack);
   } IO_CAPTURE_WEAK_IMPL_END
 }