[Bug 755862] ges-project: leaks internal file info

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Wed Sep 30 02:59:03 PDT 2015


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

Thibault Saunier <tsaunier at gnome.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #312409|none                        |accepted-commit_now
             status|                            |

--- Comment #3 from Thibault Saunier <tsaunier at gnome.org> ---
Review of attachment 312409:
 --> (https://bugzilla.gnome.org/review?bug=755862&attachment=312409)

::: ges/ges-project.c
@@ -271,3 +271,3 @@
   g_ptr_array_add (new_paths, g_strdup (value));
-  for (info = g_file_enumerator_next_file (fenum, NULL, NULL);
-      info; info = g_file_enumerator_next_file (fenum, NULL, NULL)) {
+  info = g_file_enumerator_next_file (fenum, NULL, NULL);
+  while (info != NULL) {

Why don't you simply unref and also change the loop style?

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


More information about the gstreamer-bugs mailing list