]> git.ipfire.org Git - thirdparty/asterisk.git/commitdiff
res_pjsip_outbound_registration: Fix reference leak.
authorGeorge Joseph <george.joseph@fairview5.com>
Thu, 8 Jan 2015 21:37:42 +0000 (21:37 +0000)
committerGeorge Joseph <george.joseph@fairview5.com>
Thu, 8 Jan 2015 21:37:42 +0000 (21:37 +0000)
Every time a registration started, sip_outbound_registration_response_cb bumps
the ref count on client_state then pushes a handle_registration_response task.
handle_registration_response never unreffed it though.  So every time a
registration goes out, the ref count goes up by one.

This patch adds the unreffs to handle_registration_response.

Tested-by: George Joseph
Review: https://reviewboard.asterisk.org/r/4303/

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

res/res_pjsip_outbound_registration.c

index ad1bc7ef5ff910548e4d437bffa3cc31945f55bf..75dae1f37eaf905f3228eeab3072223645950cd2 100644 (file)
@@ -398,7 +398,7 @@ static int handle_client_registration(void *data)
        pjsip_regc_info info;
        char server_uri[PJSIP_MAX_URL_SIZE], client_uri[PJSIP_MAX_URL_SIZE];
 
-       cancel_registration(client_state);
+       ao2_ref(client_state, -1);
 
        if ((client_state->status == SIP_REGISTRATION_STOPPED) ||
                (pjsip_regc_register(client_state->client, PJ_FALSE, &tdata) != PJ_SUCCESS)) {