Skip to content
Snippets Groups Projects
  1. Dec 19, 2017
    • Corey Farrell's avatar
      app_voicemail: Fix memory management issues. · dc04d1ec
      Corey Farrell authored
      * mwi_sub_event_cb: mwist leaked on separate_mailbox failure.
      * add_email_attachment: A reference to sox_gain_tmpdir was used
        after the storage was out of scope.
      
      Change-Id: I6282c542ff7b82fa091177a912d11234a8b00a30
      dc04d1ec
  2. Dec 18, 2017
  3. Dec 16, 2017
    • Richard Mudgett's avatar
      chan_pjsip.c: Improve ast_request() diagnostic msgs. · 4a461bcd
      Richard Mudgett authored
      Attempting to dial PJSIP/endpoint when the endpoint doesn't exist and
      disable_multi_domain=no results in a misleading empty endpoint name
      message.  The message should say the endpoint was not found.
      
      * Added missing endpoint not found message.
      
      * Added more information to the empty endpoint name msgs if available.
      
      * Eliminated RAII_VAR in request().
      
      Change-Id: I21da85ebd62dcc32115b2ffcb5157416ebae51e4
      4a461bcd
  4. Dec 15, 2017
  5. Dec 14, 2017
    • Corey Farrell's avatar
      loader: Rework of load_dynamic_module. · 3505cc88
      Corey Farrell authored
      * Split off load_dlopen to perform actual dlopen, check results and log
        warnings when needed.
      * Always use RTLD_NOW.
      * Use flags which minimize number of calls to dlopen required.  First
        attempt always uses RTLD_GLOBAL when global_symbols_only is enabled,
        RTLD_LOCAL when it is not.
      
      This patch significantly reduces the number of dlopen's performed.  With
      299 modules my system ran dlopen 857 times before this patch, 655 times
      after this patch.
      
      Change-Id: Ib2c9903cfddcc01aed3e01c1e7fe4a3fb9af0f8b
      3505cc88
    • Corey Farrell's avatar
      loader: Minor fix to module registration. · 80bf0ee9
      Corey Farrell authored
      This protects the module loader itself against crashing if dlopen is
      called on a module from outside loader.c.
      
      * Expand scope of lock inside ast_module_register to include reading of
        resource_being_loaded.
      * NULL check resource_being_loaded.
      * Set resource_being_loaded NULL as soon as dlopen returns.  This fixes
        some error paths where it was not NULL'ed.
      * Create module_destroy function to deduplicate code from
        ast_module_unregister and modules_shutdown.
      * Resolve leak that occured if a module did not successfully register.
      * Simplify checking for successful registration.
      
      Change-Id: I40f07a315e55b92df4fc7faf525ed6d4f396e7d2
      80bf0ee9
    • Jenkins2's avatar
    • Corey Farrell's avatar
      res_clialiases: Fix completion pass-through. · a8aa2099
      Corey Farrell authored
      Never ignore contents of line when generating completion options.
      
      Change-Id: I74389efdfea154019d3b56a9f381610614c044c8
      a8aa2099
    • Jenkins2's avatar
    • Richard Mudgett's avatar
      res_rtp_asterisk.c: Disable packet flood detection for video streams. · 98f7e925
      Richard Mudgett authored
      We should not do flood detection on video RTP streams.  Video RTP streams
      are very bursty by nature.  They send out a burst of packets to update the
      video frame then wait for the next video frame update.  Really only audio
      streams can be checked for flooding.  The others are either bursty or
      don't have a set rate.
      
      * Added code to selectively disable packet flood detection for video RTP
      streams.
      
      ASTERISK-27440
      
      Change-Id: I78031491a6e75c2d4b1e9c2462dc498fe9880a70
      98f7e925
    • George Joseph's avatar
      res_pjsip_sdp_rtp: Add NULL check in add_crypto_to_stream · 283d2df6
      George Joseph authored
      add_crypto_to_stream wasn't checking for a NULL
      session->inv_session->neg before calling pjmedia_sdp_neg_get_state.
      This was causing a crash if the negotiation hadn't already been
      completed and asterisk was compiled with --enable-dev-mode.
      
      Change-Id: I57c6229954a38145da9810fc18657bfcc4d9d0c9
      283d2df6
    • Sean Bright's avatar
      res_musiconhold: Start playlist after initial announcement · c387beb4
      Sean Bright authored
      Reset the samples counter to zero when we are done playing an
      announcement so that we don't skip into the middle of the first file in
      the playlist.
      
      Also add the selected annoucement to the output of 'moh show classes.'
      
      ASTERISK-24329 #close
      Reported by: Thomas Frederiksen
      
      Change-Id: I2a5f986a31279c981592f49391409ebf38d6f6d0
      c387beb4
Loading