[pjsip] Assertion failure in today's SVN 20080407

Benny Prijono bennylp at pjsip.org
Mon Apr 7 09:19:37 EDT 2008


On Mon, Apr 7, 2008 at 10:53 AM, Michael CHRISTOPHER <pjsip at encambio.com> wrote:
>
>  Hello list,
>
>  when using a 'config_site.h' with:
>
>   #define PJ_CONFIG_MINIMAL_SIZE
>   #include <pj/config_site_sample.h>
>
>  ...the file pj/config_site_sample.h defines:
>
>   #define PJ_IOQUEUE_MAX_HANDLES 16
>
>  This seems to be too low, because later on in line 45 of
>  pjlib/src/pj/sock_select.c:
>
>   pj_assert(sizeof(pj_fd_set_t)-sizeof(pj_sock_t) >= sizeof(fd_set));
>
>  The size of 'fd_set' is 128 at least on the Debian Linux on ARM
>  platform I'm using (Maemo SDK with scratchbox GCC compiler). I'll
>  leave it up to you to do the math, but the assertion will fail.
>
>  Because pj_fd_set_t is an opaque buffer for fd_set, it seems
>  reasonable to increase the value of PJ_IOQUEUE_MAX_HANDLES to
>  28 or even 32 should rounding off be desirable:
>  Index: pjlib/include/pj/config_site_sample.h
>  --- pjlib/include/pj/config_site_sample.h.orig  2008-04-07
>  11:19:39.000000000 +0200
>  +++ pjlib/include/pj/config_site_sample.h       2008-04-07
>  11:14:13.000000000 +0200
>  @@ -112,7 +112,7 @@
>   #   define PJ_ENABLE_EXTRA_CHECK       0
>   #   define PJ_HAS_ERROR_STRING         0
>   #   undef PJ_IOQUEUE_MAX_HANDLES
>  -#   define PJ_IOQUEUE_MAX_HANDLES      16
>  +#   define PJ_IOQUEUE_MAX_HANDLES      32
>   #   define PJ_IOQUEUE_HAS_SAFE_UNREG   0
>   #   define PJSIP_MAX_TSX_COUNT         15
>   #   define PJSIP_MAX_DIALOG_COUNT      15

Hi Michael,

First of all thanks for reporting your progress on running pjsip on
Maemo, those have been very useful for me (and I'm sure for others),
so keep them coming! I've changed MAX_HANDLES to 32
config_site_sample.h.

>  Is there a better way to report bugs than sending messages to
>  this general purpose mailing list?

I think this works fine so far, so I'd prefer to discuss stuffs like
these over the list.

Thanks
 Benny


>  Regards,
>  Michael




More information about the pjsip mailing list