Skip to content

fix: unlock correct dht_friend#2327

Merged
sudden6 merged 1 commit into
TokTok:masterfrom
sudden6:fix/dht_lock
Aug 7, 2022
Merged

fix: unlock correct dht_friend#2327
sudden6 merged 1 commit into
TokTok:masterfrom
sudden6:fix/dht_lock

Conversation

@sudden6

@sudden6 sudden6 commented Aug 7, 2022

Copy link
Copy Markdown

Thanks @zugz for pointing out the bug.


This change is Reviewable

@sudden6 sudden6 requested a review from zugz August 7, 2022 13:34
@auto-add-label auto-add-label Bot added the bug Bug fix for the user, not a fix to a build script label Aug 7, 2022
@sudden6 sudden6 added this to the v0.2.19 milestone Aug 7, 2022
@codecov

codecov Bot commented Aug 7, 2022

Copy link
Copy Markdown

Codecov Report

Merging #2327 (306693a) into master (32ed67c) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #2327      +/-   ##
==========================================
- Coverage   78.93%   78.91%   -0.03%     
==========================================
  Files         127      127              
  Lines       24078    24078              
==========================================
- Hits        19007    19002       -5     
- Misses       5071     5076       +5     
Impacted Files Coverage Δ
toxcore/DHT.c 83.78% <100.00%> (-0.08%) ⬇️
toxcore/friend_connection.c 82.83% <0.00%> (-1.40%) ⬇️
toxav/groupav.c 73.91% <0.00%> (-0.67%) ⬇️
toxav/toxav.c 69.08% <0.00%> (-0.30%) ⬇️
toxcore/TCP_server.c 78.39% <0.00%> (-0.19%) ⬇️
toxcore/group.c 84.48% <0.00%> (+0.06%) ⬆️
toxcore/net_crypto.c 87.36% <0.00%> (+0.07%) ⬆️
toxcore/onion.c 85.79% <0.00%> (+0.28%) ⬆️
toxcore/TCP_connection.c 69.45% <0.00%> (+0.28%) ⬆️
... and 2 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@pull-request-attention pull-request-attention Bot assigned sudden6 and unassigned zugz Aug 7, 2022
@sudden6 sudden6 merged commit 306693a into TokTok:master Aug 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Bug fix for the user, not a fix to a build script

Development

Successfully merging this pull request may close these issues.

2 participants