Details

    • Type: Bug Bug
    • Status: Closed
    • Severity: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Target Release Version/s: None
    • Component/s: Channels/chan_mgcp
    • Labels:
      None
    • SVN Revision Number:
      151979
    • Mantis ID:
      13785
    • Regression:
      No

      Description

      When reloading chan_mgcp.so, we are not closing mgcpsock and (if the file didn change) so not calling ast_io_remove (if the file didn't change), so we dont need to call again ast_io_add unless mgcpsock_read_id is set to NULL.

                • ADDITIONAL INFORMATION ******

      To reproduce the leak do:
      CLI*> memory show summary io.c
      CLI*> module reload chan_mgcp.so
      CLI*> memory show summary io.c

        Activity

        Hide
        Digium Subversion added a comment -

        Repository: asterisk
        Revision: 152442

        U trunk/channels/chan_mgcp.c

        ------------------------------------------------------------------------
        r152442 | tilghman | 2008-10-28 16:27:37 -0500 (Tue, 28 Oct 2008) | 7 lines

        Only re-add the io port if it was closed, otherwise reload causes a memory
        leak.
        (closes issue ASTERISK-12967)
        Reported by: eliel
        Patches:
        chan_mgcp.c.patch uploaded by eliel (license 64)

        ------------------------------------------------------------------------

        http://svn.digium.com/view/asterisk?view=rev&revision=152442

        Show
        Digium Subversion added a comment - Repository: asterisk Revision: 152442 U trunk/channels/chan_mgcp.c ------------------------------------------------------------------------ r152442 | tilghman | 2008-10-28 16:27:37 -0500 (Tue, 28 Oct 2008) | 7 lines Only re-add the io port if it was closed, otherwise reload causes a memory leak. (closes issue ASTERISK-12967 ) Reported by: eliel Patches: chan_mgcp.c.patch uploaded by eliel (license 64) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=152442

          People

          • Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development