[Home]

Summary:ASTERISK-12628: [patch] Incorrect checking of the "Channel" header on Originate action
Reporter:Tomás Laureano Peralta Tormey (laureano)Labels:
Date Opened:2008-08-21 18:38:19Date Closed:2008-08-22 16:27:28
Priority:TrivialRegression?No
Status:Closed/CompleteComponents:Core/ManagerInterface
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) originate_channel_check.patch
Description:When an Originate action is sent to AMI without the "Channel" header, the error response from the server is "Invalid channel" but it should be "Channel not specified".
This issue is produced by the incorrect check of the "Channel" header in the function action_originate in the file main/manager.c. The uploaded patches fixes this issue.

****** STEPS TO REPRODUCE ******

Connect to manager an issue an Originate action without the "Channel" header.

****** ADDITIONAL INFORMATION ******

Output from AMI:
================
tomas@newells:~$ telnet dev ami
Trying 192.168.0.6...
Connected to dev.
Escape character is '^]'.
Asterisk Call Manager/1.1
Action: Login
Username: mark
Secret: mysecret

Response: Success
Message: Authentication accepted

Action: Originate

Response: Error
Message: Invalid channel

Action: Logoff

Response: Goodbye
Message: Thanks for all the fish.

Connection closed by foreign host.

Output from patched AMI:
========================
tomas@newells:~$ telnet dev ami
Trying 192.168.0.6...
Connected to dev.
Escape character is '^]'.
Asterisk Call Manager/1.1
Action: Login
Username: mark
Secret: mysecret

Response: Success
Message: Authentication accepted

Action: Originate

Response: Error
Message: Channel not specified

Action: Logoff

Response: Goodbye
Message: Thanks for all the fish.

Connection closed by foreign host.
Comments:By: Digium Subversion (svnbot) 2008-08-22 16:27:26

Repository: asterisk
Revision: 139621

U   branches/1.4/main/manager.c

------------------------------------------------------------------------
r139621 | jpeeler | 2008-08-22 16:27:25 -0500 (Fri, 22 Aug 2008) | 5 lines

(closes issue ASTERISK-12628)
Reported by: Laureano
Patches:
     originate_channel_check.patch uploaded by Laureano (license 265)

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=139621