commit 6141c8b207b68a3df5698bb8d8ce05019b36e4da
author Patrick Venture <venture@google.com> Tue Oct 16 13:11:39 2018 -0700
committer James Feist <james.feist@linux.intel.com> Tue Oct 16 22:51:41 2018 +0000
tree 160267df70d23edefb687a33a32a192ac45c22a8
parent 6f6d0d32f282477558ab3235f1d162d6207e2c58

cleanup: close open files

Clean up missing fclose calls.

[src/getvideo_main.cpp:77]: (error) Resource leak: fp
[src/ast_video_puller_test.cpp:56]: (error) Resource leak: fp

Change-Id: I4d460e861a6275bfa6c02a319894d3154aec8ee7
Signed-off-by: Patrick Venture <venture@google.com>

src/ast_video_puller_test.cpp

diff --git a/src/ast_video_puller_test.cpp b/src/ast_video_puller_test.cpp
index 4fa3dcc..f84b302 100644
--- a/src/ast_video_puller_test.cpp
+++ b/src/ast_video_puller_test.cpp
@@ -49,6 +49,7 @@
 
     FILE *fp = fopen("/tmp/screendata.bin", "wb");
     fwrite(out.buffer.data(), sizeof(char), out.buffer.size(), fp);
+    fclose(fp);
 
     ast_video::AstJpegDecoder d;
     d.decode(out.buffer, out.width, out.height, out.mode, out.ySelector,

src/getvideo_main.cpp

diff --git a/src/getvideo_main.cpp b/src/getvideo_main.cpp
index 8d5f7af..849c75d 100644
--- a/src/getvideo_main.cpp
+++ b/src/getvideo_main.cpp
@@ -52,6 +52,7 @@
 
     FILE *fp = fopen("/tmp/screendata.bin", "wb");
     fwrite(out.buffer.data(), sizeof(char), out.buffer.size(), fp);
+    fclose(fp);
 
     ast_video::AstJpegDecoder d;
     d.decode(out.buffer, out.width, out.height, out.mode, out.ySelector,