Return to LanScape's home page Go back a page...       Active TopicsActive Topics   Display List of Forum MembersMember List   Knowledge Base SearchSearch   HelpHelp  RegisterRegister  LoginLogin

LanScape VOIP Media Engine™ - Technical Support
 LanScape Support Forum -> LanScape VOIP Media Engine™ - Technical Support
Subject Topic: DTMF CallBack cdecl problem Post ReplyPost New Topic
Author
Message << Prev Topic | Next Topic >>
Jalal
Vetran
Vetran


Joined: April 24 2006
Location: Iran
Posts: 188
Posted: January 23 2007 at 10:00am | IP Logged Quote Jalal

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 View Jalal's Profile Search for other posts by Jalal Visit Jalal's Homepage
 
support
Administrator
Administrator


Joined: January 26 2005
Location: United States
Posts: 1666
Posted: January 23 2007 at 10:33am | IP Logged Quote support

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 View support's Profile Search for other posts by support Visit support's Homepage
 
Jalal
Vetran
Vetran


Joined: April 24 2006
Location: Iran
Posts: 188
Posted: February 13 2007 at 2:33am | IP Logged Quote Jalal

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 View Jalal's Profile Search for other posts by Jalal Visit Jalal's Homepage
 
support
Administrator
Administrator


Joined: January 26 2005
Location: United States
Posts: 1666
Posted: February 13 2007 at 6:30am | IP Logged Quote support

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 View support's Profile Search for other posts by support Visit support's Homepage
 
support
Administrator
Administrator


Joined: January 26 2005
Location: United States
Posts: 1666
Posted: February 13 2007 at 7:13am | IP Logged Quote support

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 View support's Profile Search for other posts by support Visit support's Homepage
 
support
Administrator
Administrator


Joined: January 26 2005
Location: United States
Posts: 1666
Posted: February 13 2007 at 7:05pm | IP Logged Quote support

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 View support's Profile Search for other posts by support Visit support's Homepage
 
mfitzgerald
Vetran
Vetran


Joined: June 14 2006
Location: United States
Posts: 142
Posted: February 26 2007 at 1:07pm | IP Logged Quote mfitzgerald

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 View mfitzgerald's Profile Search for other posts by mfitzgerald Visit mfitzgerald's Homepage
 
support
Administrator
Administrator


Joined: January 26 2005
Location: United States
Posts: 1666
Posted: February 26 2007 at 1:27pm | IP Logged Quote support

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 View support's Profile Search for other posts by support Visit support's Homepage
 
support
Administrator
Administrator


Joined: January 26 2005
Location: United States
Posts: 1666
Posted: February 26 2007 at 1:51pm | IP Logged Quote support

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 View support's Profile Search for other posts by support Visit support's Homepage
 
mfitzgerald
Vetran
Vetran


Joined: June 14 2006
Location: United States
Posts: 142
Posted: February 26 2007 at 3:35pm | IP Logged Quote mfitzgerald

Eureka…

That seems to have fixed it.

Aspirin & sleep will solve the bashed head.

Thanks for the quick update as always.
Back to Top View mfitzgerald's Profile Search for other posts by mfitzgerald Visit mfitzgerald's Homepage
 

If you wish to post a reply to this topic you must first login
If you are not already registered you must first register

  Post ReplyPost New Topic
Printable version Printable version

Forum Jump
You cannot post new topics in this forum
You cannot reply to topics in this forum
You cannot delete your posts in this forum
You cannot edit your posts in this forum
You cannot create polls in this forum
You cannot vote in polls in this forum






Contact LanScape Hear what the Lawyers have to say How youm may use this site Read your privacy rights