[asterisk-bugs] [Asterisk 0015673]: [patch] Length of the parameters using an Async Originate

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=15673 
====================================================================== 
Reported By:                ernestojpg
Assigned To:                junky
====================================================================== 
Project:                    Asterisk
Issue ID:                   15673
Category:                   Core/ManagerInterface
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     assigned
Asterisk Version:           Older 1.4 
JIRA:                        
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
====================================================================== 
Date Submitted:             2009-08-07 08:03 CDT
Last Modified:              2009-11-26 09:57 CST
====================================================================== 
Summary:                    [patch] Length of the parameters using an Async
Originate
Description: 
In the Originate Action, when the Async option is true, the Data option can
only be 80 characters or less. When Async option is false (or it is not
present), the Data option can be much longer.

====================================================================== 

---------------------------------------------------------------------- 
 (0114314) tamiel (reporter) - 2009-11-26 09:57
 https://issues.asterisk.org/view.php?id=15673#c114314 
---------------------------------------------------------------------- 
I just noticed this bug too on 1.4.26.2.

With "Async: true", max length is 80 (including '\0') because Originate
execute fast_originate() function and this function use struct
fast_originate_helper .
In this struct, field appdata (where "Data" field will be copied with
ast_copy_string) ) is char appdata[AST_MAX_EXTENSION] and AST_MAX_EXTENSION
is set to 80 . 

Without Async mode, max length is 256 (including '\0'), because when
passing data to ast_pbx_outgoing_app function, appdata is copied into 
tmp->data in pbx.c with ast_copy_string and tmp->data is char data[256].

Here is a patch for version 1.4.x to set Data max length set to 256
(including '\0') in Async mode . 

This patch probably work with 1.6.0.x because fast_originate_helper is the
same. 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-11-26 09:57 tamiel         Note Added: 0114314                          
======================================================================