<div dir="ltr"><div>Hi Henning,</div><div><br></div><div>Thank you a lot for your input.</div><div><br></div><div>But I was asking if there is a point to create patch from this 2 commits and apply to 4.4. Is it worth? Or there is no way to make this work properly on 4.4? As I see, some part of code what is touched by this 2 commits differs quite a lot, so I'm bit afraid to create patch and apply it to our production servers, especially if I don't have a clue what it affects. :)<br></div><div><br></div><div>With kind regards,<br></div></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">Jurijs<br></div></div></div>
<br><div class="gmail_quote">On Wed, Jun 13, 2018 at 11:02 PM, Henning Westerholt <span dir="ltr"><<a href="mailto:hw@kamailio.org" target="_blank">hw@kamailio.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">Am Mittwoch, 13. Juni 2018, 09:51:45 CEST schrieb Jurijs Ivolga:<br>
> I think I have this issue and I'm using 4.4 and I can't use master for now.<br>
> I tried to cherry pick this 2 commits, but unfortunately it do not work.<br>
> <br>
> For example commit 5fe2a1a1c67b550431dcae3c987010<wbr>73f7edd953 make changes in<br>
> function t_continue_helper, but 4.4 do not has such function, it has with<br>
> slightly different name - t_continue.<br>
> <br>
> Same commit add line 258 in src/modules/tm/t_suspend.c, but in 4.4 this<br>
> part of code is slightly different. There is no " t->flags &=<br>
> ~T_ASYNC_CONTINUE; " line in same if statement.<br>
> <br>
> There is no way to remove line 390 from same file, cause in 4.4 that part<br>
> of code differs quite a lot.<br>
> <br>
> With second patch 72f5eaeeef0239ebd16a2d645b83e8<wbr>3eb1a2b506 there was much<br>
> less problems, but still, there is big difference in part of code near line<br>
> 592 of this commit, but probably in 4.4 i just need to update line 527 and<br>
> change "UNREF_FREE(new_cell); " to " UNREF_FREE(new_cell, 0);"<br>
> <br>
> <br>
> Is it a worth to try to cherry pick this 2 commits or there are too much<br>
> changes between 4.4 and Master and no way to make this work properly?<br>
<br>
</span>Hello Juris,<br>
<br>
I don't know much details about your setup. But if you don't use a lot of <br>
custom code that needs to be touched before you can go to 5.1 then an update <br>
should be not difficult. There are also some other important fixes, some of <br>
them security relevant, that you miss as well if you stay on 4.4.<br>
<br>
Best regards,<br>
<br>
Henning<br>
<span class="HOEnZb"><font color="#888888"><br>
-- <br>
If you like the work that I do in Kamailio, please consider supporting me on <br>
Patreon: <a href="https://www.patreon.com/henningw" rel="noreferrer" target="_blank">https://www.patreon.com/<wbr>henningw</a><br>
</font></span></blockquote></div><br></div>