Help request: QuickTime ActiveX Control Heap Corruption the second time I open a dial

Hello everybody,

I need to write a piece of code that opens a QuickTime VR image into an MFC Dialog.
I drew a Dialog and put an "Apple QuickTime Control 2.0" into it.
Then I created a Control variable in my Dialog class, called QtControl.
In the OnInitDialog I set the file to open, calling the put_FileName of the control.

I tested it by:

- opening a dialog --> worked
- closing it --> worked
- opening again --> crashed. The error was:

"Windows has triggered a breakpoint in TestOcx.exe.
This may be due to a corruption of the heap, which indicates a bug in TestOcx.exe or any of the DLLs it has loaded.
This may also be due to the user pressing F12 while TestOcx.exe has focus.
The output window may have more diagnostic information."

I tried many things and googled a lot, but the only way that I found to avoid this crash was hardcoding a valid URL in the QtControl resources at compile time.
In other words, if I write a valid absolute path to a .mov into the URL property of the QtControl from Visual Studio and compile it, then it works.
Obviously, I can't do this because the absolute Path is valid only on my PC.

What I'm asking you is:

- suggestion to solve this, or
- any website where I can read or,
- at worst, another implementation that is not based on the activeX and takes me 2 days of coding

You may want to download a test application with the problem at this link

Feel free to ask me any other detail, and any hint would be appreciated.

Thanks in advance,
E.Russo

Quote:

---

Originally Posted by emarusso

I tried many things

---

Nowhere do you mention that you tried to debug the problem using the debugger.

Quote:

---

and googled a lot

---

Google doesn't solve these problems -- you need to actually debug the issue by launching the application under the debugger.

Quote:

---

In other words, if I write a valid absolute path to a .mov into the URL property of the QtControl from Visual Studio and compile it, then it works.

---

Then that makes it easy if you debug both scenarios and see what different path or what functions are called for each situation.

Regards,

Paul McKenzie

Hello,

here's what I didn't explain in my last post. Every experiment has been done with the debugger attached.

1. In Visual studio, I hardcoded into the URL property of my QuickTime ActiveX control a valid absolute path to a .mov file that was on my PC, say C:\something.mov

in this case it works.

2. I wanted to do something more flexible, so I compiled my program leaving the field empty, and setting it into the OnInitDialog method of the Dialog class that contains the control.

in this case it throws exceptions when I close the dialog. I enabled the Exception flag into the debugger, and it catched the following first-chance one:

First-chance exception at 0x773ac41f in TestOcx.exe: 0x0000071A: Remote procedure call was aborted

this is the Call Stack:

Code:

---

>        KernelBase.dll!773ac41f()       
        [Frames below may be incorrect and/or missing, no symbols loaded for KernelBase.dll]       
        KernelBase.dll!773ac41f()       
        rpcrt4.dll!770c0e0b()       
        rpcrt4.dll!770c0d9a()       
        winsta.dll!6ae08930()       
        winsta.dll!6ae08981()       
        winsta.dll!6ae088ee()       
        rpcrt4.dll!7704b500()       
        rpcrt4.dll!7704b4b2()       
        rpcrt4.dll!7704d8b5()       
        rpcrt4.dll!7704dbc8()       
        rpcrt4.dll!7704db45()       
        rpcrt4.dll!7704de16()       
        rpcrt4.dll!7704dd5d()       
        rpcrt4.dll!7704daed()       
        winsta.dll!6ae08b18()       
        winsta.dll!6ae0baeb()       
        winsta.dll!6ae0baa2()       
        winsta.dll!6ae0ba49()       
        winsta.dll!6ae0b971()       
        QTOControl.dll!61bb4e29()       
        QTOControl.dll!61bb7d84()       
        QTOControl.dll!61bb0cfc()

---

Then I passed the exception to the program and the following assert is thrown:

wincore.cpp

Code:

---

LRESULT CALLBACK
AfxWndProc(HWND hWnd, UINT nMsg, WPARAM wParam, LPARAM lParam)
{
        // special message which identifies the window as using AfxWndProc
        if (nMsg == WM_QUERYAFXWNDPROC)
                return 1;

        // all other messages route through message map
        CWnd* pWnd = CWnd::FromHandlePermanent(hWnd);
        ASSERT(pWnd != NULL);

---

After ignoring 3 times the assert, the program seems to work.
But sometimes it throws the exception that I mentioned in my first post and the application crashes.


3. I tried to "Shutdown" in some way the quicktime player, by editing the OnClose method of the Dialog, but both didn't solve my problem.

I tried to:
- call the method "QuickTimeTerminate" of the control
- set the URL to empty string


That's it. If you need other info, I'm here... :-)

Regards,
emarusso

Quote:

---

Originally Posted by emarusso

2. I wanted to do something more flexible, so I compiled my program leaving the field empty, and setting it into the OnInitDialog method of the Dialog class that contains the control.

---

And how is this field set in terms of C++?

In other words, a description of what you did is not enough. Show the actual code you claim sets this field. This is especially the case when it comes to strings, as it seems to be the case that too many persons make basic mistakes when it comes to handling string data in C++.

Regards,

Paul McKenzie

The code is here

http://docs.google.com/file/d/0B7Mih...it?usp=sharing

as I wrote in my first post.

Here's the point where I set the URL:

Code:

---

BOOL CQuickTimeDlg::OnInitDialog()
{
        CDialog::OnInitDialog();

        HICON hIcon = LoadIcon(AfxGetInstanceHandle(), MAKEINTRESOURCE(IDI_ICON_QUICKTIME));
        SetIcon(hIcon, FALSE);

        m_kQtControl.put_FileName(m_strURL);
        m_kQtControl.put_FullScreen(m_bFullscreen);
        m_kQtControl.put_MovieControllerVisible(m_bMovieControllerVisible);

        return TRUE;  // return TRUE unless you set the focus to a control
        // ECCEZIONE: le pagine delle propriet� OCX devono restituire FALSE
}

---

where m_strURL is a CString, whose value is written in the CQuickTimeDlg constructor.

Quote:

---

Originally Posted by emarusso

m_kQtControl.put_FileName(m_strURL);

---

And the function prototype for put_FileName is what exactly? Is it this?
http://msdn.microsoft.com/en-us/libr...=vs.85%29.aspx

If so, then a CString is not a BSTR.

http://forums.codeguru.com/showthrea...String-to-BSTR
http://www.flounder.com/cstring.htm#CString%20to%20BSTR

Regards,

Paul McKenzie

No, the prototype is

void put_FileName(LPCTSTR newValue).

As written in QTControl.h, that I attached.

Quote:

---

Originally Posted by emarusso

No, the prototype is

void put_FileName(LPCTSTR newValue).

As written in QTControl.h, that I attached.

---

Try replacing

Code:

---

m_kQtControl.put_FileName(m_strURL);

---

with

Code:

---

m_kQtControl.put_FileName(m_strURL.GetBuffer());
m_strURL.ReleaseBuffer();

---

to access the buffer contents of m_strURL.

Nothing changes.

I think the problem is somewhere else, because the assert is thrown by the destructor of
COleControlSite, namely the method

m_pInPlaceObject->InPlaceDeactivate();

As far as I know, COleControlSite is the container of ActiveX controls, and the m_pInPlaceObject member is the control itself.

Have you checked that the contents of m_strURL.GetBuffer() are correct? What happens if you hardcode the URL as in

Code:

---

m_kQtControl.put_FileName(_T("MyURL"));

---

Quote:

---

Originally Posted by emarusso

Nothing changes.

I think the problem is somewhere else, because the assert is thrown by the destructor of
COleControlSite, namely the method

---

Sounds like heap corruption to me.

First, does this entire application consist of more than one executable module (DLL, EXE)? If so, and if you're passing CString between executables, then you need to be careful ... more to the point, drop using it. You shouldn't pass classes that can potentially use the heap internally to other executable modules, the reason being that

1) all modules must use the same heap manager, since CString uses the heap.

2) The version of CString between the modules must be the exact same version. If you compiled one module one way, and then compiled another module with a set of different options, you would more than likely have two completely different versions of CString being used by the application, causing random crashes when you use it.

Regards,

Paul McKenzie

Quote:

---

Originally Posted by 2kaud

Have you checked that the contents of m_strURL.GetBuffer() are correct? What happens if you hardcode the URL as in

Code:

---

m_kQtControl.put_FileName(_T("MyURL"));

---

---

Nothing changes, the error's still there.

Quote:

---

Originally Posted by Paul McKenzie

Sounds like heap corruption to me.
First, does this entire application consist of more than one executable module (DLL, EXE)?

---

Sounds like heap corruption to me too.
In fact, when you try to open the dialog again,
after having ignored the assertions, Visual Studio throws an error saying that heap corruption occurred.

I think it's a QuickTime issue and there's nothing wrong in my code.

Since I haven't found any code reference online,
I posted here hoping that someone else had the same problem.

The application is a MFC dialog app created with visual studio wizard, and uses the quickTime ActiveX.
You may want to download it here:

http://docs.google.com/file/d/0B7Mih...it?usp=sharing


Thanks for your suggestions,
emarusso

From your first post

Quote:

---

I tried many things and googled a lot, but the only way that I found to avoid this crash was hardcoding a valid URL in the QtControl resources at compile time.

---

If hardcoding a valid URL in the control at compile does not cause the problem, then IMO the cause of the problem is unlikely to be the control. I would be looking at how I passed the URL to the control.

Where do you hard code the URL? I suggested a test of a hardcoded URL in my post #10 but you replied that this still causes the problem. I don't understand - as in one post you say hardcoding the URL works and in another you say it doesn't.:confused:

Quote:

---

Originally Posted by emarusso

I think it's a QuickTime issue and there's nothing wrong in my code.

---

Programmers get fired for saying that, and especially if they're using C++ where crashes can occur due to the fault of the programmer.

Therefore, you must always assume it is your code unless you can prove without a doubt it isn't. My bet is that it is your code in some way, i.e. how it is built, or at runtime, some other errors you haven't detected.

Quote:

---

The application is a MFC dialog app created with visual studio wizard, and uses the quickTime ActiveX.
You may want to download it here:

---

So instead of this larger application, why not a much smaller one where all you're doing is using the control?

Regards,

Paul McKenzie

Quote:

---

Originally Posted by emarusso

Nothing changes, the error's still there.

---

You are specifying a constant, and now it doesn't work? Before you said it did work when you specify a constant.

Regards,

Paul McKenzie

Quote:

---

Originally Posted by emarusso

No, the prototype is

void put_FileName(LPCTSTR newValue).

As written in QTControl.h, that I attached.

---

You didn't attach code. You had a link to a site that my browser policies refuses to go to.

Actually attach the code, don't link to another site.

Regards,

Paul McKenzie

The zip file contains the code from the link.

Attachment 31445

Quote:

---

Originally Posted by 2kaud

Where do you hard code the URL? I suggested a test of a hardcoded URL in my post #10 but you replied that this still causes the problem.

---

When I wrote hardcoded I meant that I wrote it in the visual studio resource editor, in the property window.

Attachment 31449

What you said was to hardcode the string in the function call.

thanks for re-attaching it, I didn't know how to attach it to the thread.

Quote:

---

Originally Posted by emarusso

thanks for re-attaching it, I didn't know how to attach it to the thread.

---

Then read the Announcement: Before you post....
BTW, is your MFC project a UNICODE or ANSI one? And the same question - about QuickTime ActiveX Control.

My sample is UNICODE, I tried to compile it multi-byte but the problem's still there.
I don't know wether the ActiveX is UNICODE, and I don't know how to check it.