| Index: talk/session/media/channel.cc
|
| diff --git a/talk/session/media/channel.cc b/talk/session/media/channel.cc
|
| index 8d3efec4fa5144ef9f6a5aa99816f29dfdf88b70..ef26704f1a4f0058d895e7fb3d539030c7a300e3 100644
|
| --- a/talk/session/media/channel.cc
|
| +++ b/talk/session/media/channel.cc
|
| @@ -218,11 +218,11 @@ bool BaseChannel::Init() {
|
| return false;
|
| }
|
|
|
| - if (!SetDtlsSrtpCiphers(transport_channel(), false)) {
|
| + if (!SetDtlsSrtpCryptoSuites(transport_channel(), false)) {
|
| return false;
|
| }
|
| if (rtcp_transport_enabled() &&
|
| - !SetDtlsSrtpCiphers(rtcp_transport_channel(), true)) {
|
| + !SetDtlsSrtpCryptoSuites(rtcp_transport_channel(), true)) {
|
| return false;
|
| }
|
|
|
| @@ -809,16 +809,16 @@ void BaseChannel::SignalDtlsSetupFailure_s(bool rtcp) {
|
| SignalDtlsSetupFailure(this, rtcp);
|
| }
|
|
|
| -bool BaseChannel::SetDtlsSrtpCiphers(TransportChannel *tc, bool rtcp) {
|
| - std::vector<std::string> ciphers;
|
| - // We always use the default SRTP ciphers for RTCP, but we may use different
|
| - // ciphers for RTP depending on the media type.
|
| +bool BaseChannel::SetDtlsSrtpCryptoSuites(TransportChannel* tc, bool rtcp) {
|
| + std::vector<int> crypto_suites;
|
| + // We always use the default SRTP crypto suites for RTCP, but we may use
|
| + // different crypto suites for RTP depending on the media type.
|
| if (!rtcp) {
|
| - GetSrtpCryptoSuiteNames(&ciphers);
|
| + GetSrtpCryptoSuites(&crypto_suites);
|
| } else {
|
| - GetDefaultSrtpCryptoSuiteNames(&ciphers);
|
| + GetDefaultSrtpCryptoSuites(&crypto_suites);
|
| }
|
| - return tc->SetSrtpCiphers(ciphers);
|
| + return tc->SetSrtpCryptoSuites(crypto_suites);
|
| }
|
|
|
| bool BaseChannel::ShouldSetupDtlsSrtp() const {
|
| @@ -837,10 +837,10 @@ bool BaseChannel::SetupDtlsSrtp(bool rtcp_channel) {
|
| if (!channel->IsDtlsActive())
|
| return true;
|
|
|
| - std::string selected_cipher;
|
| + int selected_crypto_suite;
|
|
|
| - if (!channel->GetSrtpCryptoSuite(&selected_cipher)) {
|
| - LOG(LS_ERROR) << "No DTLS-SRTP selected cipher";
|
| + if (!channel->GetSrtpCryptoSuite(&selected_crypto_suite)) {
|
| + LOG(LS_ERROR) << "No DTLS-SRTP selected crypto suite";
|
| return false;
|
| }
|
|
|
| @@ -896,21 +896,15 @@ bool BaseChannel::SetupDtlsSrtp(bool rtcp_channel) {
|
| }
|
|
|
| if (rtcp_channel) {
|
| - ret = srtp_filter_.SetRtcpParams(
|
| - selected_cipher,
|
| - &(*send_key)[0],
|
| - static_cast<int>(send_key->size()),
|
| - selected_cipher,
|
| - &(*recv_key)[0],
|
| - static_cast<int>(recv_key->size()));
|
| + ret = srtp_filter_.SetRtcpParams(selected_crypto_suite, &(*send_key)[0],
|
| + static_cast<int>(send_key->size()),
|
| + selected_crypto_suite, &(*recv_key)[0],
|
| + static_cast<int>(recv_key->size()));
|
| } else {
|
| - ret = srtp_filter_.SetRtpParams(
|
| - selected_cipher,
|
| - &(*send_key)[0],
|
| - static_cast<int>(send_key->size()),
|
| - selected_cipher,
|
| - &(*recv_key)[0],
|
| - static_cast<int>(recv_key->size()));
|
| + ret = srtp_filter_.SetRtpParams(selected_crypto_suite, &(*send_key)[0],
|
| + static_cast<int>(send_key->size()),
|
| + selected_crypto_suite, &(*recv_key)[0],
|
| + static_cast<int>(recv_key->size()));
|
| }
|
|
|
| if (!ret)
|
| @@ -1580,9 +1574,8 @@ void VoiceChannel::OnAudioMonitorUpdate(AudioMonitor* monitor,
|
| SignalAudioMonitor(this, info);
|
| }
|
|
|
| -void VoiceChannel::GetSrtpCryptoSuiteNames(
|
| - std::vector<std::string>* ciphers) const {
|
| - GetSupportedAudioCryptoSuites(ciphers);
|
| +void VoiceChannel::GetSrtpCryptoSuites(std::vector<int>* crypto_suites) const {
|
| + GetSupportedAudioCryptoSuites(crypto_suites);
|
| }
|
|
|
| VideoChannel::VideoChannel(rtc::Thread* thread,
|
| @@ -1970,9 +1963,8 @@ bool VideoChannel::GetLocalSsrc(const VideoCapturer* capturer, uint32_t* ssrc) {
|
| return false;
|
| }
|
|
|
| -void VideoChannel::GetSrtpCryptoSuiteNames(
|
| - std::vector<std::string>* ciphers) const {
|
| - GetSupportedVideoCryptoSuites(ciphers);
|
| +void VideoChannel::GetSrtpCryptoSuites(std::vector<int>* crypto_suites) const {
|
| + GetSupportedVideoCryptoSuites(crypto_suites);
|
| }
|
|
|
| DataChannel::DataChannel(rtc::Thread* thread,
|
| @@ -2278,9 +2270,8 @@ void DataChannel::OnDataChannelReadyToSend(bool writable) {
|
| new DataChannelReadyToSendMessageData(writable));
|
| }
|
|
|
| -void DataChannel::GetSrtpCryptoSuiteNames(
|
| - std::vector<std::string>* ciphers) const {
|
| - GetSupportedDataCryptoSuites(ciphers);
|
| +void DataChannel::GetSrtpCryptoSuites(std::vector<int>* crypto_suites) const {
|
| + GetSupportedDataCryptoSuites(crypto_suites);
|
| }
|
|
|
| bool DataChannel::ShouldSetupDtlsSrtp() const {
|
|
|