@alexyosifov requested changes on this pull request.
Just a small note:
Change: LM_ERR("Error using old IPSEC tunnel creation\n");
To: LM_DBG("Reusing old IPSEC tunnel creation\n");
In src/modules/ims_ipsec_pcscf/cmd.c:
> @@ -352,57 +351,26 @@ static int update_contact_ipsec_params(ipsec_t* s, const struct sip_msg* m, ipse s->ik.len = ik.len; // Generate SPI - if((s->spi_pc = acquire_spi()) == 0) { - LM_ERR("Error generating client SPI for IPSEC tunnel creation\n"); - shm_free(s->ck.s); - s->ck.s = NULL; s->ck.len = 0; - shm_free(s->ik.s); - s->ik.s = NULL; s->ik.len = 0; - return -1; + if(s_old) { + if(s_old->spi_pc && s_old->spi_ps && s_old->port_pc && s_old->port_ps) { + LM_ERR("Error using old IPSEC tunnel creation\n");
Change log message to debug or info.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.