Solved

Unhandled Exception in AVIFileOpen()

Posted on 2004-08-23
7
1,630 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
  • 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
Announcing the Most Valuable Experts of 2016

MVEs are more concerned with the satisfaction of those they help than with the considerable points they can earn. They are the types of people you feel privileged to call colleagues. Join us in honoring this amazing group of Experts.

 

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

Courses: Start Training Online With Pros, Today

Brush up on the basics or master the advanced techniques required to earn essential industry certifications, with Courses. Enroll in a course and start learning today. Training topics range from Android App Dev to the Xen Virtualization Platform.

Question has a verified solution.

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

Article by: SunnyDark
This article's goal is to present you with an easy to use XML wrapper for C++ and also present some interesting techniques that you might use with MS C++. The reason I built this class is to ease the pain of using XML files with C++, since there is…
  Included as part of the C++ Standard Template Library (STL) is a collection of generic containers. Each of these containers serves a different purpose and has different pros and cons. It is often difficult to decide which container to use and …
The viewer will learn how to pass data into a function in C++. This is one step further in using functions. Instead of only printing text onto the console, the function will be able to perform calculations with argumentents given by the user.
The viewer will learn how to user default arguments when defining functions. This method of defining functions will be contrasted with the non-default-argument of defining functions.

813 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

Need Help in Real-Time?

Connect with top rated Experts

18 Experts available now in Live!

Get 1:1 Help Now