chan_pjsip: AMI action PJSIPShowEndpoint closes AMI connection on error.

Also fixed similar problem with AMI action PJSIPShowEndpoints.

ASTERISK-24872 #close
Reported by: Dmitriy Serov

Review: https://reviewboard.asterisk.org/r/4487/
........

Merged revisions 432894 from http://svn.asterisk.org/svn/asterisk/branches/13


git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@432895 65c4cc65-6c06-0410-ace0-fbb531ad65f3
changes/42/42/1
Richard Mudgett 10 years ago
parent 636d82f4d8
commit c52adca396

@ -1159,7 +1159,7 @@ static int ami_show_endpoint(struct mansession *s, const struct message *m)
ast_sip_get_sorcery(), "endpoint", endpoint_name))) { ast_sip_get_sorcery(), "endpoint", endpoint_name))) {
astman_send_error_va(s, m, "Unable to retrieve endpoint %s\n", astman_send_error_va(s, m, "Unable to retrieve endpoint %s\n",
endpoint_name); endpoint_name);
return -1; return 0;
} }
astman_send_listack(s, m, "Following are Events for each object associated with the the Endpoint", astman_send_listack(s, m, "Following are Events for each object associated with the the Endpoint",
@ -1199,7 +1199,7 @@ static int format_ami_endpoints(void *obj, void *arg, int flags)
ast_sip_create_ami_event("EndpointList", ami), ast_free); ast_sip_create_ami_event("EndpointList", ami), ast_free);
if (!buf) { if (!buf) {
return -1; return CMP_STOP;
} }
sip_sorcery_object_ami_set_type_name(endpoint, &buf); sip_sorcery_object_ami_set_type_name(endpoint, &buf);
@ -1238,7 +1238,8 @@ static int ami_show_endpoints(struct mansession *s, const struct message *m)
endpoints = ast_sip_get_endpoints(); endpoints = ast_sip_get_endpoints();
if (!endpoints) { if (!endpoints) {
return -1; astman_send_error(s, m, "Could not get endpoints\n");
return 0;
} }
if (!(num = ao2_container_count(endpoints))) { if (!(num = ao2_container_count(endpoints))) {

Loading…
Cancel
Save