Solved

Unhandled Exception in AVIFileOpen()

Posted on 2004-08-23
7
1,712 Views
Last Modified: 2011-09-20
I'm having a problem opening an AVI file.

Using VC++ 6.0

In my header file:

protected
    PAVIFILE* m_pAviFile;



in my class constructor:

CAviFile::CAviFile()
{
    AVIFileInit();
    m_pAviFile = NULL;
..//


and then later I try to open the file:

    HRESULT hr = AVIFileOpen(m_pAviFile, m_sMoviePath, OF_WRITE | OF_CREATE, NULL);


That line crashes "Unhandled exception in MYAPP.exe [AVIFIL32.dll]: 0XC0000005: Access Violation"

The file is however being created, I can browse to the folder in windows and see it, so it's not a path problem. So what's going wrong here?

thanks
-Paul
0
Comment
Question by:PMH4514
[X]
Welcome to Experts Exchange

Add your voice to the tech community where 5M+ people just like you are talking about what matters.

  • Help others & share knowledge
  • Earn cash & points
  • Learn & ask questions
  • 4
  • 3
7 Comments
 
LVL 86

Accepted Solution

by:
jkr earned 50 total points
ID: 11870586
That should be more like

protected
   PAVIFILE m_pAviFile;

    HRESULT hr = AVIFileOpen(&m_pAviFile, m_sMoviePath, OF_WRITE | OF_CREATE, NULL);

since a 'PAVIFILE *' is expected to store a PAVIFILE in there.
0
 

Author Comment

by:PMH4514
ID: 11870720
oh duh.. damn those pointers!  

but....  wouldn't passing m_pAviFile (no ampersand) into AVIFileOpen, m_pAviFile having being defined as type PAVIFILE* be the same thing? or does "P" in "PAVIFILE" infer it's already a pointer type? (ie. I'm creating a pointer to a pointer by doing what I was doing??)

anyway - I made that change, and AVIFileOpen is no longer crashing. but now I'm getting the same error on this line:

HRESULT hr = AVIFileCreateStream(m_pAviFile, &m_psWrite, &strhdr);

where m_psWrite is defined in the header:
PAVISTREAM m_psWrite

and strhdr is defined locally within my method:

AVISTREAMINFO strhdr;
memset(&strhdr, 0, sizeof(strhdr));
strhdr.fccType                = streamtypeVIDEO;   // stream type
// etc..


0
 
LVL 86

Expert Comment

by:jkr
ID: 11870759
>>or does "P" in "PAVIFILE" infer it's already a pointer type?

Yes, these are MS' naming conventions.

>>now I'm getting the same error on this line:
>>HRESULT hr = AVIFileCreateStream(m_pAviFile, &m_psWrite, &strhdr);

Hm, that actually looks good. Have you tried to

ASSERT(m_pAviFile);
HRESULT hr = AVIFileCreateStream(m_pAviFile, &m_psWrite, &strhdr);

?
0
Independent Software Vendors: We Want Your Opinion

We value your feedback.

Take our survey and automatically be enter to win anyone of the following:
Yeti Cooler, Amazon eGift Card, and Movie eGift Card!

 

Author Comment

by:PMH4514
ID: 11870767
actually, I think the CreateStream is crashing because my AVIFileOpen is still not working properly. I don't crash with the access violation, but the pointer remains NULL, and the result equals a long negative number..

(the fact that my code got to my AVIFileCreateStream() call in that state means I need better error trapping, but that's offtopic)

0
 

Author Comment

by:PMH4514
ID: 11870793
looks like we were typing at the same time..  I guess the problem is still that AVIFileOpen() isn't properly returning the file handle (though again, the file is being created, I see it in windows explorer)
0
 
LVL 86

Expert Comment

by:jkr
ID: 11871059
Are you sure you are filling in the header correctly? The sample at http://msdn.microsoft.com/library/default.asp?url=/library/en-us/multimed/htm/_win32_reading_from_one_stream_and_writing_to_another.asp ("Reading from One Stream and Writing to Another") illustrates how to use these APIs, and they're doing it exactly the same way.
0
 

Author Comment

by:PMH4514
ID: 11871062
Oh duh, once again I forgot to call CoInitialize()..

it's all set now, thanks!
-Paul
0

Featured Post

Free Tool: Subnet Calculator

The subnet calculator helps you design networks by taking an IP address and network mask and returning information such as network, broadcast address, and host range.

One of a set of tools we're offering as a way of saying thank you for being a part of the community.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

When writing generic code, using template meta-programming techniques, it is sometimes useful to know if a type is convertible to another type. A good example of when this might be is if you are writing diagnostic instrumentation for code to generat…
Written by John Humphreys C++ Threading and the POSIX Library This article will cover the basic information that you need to know in order to make use of the POSIX threading library available for C and C++ on UNIX and most Linux systems.   [s…
The viewer will learn additional member functions of the vector class. Specifically, the capacity and swap member functions will be introduced.
The viewer will learn how to clear a vector as well as how to detect empty vectors in C++.

632 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question