[OpenSIPS-Devel] [OpenSIPS/opensips] 0f86ee: clusterer: Fix sync edge-case causing SHM accumula...
Liviu Chircu
noreply at github.com
Thu Oct 24 16:04:01 UTC 2024
Branch: refs/heads/3.5
Home: https://github.com/OpenSIPS/opensips
Commit: 0f86eeb8eaca7050e80f47cf62b77dae5689c049
https://github.com/OpenSIPS/opensips/commit/0f86eeb8eaca7050e80f47cf62b77dae5689c049
Author: Liviu Chircu <liviu at opensips.org>
Date: 2024-10-24 (Thu, 24 Oct 2024)
Changed paths:
M modules/clusterer/sync.c
Log Message:
-----------
clusterer: Fix sync edge-case causing SHM accumulation
This patch fixes a specific sync sequence leading to indefinite
accumulation of BIN packets and SHM exhaustion, as follows:
1) Sync Receiver node times out the sync in sync_check_timer()
(default timeout is 5s, by no means difficult to achieve)
2) Sync Donor continues to send SYNC packets
(on Receiver side, cap state is updated: PROGRESSING)
3) ! Sync Donor node loses link-state mid-sync, due to the pings being
on the same TCP connection (and delayed, way in the back)
4) Sync Donor fails & drops remaining SYNC + SYNC-END packets, as link
is down. Due to this, the handle_sync_end(!is_timeout) procedure
is never ran on the Receiver, so the PROGRESSING flag is never
removed -> indefinite SHM buffering / memory leak
Thanks to Răzvan Crainea for helping with code & troubleshooting here!
(cherry picked from commit e5b2317ca9e9f33baef29205ecd792d8a5d5d2ed)
Commit: 82645f0ced6143ddf1cef37c9e73b25f7b11c605
https://github.com/OpenSIPS/opensips/commit/82645f0ced6143ddf1cef37c9e73b25f7b11c605
Author: Liviu Chircu <liviu at opensips.org>
Date: 2024-10-24 (Thu, 24 Oct 2024)
Changed paths:
M modules/clusterer/sync.c
Log Message:
-----------
clusterer sync: Fix several SHM leaks on error cases (Donor side)
(cherry picked from commit 101396a59c23f91bc821c36b9bc1bd2e5daa1d35)
Compare: https://github.com/OpenSIPS/opensips/compare/5170a0bdc459...82645f0ced61
To unsubscribe from these emails, change your notification settings at https://github.com/OpenSIPS/opensips/settings/notifications
More information about the Devel
mailing list