diff mbox series

[FFmpeg-devel] avformat/tests/imf: Don't use uninitialized value

Message ID AM7PR03MB66601E69C1EECF758A0B04598F4A9@AM7PR03MB6660.eurprd03.prod.outlook.com
State Accepted
Commit 20b0b2be6c6f7c2c3949e4e004fad78748e76b9b
Headers show
Series [FFmpeg-devel] avformat/tests/imf: Don't use uninitialized value | expand

Checks

Context Check Description
andriy/make_x86 success Make finished
andriy/make_fate_x86 success Make fate finished
andriy/make_ppc success Make finished
andriy/make_fate_ppc fail Make fate failed

Commit Message

Andreas Rheinhardt Jan. 4, 2022, 9:25 a.m. UTC
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
---
I'll apply this pretty soon.

 libavformat/tests/imf.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Zane van Iperen Jan. 4, 2022, 9:32 a.m. UTC | #1
On 4/1/22 19:25, Andreas Rheinhardt wrote:
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
> ---
> I'll apply this pretty soon.
> 
lgtm, Please do.
diff mbox series

Patch

diff --git a/libavformat/tests/imf.c b/libavformat/tests/imf.c
index 68e4c8b6af..142aa04261 100644
--- a/libavformat/tests/imf.c
+++ b/libavformat/tests/imf.c
@@ -347,7 +347,7 @@  static int test_bad_cpl_parsing(void)
     doc = xmlReadMemory(cpl_bad_doc, strlen(cpl_bad_doc), NULL, NULL, 0);
     if (doc == NULL) {
         printf("XML parsing failed.\n");
-        return ret;
+        return 1;
     }
 
     ret = ff_imf_parse_cpl_from_xml_dom(doc, &cpl);