[asterisk-users] sdp_crypto_process: Crypto life time unsupported: crypto

2014-10-09 Thread Jonas Kellens
Hello, I have added the following to the peer definition : ignorecryptolifetime=yes But still Asterisk tells me : [Oct 9 14:02:34] NOTICE[31980]: sip/sdp_crypto.c:244 sdp_crypto_process: Crypto life time unsupported: crypto:1 AES_CM_128_HMAC_SHA1_80

Re: [asterisk-users] sdp_crypto_process: Crypto life time unsupported: crypto

2014-10-09 Thread Joshua Colp
Jonas Kellens wrote: Hello, Kia ora, I have added the following to the peer definition : ignorecryptolifetime=yes This is not an option within the official tree so unless you've added a patch this won't actually do anything. But still Asterisk tells me : [Oct 9 14:02:34]

Re: [asterisk-users] sdp_crypto_process: Crypto life time unsupported: crypto

2014-10-09 Thread Jonas Kellens
On 09-10-14 14:11, Joshua Colp wrote: Jonas Kellens wrote: Hello, Kia ora, I have added the following to the peer definition : ignorecryptolifetime=yes This is not an option within the official tree so unless you've added a patch this won't actually do anything. But still Asterisk

Re: [asterisk-users] sdp_crypto_process: Crypto life time unsupported: crypto

2014-10-09 Thread Joshua Colp
Jonas Kellens wrote: Hello, any idea where and what to change in the source code then ? I am able to change the source code, but to do minimal damage I would like to know where to change what exactly. Yes. In channels/sip/sdp_crypto.c where the line: ast_log(LOG_NOTICE, Crypto life time

Re: [asterisk-users] sdp_crypto_process: Crypto life time unsupported: crypto

2014-10-09 Thread Jonas Kellens
On 09-10-14 14:28, Joshua Colp wrote: Jonas Kellens wrote: Hello, any idea where and what to change in the source code then ? I am able to change the source code, but to do minimal damage I would like to know where to change what exactly. Yes. In channels/sip/sdp_crypto.c where the line:

Re: [asterisk-users] sdp_crypto_process: Crypto life time unsupported: crypto

2014-10-09 Thread Joshua Colp
Jonas Kellens wrote: Ok this seems to work ! Thanks. Does Asterisk now ignore the SRTP crypto offer ? Or does it just ignore the lifetime (in this case : |2^32) ? It does not seem right that Asterisk now should ignore the whole crypto offer. That change makes the presence of a key lifetime a