#516: Dial CMD - misreads arguments (Fixed)

May 06 2010 * 02:42
Reported by: Release: 1.2
Priority: Normal Milestone: 2.0
Component: core Assigned to: mjagdis

Within app_dial.c, the process by which options checks for the inclusion of certain flags is flawed. If you include the option A(), with a decent sound file eg. A(/usr/blah/shared/sounds/test-447/IVRNoSuccessFinalMess), This will trip a number of the flags incorrectly, as they just search for the existence of the character.

I attempted to write a patch for this, however it isn’t so easy with my very limited c skills. When I would include a second options, eg: Dial(Sip/1234@1.1.2.2,360,A(/usr/blah/shared/sounds/test-447/IVRNoSuccessFinalMess),r) The current system would put the ’,r’ into the url area, and not options. At this point I gave up and manually worked around this by commenting out the options I do not ever want to be used, and manually forcing others.

I suggest a re-write of this, or finalising and completing the API build I saw mentioned in the message board.

Changelog:

Modified by May 06 2010 * 02:48

Sorry, forgot version information. CallWeaver 1.2.1 SVN tags/rel/1.2.1 r5634

Modified by Jun 05 2010 * 12:30
  • Assigned user: set to mjagdis
  • Status: changed from Open to Fixed
  • Milestone: set to 2.0

Fixed in trunk. Won’t fix in 1.2. It’s too much for a stable release.