<div dir="ltr">Hi Holger,<br><br>Yes, you are right, I retested this patch and it is enough to move only OSMO_ASSERT(conn).<br>I updated patch for this issue.<br></div><div class="gmail_extra"><br><div class="gmail_quote">2015-05-27 13:09 GMT+03:00 Holger Freyther <span dir="ltr"><<a href="mailto:holger@freyther.de" target="_blank">holger@freyther.de</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
> On 26 May 2015, at 12:53, Ivan Kluchnikov <<a href="mailto:kluchnikovi@gmail.com">kluchnikovi@gmail.com</a>> wrote:<br>
<br>
<br>
HI,<br>
<span class=""><br>
> setup_trig_pag_evt function can receive parameter conn = NULL, if T3113 expires.<br>
<br>
</span>ah thank you very much for the patch and you are obviously right that in case the<br>
paging times out there will be no subscriber connection.<br>
<span class=""><br>
<br>
> ---<br>
> openbsc/src/libmsc/gsm_04_08.c |    5 ++---<br>
> 1 file changed, 2 insertions(+), 3 deletions(-)<br>
><br>
> diff --git a/openbsc/src/libmsc/gsm_04_08.c b/openbsc/src/libmsc/gsm_04_08.c<br>
> index 5609602..d45ae08 100644<br>
> --- a/openbsc/src/libmsc/gsm_04_08.c<br>
> +++ b/openbsc/src/libmsc/gsm_04_08.c<br>
> @@ -1389,13 +1389,12 @@ static int setup_trig_pag_evt(unsigned int hooknum, unsigned int event,<br>
>       struct gsm_subscriber_connection *conn = _conn;<br>
>       struct gsm_trans *transt = _transt;<br>
><br>
> -     OSMO_ASSERT(!transt->conn);<br>
<br>
</span>Is there any reason to move this assert as well? setup_trig_pag_evt should be<br>
called exactly once right? And there should be no pre-existing subscriber connection?<br>
Do you have time to re-test with just moving one of the two asserts?<br>
<br>
<br>
thank you<br>
<span class="HOEnZb"><font color="#888888"><br>
        holger</font></span></blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature">Regards,<br>Ivan Kluchnikov.<br><a href="http://fairwaves.ru/" target="_blank">http://fairwaves.ru</a><br>
</div>
</div>