[Asterisk-code-review] chan sip: Reload peer without its old capabilities. (asterisk[master])

Joshua Colp asteriskteam at digium.com
Tue Jun 23 06:30:00 CDT 2015


Joshua Colp has posted comments on this change.

Change subject: chan_sip: Reload peer without its old capabilities.
......................................................................


Patch Set 1:

I mean after:

        /* in case the case of the peer name has changed, update the name */
        ast_copy_string(peer->name, name, sizeof(peer->name));

        /* If we have channel variables, remove them (reload) */
        if (peer->chanvars) {
                ast_variables_destroy(peer->chanvars);
                peer->chanvars = NULL;
                /* XXX should unregister ? */
        }

It's safe to always do it.

-- 
To view, visit https://gerrit.asterisk.org/680
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I62a06bcf15e08e8c54a35612195f97179ebe5802
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Alexander Traud <pabstraud at compuserve.com>
Gerrit-Reviewer: Alexander Traud <pabstraud at compuserve.com>
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-HasComments: No



More information about the asterisk-code-review mailing list