[PATCH] qedit: Drop redundant NULL check before free()
Signed-off-by: Michael Stefaniuc <mstefani(a)winehq.org> --- dlls/qedit/mediadet.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/dlls/qedit/mediadet.c b/dlls/qedit/mediadet.c index e1defaa9a67..bacf5208a08 100644 --- a/dlls/qedit/mediadet.c +++ b/dlls/qedit/mediadet.c @@ -66,8 +66,7 @@ static void MD_cleanup(MediaDetImpl *This) This->splitter = NULL; if (This->graph) IGraphBuilder_Release(This->graph); This->graph = NULL; - if (This->filename) - free(This->filename); + free(This->filename); This->filename = NULL; This->num_streams = -1; This->cur_stream = 0; -- 2.26.2
Hi, While running your changed tests, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at: https://testbot.winehq.org/JobDetails.pl?Key=79085 Your paranoid android. === debiant (build log) === Task: WineTest did not produce the wow32 report
participants (3)
-
Marvin -
Michael Stefaniuc -
Zebediah Figura