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 794800 - GstDtlsConnection throws a warning every minute
GstDtlsConnection throws a warning every minute
Status: RESOLVED OBSOLETE
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
git master
Other Linux
: Normal normal
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2018-03-29 10:22 UTC by Andreas Frisch
Modified: 2018-11-03 14:20 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
dtls: Properly display all errors/warnings from ERR queue (3.70 KB, patch)
2018-03-29 11:38 UTC, Andreas Frisch
none Details | Review

Description Andreas Frisch 2018-03-29 10:22:09 UTC
During a WebRTC session, we've encountered

dtlsconnection gstdtlsconnection.c:747:openssl_poll:<GstDtlsConnection@0x7fb63c0aa160> do_handshake encountered BIO error

after 15, 30, 60 seconds and then every minute.

further digging shows that the BIO read gets an EOF and the SSL syscall errno is actually 0 (success) and the error queue is empty.

it may indicate that the peer simply terminated the TCP connection.
maybe something goes wrong with rtcp here, or is this intentional behaviour?
Comment 1 Matthew Waters (ystreet00) 2018-03-29 10:32:36 UTC
This, unfortunately seems to be reasonably normal behaviour for openssl/dtls elements.  The question is if it causes any problems.
Comment 2 Andreas Frisch 2018-03-29 11:02:19 UTC
that's what we thought.
it doesn't seem to be triggering any apparent problems.
we were thinking about de-escalating that WARNING to a DEBUG
Comment 3 Matthew Waters (ystreet00) 2018-03-29 11:09:50 UTC
Or detecting the success case and not throwing the warning would probably be better :)
Comment 4 Andreas Frisch 2018-03-29 11:38:21 UTC
Created attachment 370289 [details] [review]
dtls: Properly display all errors/warnings from ERR queue
Comment 5 Nirbheek Chauhan 2018-03-31 06:14:43 UTC
Review of attachment 370289 [details] [review]:

::: ext/dtls/gstdtlsconnection.c
@@ +775,3 @@
+    case SSL_ERROR_SYSCALL:
+      GST_LOG_OBJECT (self, "SSL syscall error: errno %d: %s", errno,
+          strerror (errno));

Since you have errno, maybe upgrade the log to a warning if errno != 0?
Comment 6 GStreamer system administrator 2018-11-03 14:20:13 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/issues/677.