Bruno Jesus : quartz: Skip AVI bad chunks while looking for the LIST chunk.
Alexandre Julliard
julliard at wine.codeweavers.com
Fri Sep 4 07:03:43 CDT 2015
Module: wine
Branch: master
Commit: c49e9d085d905cdb2aac021dfefad7bc3f409b7e
URL: http://source.winehq.org/git/wine.git/?a=commit;h=c49e9d085d905cdb2aac021dfefad7bc3f409b7e
Author: Bruno Jesus <00cpxxx at gmail.com>
Date: Thu Sep 3 23:12:58 2015 +0800
quartz: Skip AVI bad chunks while looking for the LIST chunk.
---
dlls/quartz/avisplit.c | 39 +++++++++++++++------------------------
1 file changed, 15 insertions(+), 24 deletions(-)
diff --git a/dlls/quartz/avisplit.c b/dlls/quartz/avisplit.c
index 6059511..d62ec2b 100644
--- a/dlls/quartz/avisplit.c
+++ b/dlls/quartz/avisplit.c
@@ -1109,24 +1109,18 @@ static HRESULT AVISplitter_InputPin_PreConnect(IPin * iface, IPin * pConnectPin,
return E_FAIL;
}
- pos += sizeof(RIFFCHUNK) + list.cb;
- hr = IAsyncReader_SyncRead(This->pReader, pos, sizeof(list), (BYTE *)&list);
-
- while (list.fcc == ckidAVIPADDING || (list.fcc == FOURCC_LIST && list.fccListType != listtypeAVIMOVIE))
+ /* Skip any chunks until we find the LIST chunk */
+ do
{
pos += sizeof(RIFFCHUNK) + list.cb;
-
hr = IAsyncReader_SyncRead(This->pReader, pos, sizeof(list), (BYTE *)&list);
}
+ while (hr == S_OK && (list.fcc != FOURCC_LIST ||
+ (list.fcc == FOURCC_LIST && list.fccListType != listtypeAVIMOVIE)));
- if (list.fcc != FOURCC_LIST)
- {
- ERR("Expected LIST, but got %.04s\n", (LPSTR)&list.fcc);
- return E_FAIL;
- }
- if (list.fccListType != listtypeAVIMOVIE)
+ if (hr != S_OK)
{
- ERR("Expected AVI movie list, but got %.04s\n", (LPSTR)&list.fccListType);
+ ERR("Failed to find LIST chunk from AVI file\n");
return E_FAIL;
}
@@ -1134,21 +1128,18 @@ static HRESULT AVISplitter_InputPin_PreConnect(IPin * iface, IPin * pConnectPin,
/* FIXME: AVIX files are extended beyond the FOURCC chunk "AVI ", and thus won't be played here,
* once I get one of the files I'll try to fix it */
- if (hr == S_OK)
- {
- This->rtStart = pAviSplit->CurrentChunkOffset = MEDIATIME_FROM_BYTES(pos + sizeof(RIFFLIST));
- pos += list.cb + sizeof(RIFFCHUNK);
-
- pAviSplit->EndOfFile = This->rtStop = MEDIATIME_FROM_BYTES(pos);
- if (pos > total)
- {
- ERR("File smaller (%x%08x) then EndOfFile (%x%08x)\n", (DWORD)(total >> 32), (DWORD)total, (DWORD)(pAviSplit->EndOfFile >> 32), (DWORD)pAviSplit->EndOfFile);
- return E_FAIL;
- }
+ This->rtStart = pAviSplit->CurrentChunkOffset = MEDIATIME_FROM_BYTES(pos + sizeof(RIFFLIST));
+ pos += list.cb + sizeof(RIFFCHUNK);
- hr = IAsyncReader_SyncRead(This->pReader, BYTES_FROM_MEDIATIME(pAviSplit->CurrentChunkOffset), sizeof(pAviSplit->CurrentChunk), (BYTE *)&pAviSplit->CurrentChunk);
+ pAviSplit->EndOfFile = This->rtStop = MEDIATIME_FROM_BYTES(pos);
+ if (pos > total)
+ {
+ ERR("File smaller (%x%08x) then EndOfFile (%x%08x)\n", (DWORD)(total >> 32), (DWORD)total, (DWORD)(pAviSplit->EndOfFile >> 32), (DWORD)pAviSplit->EndOfFile);
+ return E_FAIL;
}
+ hr = IAsyncReader_SyncRead(This->pReader, BYTES_FROM_MEDIATIME(pAviSplit->CurrentChunkOffset), sizeof(pAviSplit->CurrentChunk), (BYTE *)&pAviSplit->CurrentChunk);
+
props->cbAlign = 1;
props->cbPrefix = 0;
/* Comrades, prevent shortage of buffers, or you will feel the consequences! DA! */
More information about the wine-cvs
mailing list