[sr-dev] git:5.0:6c749dea: tm: reset T_ASYNC_CONTINUE once t_continue() callback is executed

Daniel-Constantin Mierla miconda at gmail.com
Thu Sep 7 15:30:51 CEST 2017


Module: kamailio
Branch: 5.0
Commit: 6c749dea59d8f35b8a5303baba540edc2295cd92
URL: https://github.com/kamailio/kamailio/commit/6c749dea59d8f35b8a5303baba540edc2295cd92

Author: Daniel-Constantin Mierla <miconda at gmail.com>
Committer: Daniel-Constantin Mierla <miconda at gmail.com>
Date: 2017-09-07T15:29:58+02:00

tm: reset T_ASYNC_CONTINUE once t_continue() callback is executed

- by having it still set, the reply field was not reset, causing to free
  an invalid pointer when transaction was destroyed, reported by Vitaliy
  Aleksandrov
- set reply of suspended request branch to FAKED_REPLY on continue,
  the code was already set to 500 - make it coherent with the local
  replied transactions
- reset cloned reply under lock for suspended reply branch and free it
  later via backup pointer - safer if someone risks to access the
  suspended reply branch (should happen now, just future proof)

(cherry picked from commit b672d8ef63715cf816390a05ce7a441377c3e468)

---

Modified: src/modules/tm/t_suspend.c

---

Diff:  https://github.com/kamailio/kamailio/commit/6c749dea59d8f35b8a5303baba540edc2295cd92.diff
Patch: https://github.com/kamailio/kamailio/commit/6c749dea59d8f35b8a5303baba540edc2295cd92.patch




More information about the sr-dev mailing list