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 757449 - udpsink: new unit test exposing IPv6 handling of IPv4 socket
udpsink: new unit test exposing IPv6 handling of IPv4 socket
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
git master
Other Linux
: Normal enhancement
: 1.12.5
Assigned To: GStreamer Maintainers
GStreamer Maintainers
: 782864 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2015-11-01 23:45 UTC by Sebastian Rasmussen
Modified: 2018-01-17 14:19 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Proposed patch 1/2 (2.85 KB, patch)
2015-11-01 23:50 UTC, Sebastian Rasmussen
committed Details | Review
Proposed patch 2/2 (1.14 KB, patch)
2015-11-01 23:50 UTC, Sebastian Rasmussen
committed Details | Review

Description Sebastian Rasmussen 2015-11-01 23:45:19 UTC
In the first patch in the series I add a new unit test for udpsink that attempts to set QoS DSCP for an IPv4 socket previously given to the sink. This results in the following error message:

0:00:00.053175184 24469      0x1711e60 ERROR           multiudpsink gstmultiudpsink.c:1056:gst_multiudpsink_setup_qos_dscp:<udpsink> could not set TCLASS: Protocol not available

Which, while true, is not pretty. The second patch in the series queries the family of the socket before attempting to use setsockopt() to set IPV6_TCLASS.
Comment 1 Sebastian Rasmussen 2015-11-01 23:50:15 UTC
Created attachment 314597 [details] [review]
Proposed patch 1/2
Comment 2 Sebastian Rasmussen 2015-11-01 23:50:37 UTC
Created attachment 314598 [details] [review]
Proposed patch 2/2
Comment 3 Sebastian Dröge (slomo) 2015-11-02 07:34:11 UTC
Comment on attachment 314598 [details] [review]
Proposed patch 2/2

Should the other one also only be done if the socket family is V4?
Comment 4 Sebastian Rasmussen 2015-11-02 07:48:53 UTC
Perhaps, let me look into this a bit further.
Comment 5 Tim-Philipp Müller 2017-12-23 11:46:19 UTC
commit 5091b5f39ffd22052802c1c41ee568fef65efeb5 (HEAD -> master)
Author: Sebastian Rasmussen <sebrn@hotmail.com>
Date:   Mon Nov 2 00:41:28 2015 +0100

    multiudpsink: don't try to set IPV6_TCLASS on IPV4 sockets
    
    Avoids ERROR log message.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=757449

commit 0d57709d388f548910dcab8b193ef85bb0de7341
Author: Sebastian Rasmussen <sebrn@hotmail.com>
Date:   Mon Nov 2 00:41:28 2015 +0100

    tests: udpsink: add check that sets QoS on IPv4/6 sockets
    
    https://bugzilla.gnome.org/show_bug.cgi?id=757449
Comment 6 Tim-Philipp Müller 2018-01-15 16:55:31 UTC
*** Bug 782864 has been marked as a duplicate of this bug. ***