[Freeswitch-dev] switch_ivr_bridge.c question

[Anthony Minessale]

The code is not the same on the latest.
Look closer ;)

You should probably keep yourself on the latest for this deep of an examination.



On Thu, Feb 2, 2012 at 11:43 AM, teldev <ustcorporation at yahoo.com> wrote:
> Re-worded my original post a bit, sorry if it becomes a new thread...having
> Nabble issues.
>
> In switch_ivr_bridge.c on line 1474, we have the following lines of code:
>
>  1474
> switch_channel_clear_state_handler(originator_channel, NULL);
>  1475
> switch_channel_clear_state_handler(originatee_channel, NULL);
>  1476                    switch_channel_set_state_flag(originator_channel,
> CF_BRIDGE_ORIGINATOR);
>  1477
> switch_channel_clear_flag_recursive(originatee_channel,
> CF_BRIDGE_ORIGINATOR);
>  1478
> switch_channel_add_state_handler(originator_channel,
> &uuid_bridge_state_handlers);
>  1479
> switch_channel_add_state_handler(originatee_channel,
> &uuid_bridge_state_handlers);
>
> Is the call to “switch_channel_set_state_flag” a typo? Since line 1476
> triggers an asynchronous flag change, and line 1477 changes the same flag
> synchronously, on line 1478 the value of the flag becomes unpredictable
> since the asynchronous change from line 1476 can happen before or after line
> 1477 (mostly after, of course).
>
> Thanks again.
>
> --
> View this message in context: http://freeswitch-dev.4355.n6.nabble.com/switch-ivr-bridge-c-question-tp4359635p4359635.html
> Sent from the FreeSWITCH-dev mailing list archive at Nabble.com.
>
> _________________________________________________________________________
> Professional FreeSWITCH Consulting Services:
> consulting at freeswitch.org
> http://www.freeswitchsolutions.com
>
> 
> 
>
> Official FreeSWITCH Sites
> http://www.freeswitch.org
> http://wiki.freeswitch.org
> http://www.cluecon.com
>
> FreeSWITCH-dev mailing list
> FreeSWITCH-dev at lists.freeswitch.org
> http://lists.freeswitch.org/mailman/listinfo/freeswitch-dev
> UNSUBSCRIBE:http://lists.freeswitch.org/mailman/options/freeswitch-dev
> http://www.freeswitch.org



-- 
Anthony Minessale II

FreeSWITCH http://www.freeswitch.org/
ClueCon http://www.cluecon.com/
Twitter: http://twitter.com/FreeSWITCH_wire

AIM: anthm
MSN:anthony_minessale at hotmail.com
GTALK/JABBER/PAYPAL:anthony.minessale at gmail.com
IRC: irc.freenode.net #freeswitch

FreeSWITCH Developer Conference
sip:888 at conference.freeswitch.org
googletalk:conf+888 at conference.freeswitch.org
pstn:+19193869900