Skip to content
Snippets Groups Projects
  1. Mar 08, 2017
  2. Mar 07, 2017
    • zuul's avatar
      9cd1a754
    • Mark Michelson's avatar
      res_http_websocket: Fix faulty read logic. · 5d0371d7
      Mark Michelson authored
      When doing some WebRTC testing, I found that the websocket would
      disconnect whenever I attempted to place a call into Asterisk. After
      looking into it, I pinpointed the problem to be due to the iostreams
      change being merged in.
      
      Under certain circumstances, a call to ast_iostream_read() can return a
      negative value. However, in this circumstance, the websocket code was
      treating this negative return as if it were a partial read from the
      websocket. The expected length would get adjusted by this negative
      value, resulting in the expected length being too large.
      
      This patch simply adds an if check to be sure that we are only updating
      the expected length of a read when the return from a read is positive.
      
      ASTERISK-26842 #close
      Reported by Mark Michelson
      
      Change-Id: Ib4423239828a013d27d7bc477d317d2f02db61ab
      5d0371d7
  3. Mar 06, 2017
    • Daniel Journo's avatar
      Saynumber is trying to get "and" from "digits/" subfolder · 272259a2
      Daniel Journo authored
      * say.c Changed 'digits/and' to 'vm-and' for en_GB
      
      ASTERISK-26598 #close
      
      Change-Id: If1b713e5daea6f952b339f139178d292a6c4fcfe
      272259a2
    • Sean Bright's avatar
      pbx_spool: Gracefully handle long lines in call files · 5a74abc5
      Sean Bright authored
      Per the linked issue, we aren't checking the buffer filled by fgets()
      to determine if it contains a newline, so we will fail to correctly
      parse the trailing portion of a long line.
      
      This patch increases the buffer size from 256 to 1024, and skips any
      line that exceeds that length, logging a warning in the process.
      
      ASTERISK-17067 #close
      Reported by: Dave Olszewski
      
      Change-Id: I51bcf270c1b4347ba05b43f18dc2094c76f5d7b0
      5a74abc5
  4. Mar 03, 2017
    • Richard Mudgett's avatar
      core: Cleanup ast_get_hint() usage. · c9296b23
      Richard Mudgett authored
      * manager.c:manager_state_cb() Fix potential use of uninitialized hint[]
      if a hint does not exist for the requested extension.  Ran into this when
      developing a testsuite test.  The AMI event ExtensionStatus came out with
      the hint header value containing garbage.  The AMI event PresenceStatus
      also had the same issue.
      
      * manager.c:action_extensionstate() no need to completely initialize the
      hint[].  Only initialize the first element.
      
      * pbx.c:ast_add_hint() Remove unnecessary assignment.
      
      * chan_sip.c: Eliminate an unneeded hint[] local variable.  We only care
      about the return value of ast_get_hint() there.
      
      Change-Id: Ia9a8786f01f93f1f917200f0a50bead0319af97b
      c9296b23
  5. Mar 01, 2017
  6. Feb 28, 2017
  7. Feb 27, 2017
    • Jørgen H's avatar
      res_pjsip: Fix crash when contact has no status · ee0a123f
      Jørgen H authored
      This change fixes an assumption in res_pjsip that a contact will
      always have a status. There is a race condition where this is
      not true and would crash. The status will now be unknown when
      this situation occurs.
      
      ASTERISK-26623 #close
      
      Change-Id: Id52d3ca4d788562d236da49990a319118f8d22b5
      ee0a123f
    • George Joseph's avatar
      res_pjsip_outbound_registration: Subscribe to network change events · 22242fef
      George Joseph authored
      Outbound registration now subscribes to network change events
      published by res_stun_monitor and refreshes all registrations
      when an event happens.
      
      The 'pjsip send (un)register' CLI commands were updated to accept
      '*all' as an argument to operate on all registrations.
      
      The 'PJSIP(Un)Register' AMI commands were also updated to
      accept '*all'.
      
      ASTERISK-26808 #close
      
      Change-Id: Iad58a9e0aa5d340477fca200bf293187a6ca5a25
      22242fef
    • George Joseph's avatar
      build: Warn if asterisk is installed in both 32 and 64 bit sys dirs · 4692a32e
      George Joseph authored
      ... and clean them both up on uninstall.
      
      We've fixed the issue where 'make install' was installing to
      /usr/lib on 64-bit systems that use /usr/lib64.  Now we need
      to clean up the remnants in /usr/lib.
      
      * 'make install' now prints a warning if DESTDIR/ASTLIBDIR
        contains 'lib64' and libasterisk* shared libraries or modules
        are also found in DESTDIR/ASTLIBDIR with 'lib64' transformed
        to 'lib'.
      
      * 'make uninstall' ALWAYS cleans up both DESTDIR/ASTLIBDIR and
        DESTDIR/ASTLIBDIR with 'lib64' transformed to 'lib'.
      
      ASTERISK-26705
      
      Change-Id: I6edddeb3c07a51e7c7ba7cac3c05e4bf3ec3f01f
      4692a32e
    • Joshua Colp's avatar
      bridge_native_rtp: Handle case where channel joins already suspended. · ff2b4308
      Joshua Colp authored
      The bridge_native_rtp module did not properly handle the case where
      a smart bridge operation occurs while a channel is suspended. In this
      scenario the module would incorrectly set up local or remote RTP
      bridging despite the media having to flow through Asterisk. The remote
      endpoint would see two media streams and experience wonky audio.
      
      The module has been changed so that it ensures both channels are
      not suspended when performing the native RTP bridging and this
      requirement has been documented in the bridge technology.
      
      ASTERISK-26781
      
      Change-Id: Id4022d73ace837d4a293106445e3ade10dbc7c7c
      ff2b4308
    • George Joseph's avatar
  8. Feb 24, 2017
  9. Feb 23, 2017
    • George Joseph's avatar
      pjproject_bundled: Update for pjproject 2.6 · a537dae6
      George Joseph authored
       * Removed all 2.5.5 functional patches.
       * Updated usages of pj_release_pool to be "safe".
       * Updated configure options to disable webrtc.
       * Updated config_site.h to disable webrtc in pjmedia.
       * Added Richard Mudgett's recent resolver patches.
      
      Change-Id: Ib400cc4dfca68b3d07ce14d314e829bfddc252c7
      a537dae6
    • George Joseph's avatar
      build: Execute ldconfig to build cache. (take two) · b0067bcf
      George Joseph authored
      On some platforms a multiarch approach is used for libraries.
      The build system does not take this into account and still
      places libraries into the lib directory if no --libdir is
      specified to configure. On initial startup this results in
      libasteriskssl.so not being found, as it is not in the multiarch
      lib directory.  To make matters worse, options were being passed
      to ldconfig on both Linux and FreeBSD that actually prevented
      the rebuild of the cache.
      
       * Fedora has a /usr/share/config.site that automatically tells
         autoconf to use /usr/lib64 but CentOS does not. This logic was
         copied to configure.ac and modified so systems like Ubuntu,
         which still use /usr/lib for 64-bit systems, aren't affected.
      
      Now that we have them in the correct directory...
      
      In order for the system loader to find libasteriskssl and
      libasteriskpj, one of 3 things has to happen...
      
        - The linker cache must be rebuilt including the directory
          where the libasterisk* libraries were installed.  Only root
          can rebuild the cache.  This was busted.
        - We have to link the asterisk binary with an rpath pointing
          to the directrory where the libasterisk* libraries were
          installed.  This makes things very complicated and will happen
          over the collective dead bodies of everyone who's had to
          package a distribution with an rpath.
        - Finally, you can start asterisk with LD_LIBRARY_PATH set to the
          directrory where the libasterisk* libraries were installed.
      
      There are no other options. So...
      
       * The invokation of ldconfig has been moved from main/Makefile
         to ASTTOPDIR/Makefile, the options have been removed, and
         DESTDIR/ASTLIBDIR appended.  If you aren't root, you will be
         warned after the "Asterisk Installation Compete" banner that
         you must re-run 'make install' as root, manually run
         'ldconfig DESTDIR/ASTLIBDIR' as root, or run asterisk with
         LD_LIBRARY_PATH.
      
      ASTERISK-26705
      
      Change-Id: I2a64b7c33a7d3e9bde20f47e3d3ab771977af982
      b0067bcf
Loading