Author |
|
Jalal Vetran
Joined: April 24 2006 Location: Iran Posts: 188
|
Posted: January 23 2007 at 10:00am | IP Logged
|
|
|
Hi
As we were developing Delphi Wrapper for single line phone sample we encountered access violation exception when using DtmfGeneratorCallback. After spending about hour we found that you have forgot to put __stdcall (VOIP_API) in your SipTelephonyApi.h.
Please update following two lines in next version.
Code:
typedef void (VOIP_API *DTMF_DECODER_CALLBACK_PROC)(DTMF_DETECT_DATA *pDtmfDetectData);
typedef void (VOIP_API *DTMF_GENERATOR_CALLBACK_PROC)(DTMF_GEN_DATA *pGeneratorData);
|
|
|
|
Back to Top |
|
|
support Administrator
Joined: January 26 2005 Location: United States Posts: 1666
|
Posted: January 23 2007 at 10:33am | IP Logged
|
|
|
Hi Jalal,
Yes. Exactly. You are correct. All the media engine callbacks should be the same calling convention for consistency.
The change has been sent to the developers. Very good work. Thank you.
Support
|
Back to Top |
|
|
Jalal Vetran
Joined: April 24 2006 Location: Iran Posts: 188
|
Posted: February 13 2007 at 2:33am | IP Logged
|
|
|
Hi
In Media Engine Version 5.12.3.13 you have changed DTMF_DECODER_CALLBACK_PROC, DTMF_GENERATOR_CALLBACK_PROC to VOIP_API as we have requested. But it seems you have only changed SipTelephonyApi.h and related samples and you have not changed the LMEVoip.dll.
I mean both above functions are still cdecl in LMEVoip.dll v5.12.3.13 but the samples and SipTelephonyApi.h are changed to be stdcall. This makes your recompiled sample applications raise Access Violation Exception when working with these two CallBacks.
I don't know why you don't test any released version before deployment. !!!
Thanks
Jalal Abedinejad
|
Back to Top |
|
|
support Administrator
Joined: January 26 2005 Location: United States Posts: 1666
|
Posted: February 13 2007 at 6:30am | IP Logged
|
|
|
Hi Jalal,
We will look into it and verify that things are correct.
We do test before deployment. It’s possible though that a mistake was made.
Support
|
Back to Top |
|
|
support Administrator
Joined: January 26 2005 Location: United States Posts: 1666
|
Posted: February 13 2007 at 7:13am | IP Logged
|
|
|
Jalal,
We located the problem. The DTMF support the media engine uses is built into a stand alone static library developed by LanScape.
The DTMF callback procs in the media engine API were changed to __stacall as required. However due to internal type cast issues, the static library callback was still _cdecl and not __stacall. In this case, any DTMF generation or detection will blow up the stack because of miss matched calling conventions. We will be changing source code so that our build tools will catch this problem in the future.
Sorry for the bug - and thank you for the report. We will rebuild product distribution images and get you an update in your support FTP account.
When we have updated product images, we will repost to this thread.
By the way - I'm supposed to tell you that work continues regarding improving internal digital mixer accuracy (good results so far) and further minimizing record and playback accumulated delay errors.
Support
|
Back to Top |
|
|
support Administrator
Joined: January 26 2005 Location: United States Posts: 1666
|
Posted: February 13 2007 at 7:05pm | IP Logged
|
|
|
Jalal,
You have a new voip media engine product image waiting for you in your support FTP user account. We have made changes so that this problem does not occur again.
Support
|
Back to Top |
|
|
mfitzgerald Vetran
Joined: June 14 2006 Location: United States Posts: 142
|
Posted: February 26 2007 at 1:07pm | IP Logged
|
|
|
After much head bashing, I decided to check to see if anyone else had a similar problem with the DTMF Generator.
Voila… I found this thread right off.
We appear to be having a similar issue. Will a new Media Engine be made available for our support FTP account?
Thanks.
|
Back to Top |
|
|
support Administrator
Joined: January 26 2005 Location: United States Posts: 1666
|
Posted: February 26 2007 at 1:27pm | IP Logged
|
|
|
Hi Fitz,
Yes. we will create a new product image for you. When it's ready we will repost. Sorry for the head bashing...
Support
|
Back to Top |
|
|
support Administrator
Joined: January 26 2005 Location: United States Posts: 1666
|
Posted: February 26 2007 at 1:51pm | IP Logged
|
|
|
Fitz,
You can download v5.12.3.14 from your support FTP account. Use the login name and password you previously had. The FTP account will auto disable sometime later today.
Thanks,
Support
|
Back to Top |
|
|
mfitzgerald Vetran
Joined: June 14 2006 Location: United States Posts: 142
|
Posted: February 26 2007 at 3:35pm | IP Logged
|
|
|
Eureka…
That seems to have fixed it.
Aspirin & sleep will solve the bashed head.
Thanks for the quick update as always.
|
Back to Top |
|
|