bug in timer heap symbian + correction

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Oct 15, 2009 at 1:09 AM, nir elkayam <nir.elkayam at gmail.com> wrote:
>
> in timer_symbian.cpp:
>
> pj_int32_t? interval = PJ_TIME_VAL_MSEC(*delay) * 1000;
>
> but when the delay is too big (more then ~2100 sec) the 32 bit sign interval
> in not enougth to hold the timer time;
> i have modified to:
>
> ??? pj_int32_t interval;
> ??? if(delay->sec > 2000) {
> ??????? interval = 2000 * 1000 * 1000;
> ??? }
> ??? else {
> ??? ??? interval = PJ_TIME_VAL_MSEC(*delay) * 1000;
> ??? }
>
> this will also correct the resending publish as the expire time was about
> 3500 sec (at least with my server),
>

This is a very good catch indeed! I just created
http://trac.pjsip.org/repos/ticket/975 for this, will fix this asap.

Thanks
 Benny



[Index of Archives]     [Asterisk Users]     [Asterisk App Development]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [Linux API]
  Powered by Linux