diff options
| author | George Joseph <george.joseph@fairview5.com> | 2016-01-09 16:57:45 -0700 |
|---|---|---|
| committer | George Joseph <george.joseph@fairview5.com> | 2016-01-09 17:08:46 -0700 |
| commit | 4b10fc917340bbbfe6222fa7c6131f004912879a (patch) | |
| tree | 59d00b5a2ebbcdcc74ec424d07702a2f4269228a /res/res_pjsip | |
| parent | 573b37c6581b54cfe80dfd9a956c29c33f11f4bd (diff) | |
Revert "pjsip_location: Delete contact_status object when contact is deleted"
This reverts commit 0a9941de9d24093b5ff44096d1d7406f29d11e45.
Matt,
This patch causes another problem and should not have been needed.
Before this patch, persistent_endpoint_contact_deleted_observer WAS
deleting the contact_status when ast_sip_location_delete_contact was
called. By deleting it yourself in ast_sip_location_delete_contact
it was gone before the observer could run and the observer therefore
was throwing an error and not sending stasis/AMI/statsd messages.
So, I don't think this was the cause of your original issue. I also
had verified the contact AMI and statsd lifecycle and it was working.
I'll double check now though.
ASTERISK-25675
Reported-by: Daniel Journo
Change-Id: Ib586a6b7f90acb641b0c410f659743ab90e84f1a
Diffstat (limited to 'res/res_pjsip')
| -rw-r--r-- | res/res_pjsip/location.c | 8 |
1 files changed, 0 insertions, 8 deletions
diff --git a/res/res_pjsip/location.c b/res/res_pjsip/location.c index a34e5cdf3..2908f6f70 100644 --- a/res/res_pjsip/location.c +++ b/res/res_pjsip/location.c @@ -322,14 +322,6 @@ int ast_sip_location_update_contact(struct ast_sip_contact *contact) int ast_sip_location_delete_contact(struct ast_sip_contact *contact) { - void *contact_status_obj; - - contact_status_obj = ast_sorcery_retrieve_by_id(ast_sip_get_sorcery(), CONTACT_STATUS, ast_sorcery_object_get_id(contact)); - if (contact_status_obj) { - ast_sorcery_delete(ast_sip_get_sorcery(), contact_status_obj); - ao2_ref(contact_status_obj, -1); - } - return ast_sorcery_delete(ast_sip_get_sorcery(), contact); } |