glbufferpool: explicitly initialize a possibly uninitialized variable
authorMatthew Waters <matthew@centricular.com>
Sat, 31 Oct 2015 12:28:20 +0000 (23:28 +1100)
committerTim-Philipp Müller <tim@centricular.com>
Sat, 9 Dec 2017 19:32:10 +0000 (19:32 +0000)
The ret variable may be uninitialized and so its contents were undefined and
the results were erratic (failing with glvideomixer, succeeding in other cases)

P.S. No idea why gcc/clang et al never picked up on this like they normally do
(probably due to some optimisation pass figuring out it's only set once...)

gst-libs/gst/gl/gstglbufferpool.c

index ed91eaa..784964f 100644 (file)
@@ -99,7 +99,7 @@ gst_gl_buffer_pool_set_config (GstBufferPool * pool, GstStructure * config)
   guint max_align, n;
   GstAllocator *allocator = NULL;
   GstAllocationParams alloc_params;
-  gboolean reset = TRUE, ret;
+  gboolean reset = TRUE, ret = TRUE;
   gint p;
 
   if (!gst_buffer_pool_config_get_params (config, &caps, NULL, &min_buffers,