clean memory: re-init AVPacket

MarcAntoine-Arnaud · MarcAntoine-Arnaud · commit 7d6eefa88ab1 · 2014-02-12T18:32:57.000-05:00
diff --git a/src/AvTranscoder/InputFile.cpp b/src/AvTranscoder/InputFile.cpp
@@ -210,6 +210,7 @@ bool InputFile::readNextPacket( AVPacket& packet, const size_t streamIndex )
 		int ret = av_read_frame( m_formatContext, &packet );
 		if( ret < 0 ) // error or end of file
 		{
+			av_free_packet( &packet );
 			return false;
 		}
 
@@ -218,6 +219,10 @@ bool InputFile::readNextPacket( AVPacket& packet, const size_t streamIndex )
 		{
 			return true;
 		}
+
+		// unused packet, require to delete and re-init to read the next one
+		av_free_packet( &packet );
+		av_init_packet( &packet );
 	}
 }
 

Original file line number	Diff line number	Diff line change
`@@ -210,6 +210,7 @@ bool InputFile::readNextPacket( AVPacket& packet, const size_t streamIndex )`
`210`	`210`	`int ret = av_read_frame( m_formatContext, &packet );`
`211`	`211`	`if( ret < 0 ) // error or end of file`
`212`	`212`	`{`
	`213`	`+ av_free_packet( &packet );`
`213`	`214`	`return false;`
`214`	`215`	`}`
`215`	`216`
`@@ -218,6 +219,10 @@ bool InputFile::readNextPacket( AVPacket& packet, const size_t streamIndex )`
`218`	`219`	`{`
`219`	`220`	`return true;`
`220`	`221`	`}`
	`222`	`+`
	`223`	`+ // unused packet, require to delete and re-init to read the next one`
	`224`	`+ av_free_packet( &packet );`
	`225`	`+ av_init_packet( &packet );`
`221`	`226`	`}`
`222`	`227`	`}`
`223`	`228`