If FS is not behind NAT, then every call generates at least three
INFO-level log messages:
[INFO] switch_nat.c:589 NAT port mapping disabled
This is useless noise. The message is only interesting if you do have
NAT enabled but mapping disabled, which might indicate a configuration
issue.
With this change, we just skip the entire nat_add_mapping function if
the NAT system isn't initialized or we're not behind NAT.
Store some initial details about the channel separately so we can
preserve them across e.g. transfers and display them with show
channels.
With this commit, we store the original caller ID (name and number)
and IP address, and the call destination, dialplan, and context.
FS-4296 --resolve
Thanks-to: Mariusz Czułada <manieq.net@gmail.com>
Store initial caller and destination numbers in 'channels' table,
so `show channels` can display it.
Patch provided by Mariusz Czułada <manieq.net@gmail.com>
The correct fix is to use gcc-4.7 or newer. If the compiler borked
this, one should trust nothing else it produces.
Revert "FS-6524,FS-6494: the compiler for rpi seems to be strangely
mis-optimizing this if/else block. Yes, its already set to this value,
but without the else block, it 'magically' hits the if blocks above,
unless you sleep, print a log, or add the else"
This reverts commit 1ca1d63795.
We were using the libopus directory itself rather than a file within
it as a makefile target. This is a bad idea as make will see the
target as updated whenever files are created in or removed from the
directory. Since running configure creates and removes files, the
target will always appear dirty.
The effect was that when mod_opus was enabled in FS make would always
cause libopus to be reconfigured.
We'll target instead the configure file in libopus as that's the first
thing we need anyway.
sip_header_as_string returns new memory alloc'd on the nua_handle's memory pool. This memory would not be freed until
the handle was destroyed. Since there is no usage of the call-info header after it is added to the event as a header value,
we should make sure to both su_free the sofia returned string, but also more cleanly create the event header value to reduce a strdup.
Travis Cross