Created attachment 306162 [details] [review]
Patch to only add the default logger in gst_init if no loggers have been added yet.

When trying to only use external logging functions, the user might try to add its own log handler and remove the default logging function before gst_init (e.g. see http://cgit.freedesktop.org/gstreamer/cerbero/tree/data/ndk-build/gstreamer_android-1.0.c.in#n482). However, because gst_init (i.e. _priv_gst_debug_init, gstinfo.c) unconditionally adds the gst_debug_log_default function to the list of active loggers, the default log handlers is still being added and additionally used later on.

This could be partially worked around by removing gst_debug_log_default after gst_init (notice, though that removal of log functions might be racy, see 751440), but that would still cause logging during gst_init to be logged to the default logger.

I have attached a patch for a simple solution for this use case: Only add the default log function during gst_init if no log function has been added by the user yet.