[Bug 752866] souphttpsrc: check null string before setting proxy

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Wed Jul 29 15:27:46 PDT 2015


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

--- Comment #5 from Reynaldo H. Verdejo Pinochet <reynaldo at opendot.cl> ---
Review of attachment 308211:
 --> (https://bugzilla.gnome.org/review?bug=752866&attachment=308211)

Good catch but:

::: ext/soup/gstsouphttpsrc.c
@@ +2046,3 @@
   if (g_str_has_prefix (uri, "http://")) {
     src->proxy = soup_uri_new (uri);
   } else {

I think it would be better to move the test here
as else if (uri[0] != '\0') { ... } else return FALSE;

Also, it might make sense to extend the test to avoid
setting nonsensical values altogether but that might
be too much, just a thought.

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