I think this PR is completely wrong as it was done, because the API exports the functions
for kamailio.cfg, which take pseudo-variables in parameters that are fixed-up at startup,
thus at runtime the parameters are expected to be pv-string-parsed pointer, not pointer to
a string value. I expect crashes on minimal tests as it is done right now. Or the user of
the API functions, as they were done, have to call the corresponding fixup functions and
provide the result to the functions, afterwards also free the fixed-up result (which are
not set right now), otherwise will be memory leaks.
Probably the API should export the corresponding kemi functions, which work directly with
`str*` parameters.
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3948#issuecomment-2296756994
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3948/c2296756994(a)github.com>