Marcus Meissner : dmime: Check return code of create_dmgraph (Coverity).

Alexandre Julliard julliard at wine.codeweavers.com
Wed May 6 09:29:05 CDT 2015


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

Author: Marcus Meissner <marcus at jet.franken.de>
Date:   Wed May  6 08:49:33 2015 +0200

dmime: Check return code of create_dmgraph (Coverity).

---

 dlls/dmime/audiopath.c | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/dlls/dmime/audiopath.c b/dlls/dmime/audiopath.c
index 3dff0c9..e6654a4 100644
--- a/dlls/dmime/audiopath.c
+++ b/dlls/dmime/audiopath.c
@@ -132,6 +132,7 @@ static HRESULT WINAPI IDirectMusicAudioPathImpl_GetObjectInPath (IDirectMusicAud
     REFGUID guidObject, WORD dwIndex, REFGUID iidInterface, void** ppObject)
 {
 	struct IDirectMusicAudioPathImpl *This = impl_from_IDirectMusicAudioPath(iface);
+	HRESULT hr;
 
 	FIXME("(%p, %d, %d, %d, %s, %d, %s, %p): stub\n", This, dwPChannel, dwStage, dwBuffer, debugstr_dmguid(guidObject),
             dwIndex, debugstr_dmguid(iidInterface), ppObject);
@@ -169,7 +170,9 @@ static HRESULT WINAPI IDirectMusicAudioPathImpl_GetObjectInPath (IDirectMusicAud
 	    if (IsEqualIID (iidInterface, &IID_IDirectMusicGraph)) {
 	      if (NULL == This->pToolGraph) {
 		IDirectMusicGraphImpl* pGraph;
-                create_dmgraph(&IID_IDirectMusicGraph, (void**)&pGraph);
+		hr = create_dmgraph(&IID_IDirectMusicGraph, (void**)&pGraph);
+		if (FAILED(hr))
+		  return hr;
 		This->pToolGraph = (IDirectMusicGraph*) pGraph;
 	      }
 	      *ppObject = This->pToolGraph;
@@ -200,7 +203,9 @@ static HRESULT WINAPI IDirectMusicAudioPathImpl_GetObjectInPath (IDirectMusicAud
 	    IDirectMusicPerformance8_GetGraph(This->pPerf, &pPerfoGraph);
 	    if (NULL == pPerfoGraph) {
 	      IDirectMusicGraphImpl* pGraph = NULL; 
-              create_dmgraph(&IID_IDirectMusicGraph, (void**)&pGraph);
+	      hr = create_dmgraph(&IID_IDirectMusicGraph, (void**)&pGraph);
+	      if (FAILED(hr))
+		return hr;
 	      IDirectMusicPerformance8_SetGraph(This->pPerf, (IDirectMusicGraph*) pGraph);
 	      /* we need release as SetGraph do an AddRef */
 	      IDirectMusicGraph_Release((LPDIRECTMUSICGRAPH) pGraph);




More information about the wine-cvs mailing list