Skip to content

Commit

Permalink
juice, refactor: reset fingerprint in remote description when disable…
Browse files Browse the repository at this point in the history
…FingerprintVerification.
  • Loading branch information
xicilion committed May 19, 2024
1 parent 67bce3d commit bc4aec8
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 3 deletions.
11 changes: 9 additions & 2 deletions datachannel/src/impl/peerconnection.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -429,13 +429,20 @@ void PeerConnection::rollbackLocalDescription() {
}
}

bool PeerConnection::checkFingerprint(const std::string &fingerprint) const {
bool PeerConnection::checkFingerprint(const std::string &fingerprint) {
std::lock_guard lock(mRemoteDescriptionMutex);
if (!mRemoteDescription || !mRemoteDescription->fingerprint())
return false;

if (config.disableFingerprintVerification)
if (config.disableFingerprintVerification) {
CertificateFingerprint fp;

fp.algorithm = mRemoteDescription->fingerprint()->algorithm;
fp.value = fingerprint;
mRemoteDescription->setFingerprint(fp);

return true;
}

auto expectedFingerprint = mRemoteDescription->fingerprint()->value;
if (expectedFingerprint == fingerprint) {
Expand Down
2 changes: 1 addition & 1 deletion datachannel/src/impl/peerconnection.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ struct PeerConnection : std::enable_shared_from_this<PeerConnection> {

void endLocalCandidates();
void rollbackLocalDescription();
bool checkFingerprint(const std::string &fingerprint) const;
bool checkFingerprint(const std::string &fingerprint);
void forwardMessage(message_ptr message);
void forwardMedia(message_ptr message);
void forwardBufferedAmount(uint16_t stream, size_t amount);
Expand Down

0 comments on commit bc4aec8

Please sign in to comment.