About duplicating the concept from preload_udomain() - this is generally fine, 
but the purpose of this function is a different. The preload_udomain() loads 
entries during startup to keep them in the cache and expire them later during 
runtime. This is not used in mode DB_ONLY. If I understand your code correctly, 
it just load all the entries to call the expiry callback and then it is not 
needed anymore.

You call mem_insert_urecord(), this calls new_urecord(), and this allocates new 
memory with shm_malloc(). As you are running in DB_ONLY mode, you should not 
keep any urecord entries after the function completes according to my 
understanding of your patch.

I would suggest to execute your patch in Kamailio debugging mode, maybe add a 
bit of logging to the places I mentioned, and verify this allocation and 
de-allocation of the memory.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/1683#issuecomment-432943165
_______________________________________________
Kamailio (SER) - Development Mailing List
[email protected]
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

Reply via email to