[Asterisk-code-review] Replace calls to strtok() with strtok_r() (...asterisk[master])

Sean Bright asteriskteam at digium.com
Thu Mar 7 16:44:54 CST 2019


Hello Friendly Automation, Joshua Colp, 

I'd like you to reexamine a change. Please visit

    https://gerrit.asterisk.org/c/asterisk/+/11105

to look at the new patch set (#4).

Change subject: Replace calls to strtok() with strtok_r()
......................................................................

Replace calls to strtok() with strtok_r()

strtok() uses a static buffer, making it not thread safe.

Also add a #define to cause a compile failure if strtok is used.

Change-Id: Icce265153e1e65adafa8849334438ab6d190e541
---
M channels/chan_sip.c
M include/asterisk.h
M pbx/pbx_dundi.c
M res/res_fax.c
4 files changed, 19 insertions(+), 13 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/05/11105/4
-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/11105
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Change-Id: Icce265153e1e65adafa8849334438ab6d190e541
Gerrit-Change-Number: 11105
Gerrit-PatchSet: 4
Gerrit-Owner: Sean Bright <sean.bright at gmail.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-MessageType: newpatchset
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20190307/6f55773f/attachment.html>


More information about the asterisk-code-review mailing list