commit 321ec3ba99349271d631f35b79301fe4538cb36f
author Harald Alvestrand <hta@webrtc.org> Thu Feb 10 13:42:18 2022
committer WebRTC LUCI CQ <webrtc-scoped@luci-project-accounts.iam.gserviceaccount.com> Thu Feb 10 15:52:16 2022
tree 1542c4507e3fc5eeedd993b80dd9b3fbeb377752
parent cb03e385ad9d20547300f722820047ab229a9530

Add tests for SSL role and datachannel ID assignment.

Document with a comment the suspected place that could cause a bug.
Also fix an error in previous role observation code.

Bug: webrtc:13668
Change-Id: Id7f6af6905d90f7974b5570145c201c8339aaf72
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/251388
Reviewed-by: Henrik Boström <hbos@webrtc.org>
Commit-Queue: Harald Alvestrand <hta@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#35973}

pc/dtls_transport.cc

diff --git a/pc/dtls_transport.cc b/pc/dtls_transport.cc
index 5cee54c..e8d6ae9 100644
--- a/pc/dtls_transport.cc
+++ b/pc/dtls_transport.cc
@@ -114,10 +114,10 @@
       if (success) {
         switch (internal_role) {
           case rtc::SSL_CLIENT:
-            role = DtlsTransportTlsRole::kServer;
+            role = DtlsTransportTlsRole::kClient;
             break;
           case rtc::SSL_SERVER:
-            role = DtlsTransportTlsRole::kClient;
+            role = DtlsTransportTlsRole::kServer;
             break;
         }
       }