Bug in pj_generate_unique_string() ?

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

 



Thanks Roland, I've fixed this in r1956. But actually you should use
guid_win32.c on Windows rather than guid_simple.c, that's why this was
not tested.

Cheers
 Benny

On Wed, May 14, 2008 at 1:29 PM, Roland Klabunde
<roland.klabunde at freenet.de> wrote:
> Hi Benny,
>
>  I got this message from Thomas:
>
>  <snip>
>  Hi,
>
>  I found a bug in the file "os_core_win32.c"
>
>  The function pj_generate_unique_string() is called before initiializing
>  mutex but needs access to it.
>  You should change it this way:
>
>
>
>  --------------------------------------------------------------------------------
>
>
>  /* Initialize critical section. */
>
>  if ((rc=init_mutex(&critical_section_mutex, "pj%p")) != PJ_SUCCESS)
>     return rc;
>
>  /* Init random seed. */
>  /* Or probably not. Let application in charge of this */
>  /* pj_srand( GetCurrentProcessId() ); */
>  /* Startup GUID. */
>
>  guid.ptr = dummy_guid;
>  pj_generate_unique_string( &guid );
>
>  </snip>
>
>  Nearly the latest SVN version.
>
>  Regards
>
>
>
>
>
>
>
>  _______________________________________________
>  Visit our blog: http://blog.pjsip.org
>
>  pjsip mailing list
>  pjsip at lists.pjsip.org
>  http://lists.pjsip.org/mailman/listinfo/pjsip_lists.pjsip.org
>



[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