After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 541650 - udpsrc fails to parse URI with IPv6 address
udpsrc fails to parse URI with IPv6 address
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
git master
Other All
: Normal normal
: 0.10.9
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2008-07-05 12:30 UTC by Tero Saarni
Modified: 2008-07-05 19:00 UTC
See Also:
GNOME target: ---
GNOME version: 2.21/2.22

Attachments
sdp file with ipv6 address (786 bytes, text/plain)
2008-07-05 12:32 UTC, Tero Saarni 		  Details
suggested fix (400 bytes, patch)
2008-07-05 12:33 UTC, Tero Saarni 		committed Details | Review

Description Tero Saarni 2008-07-05 12:30:31 UTC 
Please describe the problem:
I'm using sdpdemux to open stream with IPv6 address.  It sets URI like udp://ff15:0:0:0:0:0:1:0600:10122 where 10122 is the port number.  When the URI is parsed by udpsrc it will try find the port number by searching for the first colon (:) from left which is ok for IPv4 dotted decimal notation but fails on IPv6 notation.

Steps to reproduce:
Set URI with IPv6 address to udpsrc

Actual results:
opening socket fails

Expected results:


Does this happen every time?
yes

Other information:
see attached .sdp file and suggested patch for udpsrc.
Comment 1 Tero Saarni 2008-07-05 12:32:54 UTC 
Created attachment 114019 [details]
sdp file with ipv6 address
Comment 2 Tero Saarni 2008-07-05 12:33:39 UTC 
Created attachment 114020 [details] [review]
suggested fix
Comment 3 Sebastian Dröge (slomo) 2008-07-05 19:00:56 UTC 
2008-07-05  Sebastian Dröge  <sebastian.droege@collabora.co.uk>

        Patch by: Tero Saarni <tero dot saarni at gmail dot com>

        * gst/udp/gstudpsrc.c: (gst_udpsrc_set_uri):
        Fix parsing of udp:// URIs containing IPv6 addresses.
        Fixes bug #541650.