Skip to content

Commit

Permalink
Use SIGHUP instead of SIGTERM to shut down the proxy. The difference
Browse files Browse the repository at this point in the history
is that SIGTERM tries to tear down everything from the signal handler,
while SIGHUP actually makes main() to exit gracefully so that there
is no racing condition between main() posting event into the rtpp_proc
queue and the rtpp_proc actually exiting.
  • Loading branch information
sobomax committed Jun 11, 2015
1 parent 47f7608 commit 55438f1
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions tests/basic_versions
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,8 @@ do
${TOP_BUILDDIR}/python/rtpp_query.py -b -s "${socket}" \
-S "${TOP_BUILDDIR}/python/sippy_lite" -i $BASEDIR/basic_versions.input -o basic_versions.qout
report "rtpp_query on ${socket}"
rtpproxy_stop TERM
report "rtpproxy stop (SIGTERM) on ${socket}"
rtpproxy_stop HUP
report "rtpproxy stop (SIGHUP) on ${socket}"
cat basic_versions.qout basic_versions.rout | ${DIFF} ${BASEDIR}/basic_versions.output -
report "basic_versions on ${socket}"
fi
Expand Down

0 comments on commit 55438f1

Please sign in to comment.