[Bug 751306] good plugins: fix some issues found using static analysis tool

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=751306

Luis de Bethencourt <luis at debethencourt.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #305801|none                        |reviewed
             status|                            |

--- Comment #10 from Luis de Bethencourt <luis at debethencourt.com> ---
Review of attachment 305801:
 --> (https://bugzilla.gnome.org/review?bug=751306&attachment=305801)

True. It is good practice to close the file before returning.

No need to set it to NULL though. You can remove that line in the patch.

Notice in the comments at the top of the file it says:
 * This file is part of FFmpeg.

Does the file need any further updates?

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.