[Bug 785112] Freeing NONHEAP_MEMORY.STRING

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Tue Jul 25 10:44:50 UTC 2017


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

--- Comment #15 from Ashish Kumar <kr.ashish at samsung.com> ---
(In reply to Sebastian Dröge (slomo) from comment #13)
> Review of attachment 356344 [details] [review]:
> 
> ::: gst/gstdebugutils.c
> @@ +198,2 @@
>          if (details & GST_DEBUG_GRAPH_SHOW_FULL_PARAMS) {
> +          param_name = g_strdup_printf ("\\n%s=%s", property->name,
> 
> This is changing behaviour. You're not prefixing with the property_name
> anymore. Also below

----------------------------------
In the original code 'property->name' is always prefixed by an empty string as
'tmp' is always assigned empty string ""
------------------
LnNo.198        if (param_name)     //'param_name' always 'NULL' here
LnNo.199          tmp = param_name; //control never reaches here
LnNo.200        else
LnNo.201          tmp = (char *) "";
LnNo.202
LnNo.203        if (details & GST_DEBUG_GRAPH_SHOW_FULL_PARAMS) {
LnNo.204          param_name = g_strdup_printf ("%s\\n%s=%s", tmp,
property->name,
LnNo.205              value_str);
LnNo.206        } else {
LnNo.207          param_name = g_strdup_printf ("%s\\n%s=%."
LnNo.208              G_STRINGIFY (PARAM_MAX_LENGTH) "s%s", tmp,
property->name,
LnNo.209              value_str, ellipses);
LnNo.210        }

Removing prefixing of 'property->name' by an empty string should not make any
difference.
But, if it is still changing the behaviour, please review the new patch
attached in which 'property->name' is prefixed with an empty string.

-- 
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