]> git.ipfire.org Git - thirdparty/asterisk.git/commit
message: Don't close an AMI connection on SendMessage action error
authorMatthew Jordan <mjordan@digium.com>
Mon, 6 Oct 2014 18:37:13 +0000 (18:37 +0000)
committerMatthew Jordan <mjordan@digium.com>
Mon, 6 Oct 2014 18:37:13 +0000 (18:37 +0000)
commit54d1ffd3681b769391e733618db8beb53c8aee30
treec4868aee222e16a0157ec252ec2d97e0896fdffe
parent55a4461798a8256accf4831f2902582a0321b597
message: Don't close an AMI connection on SendMessage action error

If SendMessage encounters an error (such as incorrect input provided to the
action), it will currently return -1. Actions should only return -1 if the
connection to the AMI client should be closed. In this case, SendMessage
causing the client to disconnect is inappropriate.

This patch causes the action to return 0, which simply causes the action to
fail.

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

ASTERISK-24354 #close
Reported by: Peter Katzmann
patches:
  sendMessage.patch uploaded by Peter Katzmann (License 5968)
........

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

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