Rob Shearman : quartz: Use memcmp instead of strncmp in MPEGSplitter_pre_connect.

Alexandre Julliard julliard at winehq.org
Mon Aug 18 10:47:28 CDT 2008


Module: wine
Branch: master
Commit: a35431a194c4924ea447cb9e6278954947aa2fbe
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=a35431a194c4924ea447cb9e6278954947aa2fbe

Author: Rob Shearman <robertshearman at gmail.com>
Date:   Sun Aug 17 18:33:17 2008 +0100

quartz: Use memcmp instead of strncmp in MPEGSplitter_pre_connect.

header isn't a string and therefore using strncmp doesn't make sense.

---

 dlls/quartz/mpegsplit.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/dlls/quartz/mpegsplit.c b/dlls/quartz/mpegsplit.c
index ee93e2e..8625277 100644
--- a/dlls/quartz/mpegsplit.c
+++ b/dlls/quartz/mpegsplit.c
@@ -471,7 +471,7 @@ static HRESULT MPEGSplitter_pre_connect(IPin *iface, IPin *pConnectPin, ALLOCATO
         pos += 4;
 
     /* Skip ID3 v2 tag, if any */
-    if (SUCCEEDED(hr) && !strncmp("ID3", (char*)header, 3))
+    if (SUCCEEDED(hr) && !memcmp("ID3", header, 3))
     do {
         UINT length;
         hr = IAsyncReader_SyncRead(pPin->pReader, pos, 6, header + 4);




More information about the wine-cvs mailing list