chan_sip: Simplify some unref code by removing unlink_peer_from_tables.

ASTERISK-22945 #related
Reported by: ibercom
Patches:
  asterisk11-chan_sip-simplifies.patch uploaded by ibercom (License #6599)
........

Merged revisions 424181 from http://svn.asterisk.org/svn/asterisk/branches/1.8
........

Merged revisions 424182 from http://svn.asterisk.org/svn/asterisk/branches/11

git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/12@424183 65c4cc65-6c06-0410-ace0-fbb531ad65f3

diff --git a/channels/chan_sip.c b/channels/chan_sip.c
index e6be28f7eb6a94363b35489d4141f3c5c98e8423..0bb5cb19844728ee9ffa506059b0488e7bc7717c 100644 (file)
--- a/channels/chan_sip.c
+++ b/channels/chan_sip.c
@@ -3285,15 +3285,6 @@ static void unlink_all_peers_from_tables(void)
        unlink_peers_from_tables(SIP_PEERS_ALL);
 }
 
-/* \brief Unlink single peer from all ao2 containers */
-static void unlink_peer_from_tables(struct sip_peer *peer)
-{
-       ao2_t_unlink(peers, peer, "ao2_unlink of peer from peers table");
-       if (!ast_sockaddr_isnull(&peer->addr)) {
-               ao2_t_unlink(peers_by_ip, peer, "ao2_unlink of peer from peers_by_ip table");
-       }
-}
-
 /*! \brief maintain proper refcounts for a sip_pvt's outboundproxy
  *
  * This function sets pvt's outboundproxy pointer to the one referenced
@@ -15850,11 +15841,12 @@ static int expire_register(const void *data)
 
        if (peer->selfdestruct ||
            ast_test_flag(&peer->flags[1], SIP_PAGE2_RTAUTOCLEAR)) {
-               unlink_peer_from_tables(peer);
-       } else if (!ast_sockaddr_isnull(&peer->addr)) {
-               /* If we aren't self-destructing a temp_peer, we still need to unlink the peer
-                * from the peers_by_ip table, otherwise we end up with multiple copies hanging
-                * around each time a registration expires and the peer re-registers. */
+               ao2_t_unlink(peers, peer, "ao2_unlink of peer from peers table");
+       }
+       if (!ast_sockaddr_isnull(&peer->addr)) {
+               /* We still need to unlink the peer from the peers_by_ip table,
+                * otherwise we end up with multiple copies hanging around each
+                * time a registration expires and the peer re-registers. */
                ao2_t_unlink(peers_by_ip, peer, "ao2_unlink of peer from peers_by_ip table");
        }