Module: kamailio Branch: master Commit: 6b078c89facce89cfcd1e8cd8148f419b63ac5af URL: https://github.com/kamailio/kamailio/commit/6b078c89facce89cfcd1e8cd8148f419...

Author: Daniel-Constantin Mierla miconda@gmail.com Committer: Daniel-Constantin Mierla miconda@gmail.com Date: 2015-04-15T11:19:09+02:00

Merge pull request #126 from kamailio/coudot/tls_shm_available_bug

fix race condition in tls module when checking available memory limits

---

Modified: mem/shm_mem.c Modified: mem/shm_mem.h Modified: modules/tls/tls_server.c

---

Diff: https://github.com/kamailio/kamailio/commit/6b078c89facce89cfcd1e8cd8148f419... Patch: https://github.com/kamailio/kamailio/commit/6b078c89facce89cfcd1e8cd8148f419...

---

diff --git a/mem/shm_mem.c b/mem/shm_mem.c index de78213..5b6a0cf 100644 --- a/mem/shm_mem.c +++ b/mem/shm_mem.c @@ -254,5 +254,13 @@ void shm_mem_destroy(void) #endif }

+inline unsigned long safe_shm_available() +{ + unsigned long ret; + shm_lock(); + ret = shm_available(); + shm_unlock(); + return ret; +}

#endif diff --git a/mem/shm_mem.h b/mem/shm_mem.h index eadccb5..97032d4 100644 --- a/mem/shm_mem.h +++ b/mem/shm_mem.h @@ -312,6 +312,11 @@ do{\

#endif /* ! SHM_SAFE_MALLOC */

+/* multi-process safe version of shm_available() + */ +inline unsigned long safe_shm_available(); + + #endif /* shm_mem_h */

#endif /* SHM_MEM */ diff --git a/modules/tls/tls_server.c b/modules/tls/tls_server.c index 1d86140..be020ec 100644 --- a/modules/tls/tls_server.c +++ b/modules/tls/tls_server.c @@ -59,10 +59,10 @@ int tls_run_event_routes(struct tcp_connection *c); /* low memory treshold for openssl bug #1491 workaround */ #define LOW_MEM_NEW_CONNECTION_TEST() \ (cfg_get(tls, tls_cfg, low_mem_threshold1) && \ - (shm_available() < cfg_get(tls, tls_cfg, low_mem_threshold1))) + (safe_shm_available() < cfg_get(tls, tls_cfg, low_mem_threshold1))) #define LOW_MEM_CONNECTED_TEST() \ (cfg_get(tls, tls_cfg, low_mem_threshold2) && \ - (shm_available() < cfg_get(tls, tls_cfg, low_mem_threshold2))) + (safe_shm_available() < cfg_get(tls, tls_cfg, low_mem_threshold2)))

#define TLS_RD_MBUF_SZ 65536 #define TLS_WR_MBUF_SZ 65536