Git-Mirrors/RetroShare
1
0
Fork 0
mirror of https://github.com/RetroShare/RetroShare.git synced 2024-10-01 02:35:48 -04:00
Code Issues Actions Packages Projects Releases Wiki Activity

Merge pull request #1623 from G10h4ck/fix_crash_in_p3banlist

Browse Source 
Protect p3BanList::isAddressAccepted with mutex
This commit is contained in:
G10h4ck 2019-08-28 23:39:08 +02:00 committed by GitHub
commit 9f95649bbc
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 7 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
libretroshare/src/services/p3banlist.cc
View File

@ -306,18 +306,23 @@ bool p3BanList::acceptedBanRanges_locked(const BanListPeer& blp)
} }
return false ; return false ;
} }
bool p3BanList::isAddressAccepted( bool p3BanList::isAddressAccepted(
const sockaddr_storage& dAddr, uint32_t checking_flags, const sockaddr_storage& dAddr, uint32_t checking_flags,
uint32_t& check_result ) uint32_t& check_result )
{ {
check_result = RSBANLIST_CHECK_RESULT_NOCHECK; check_result = RSBANLIST_CHECK_RESULT_NOCHECK;
if(!mIPFilteringEnabled) return true;
sockaddr_storage addr; sockaddr_storage_copy(dAddr, addr); sockaddr_storage addr; sockaddr_storage_copy(dAddr, addr);
if(!sockaddr_storage_ipv6_to_ipv4(addr)) return true; if(!sockaddr_storage_ipv6_to_ipv4(addr)) return true;
if(sockaddr_storage_isLoopbackNet(addr)) return true; if(sockaddr_storage_isLoopbackNet(addr)) return true;
RS_STACK_MUTEX(mBanMtx);
if(!mIPFilteringEnabled) return true;
#ifdef DEBUG_BANLIST #ifdef DEBUG_BANLIST
std::cerr << "isAddressAccepted(): tested addr=" << sockaddr_storage_iptostring(addr) << ", checking flags=" << checking_flags ; std::cerr << "isAddressAccepted(): tested addr=" << sockaddr_storage_iptostring(addr) << ", checking flags=" << checking_flags ;
#endif #endif
@ -409,6 +414,7 @@ bool p3BanList::isAddressAccepted(
check_result = RSBANLIST_CHECK_RESULT_ACCEPTED; check_result = RSBANLIST_CHECK_RESULT_ACCEPTED;
return true ; return true ;
} }
void p3BanList::getWhiteListedIps(std::list<BanListPeer> &lst) void p3BanList::getWhiteListedIps(std::list<BanListPeer> &lst)
{ {
RS_STACK_MUTEX(mBanMtx) ; RS_STACK_MUTEX(mBanMtx) ;
@ -582,11 +588,6 @@ int p3BanList::tick()
return 0; return 0;
} }
int p3BanList::status()
{
return 1;
}
void p3BanList::getDhtInfo() void p3BanList::getDhtInfo()
{ {
// Get the list of masquerading peers from the DHT. Add them as potential IPs to be banned. // Get the list of masquerading peers from the DHT. Add them as potential IPs to be banned.

1
libretroshare/src/services/p3banlist.h
View File

@ -105,7 +105,6 @@ public:
*/ */
virtual int tick(); virtual int tick();
virtual int status();
int sendPackets(); int sendPackets();
bool processIncoming(); bool processIncoming();