Hello,

On 26/09/16 12:12, Sebastian Damm wrote:

Hi,

Kamailio release 4.4.3 was tagged as a "bug fix release", no config changes should be needed.

no changes to good configs, but some bug fixes might reveal some invalid configs ... and, of course, from time to time might be regressions...

However, we had to change two things in our config file to get 4.4.3 running. I thought, I might share those on the list.

1. Defines must not have '-' in the names.

In 4.4.2 we had a Define like this:

#!define NO-FRAUDCHECK

This threw an error when trying to restart with 4.4.3. Replacing it with

#!define NO_FRAUDCHECK

fixed the problem.

This was a bug fix, due to a report on bug tracker when someone was having two route blocks with the name like ABC-XYZ and ABC-JKL, practically overwriting the route blocks and resulting in unexpected behaviour.

Given that minus is reserved character used for difference operations, an ID is ended by encountering it and the name was kept up to the sign.

2. Module Order problems.

We introduced the usage of http_async_client module a few weeks ago, and this worked perfectly. However, 4.4.3 complained that the pv module was not available before loading http_async_client. After moving up the pv module above http_async_client, everything worked again.

I haven't been involved directly in the fix, although being asked about the backport and I agreed to be done if it is a fix -- iirc, the issue was causing a crash and the fix was ensuring that the module is loaded in the proper order.

3. There's a bug in uac_auth function. I've opened an issue for that:

https://github.com/kamailio/kamailio/issues/797

This seems to be a regression of the cseq increment that was reported by one of your colleagues and he reported to be fixed after several patches pushed by myself. There is another related issue open on the tracker, but I got into traveling and didn't have the time to investigate more. It is in my short to-do list.

Cheers, Daniel