Asterisk
  1. Asterisk
  2. ASTERISK-14926

[patch] Clean valgrind output by suppressing false errors

    Details

    • Type: Bug Bug
    • Status: Closed
    • Severity: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Target Release Version/s: None
    • Component/s: Documentation
    • Labels:
      None
    • SVN Revision Number:
      221627
    • Mantis ID:
      16007
    • Regression:
      No

      Description

      There are some false errors that are flooding valgrind log.

      Attached small valgrind suppression file that filters out most common errors. It's supposed to be placed in utils/asterisk.supp but if other dir is chosen, it should be updated in valgrind.txt

      Also updated valgrind.txt so that

      • it doesn't write malloc_debug.txt from stderr (which never ever contained anything useful).
      • --log-fd is used instead of --log-file. This helps to avoid flood with "Use --log-fd=<number> to select an alternative log fd" messages.
      1. asterisk2.supp
        0.4 kB
        Atis Lezdins
      2. core-show-settings.txt
        1 kB
      3. valgrind.txt.diff
        1 kB
        Atis Lezdins

        Activity

        Hide
        Digium Subversion added a comment -

        Repository: asterisk
        Revision: 225484

        A branches/1.4/contrib/valgrind.supp
        U branches/1.4/doc/valgrind.txt

        ------------------------------------------------------------------------
        r225484 | lmadsen | 2009-10-22 16:56:20 -0500 (Thu, 22 Oct 2009) | 11 lines

        Clean valgrind output by suppressing false errors.
        Update valgrind.txt documentation and add valgrind.supp file in order to
        allow those who are creating valgrind output to have less false errors in
        the logfile.

        (closes issue ASTERISK-14926)
        Reported by: atis
        Patches:
        valgrind.txt.diff uploaded by atis (license 242)
        asterisk2.supp uploaded by atis (license 242)
        Tested by: atis, amorsen
        ------------------------------------------------------------------------

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

        Show
        Digium Subversion added a comment - Repository: asterisk Revision: 225484 A branches/1.4/contrib/valgrind.supp U branches/1.4/doc/valgrind.txt ------------------------------------------------------------------------ r225484 | lmadsen | 2009-10-22 16:56:20 -0500 (Thu, 22 Oct 2009) | 11 lines Clean valgrind output by suppressing false errors. Update valgrind.txt documentation and add valgrind.supp file in order to allow those who are creating valgrind output to have less false errors in the logfile. (closes issue ASTERISK-14926 ) Reported by: atis Patches: valgrind.txt.diff uploaded by atis (license 242) asterisk2.supp uploaded by atis (license 242) Tested by: atis, amorsen ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=225484
        Hide
        Digium Subversion added a comment -

        Repository: asterisk
        Revision: 225485

        _U trunk/
        A trunk/contrib/valgrind.supp
        U trunk/doc/valgrind.txt

        ------------------------------------------------------------------------
        r225485 | lmadsen | 2009-10-22 16:56:58 -0500 (Thu, 22 Oct 2009) | 19 lines

        Merged revisions 225484 via svnmerge from
        https://origsvn.digium.com/svn/asterisk/branches/1.4

        ........
        r225484 | lmadsen | 2009-10-22 16:51:52 -0500 (Thu, 22 Oct 2009) | 11 lines

        Clean valgrind output by suppressing false errors.
        Update valgrind.txt documentation and add valgrind.supp file in order to
        allow those who are creating valgrind output to have less false errors in
        the logfile.

        (closes issue ASTERISK-14926)
        Reported by: atis
        Patches:
        valgrind.txt.diff uploaded by atis (license 242)
        asterisk2.supp uploaded by atis (license 242)
        Tested by: atis, amorsen
        ........

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

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

        Show
        Digium Subversion added a comment - Repository: asterisk Revision: 225485 _U trunk/ A trunk/contrib/valgrind.supp U trunk/doc/valgrind.txt ------------------------------------------------------------------------ r225485 | lmadsen | 2009-10-22 16:56:58 -0500 (Thu, 22 Oct 2009) | 19 lines Merged revisions 225484 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r225484 | lmadsen | 2009-10-22 16:51:52 -0500 (Thu, 22 Oct 2009) | 11 lines Clean valgrind output by suppressing false errors. Update valgrind.txt documentation and add valgrind.supp file in order to allow those who are creating valgrind output to have less false errors in the logfile. (closes issue ASTERISK-14926 ) Reported by: atis Patches: valgrind.txt.diff uploaded by atis (license 242) asterisk2.supp uploaded by atis (license 242) Tested by: atis, amorsen ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=225485
        Hide
        Digium Subversion added a comment -

        Repository: asterisk
        Revision: 225486

        A branches/1.6.0/contrib/valgrind.supp
        U branches/1.6.0/doc/valgrind.txt

        ------------------------------------------------------------------------
        r225486 | lmadsen | 2009-10-22 16:57:50 -0500 (Thu, 22 Oct 2009) | 26 lines

        Merged revisions 225485 via svnmerge from
        https://origsvn.digium.com/svn/asterisk/trunk

        ................
        r225485 | lmadsen | 2009-10-22 16:52:30 -0500 (Thu, 22 Oct 2009) | 19 lines

        Merged revisions 225484 via svnmerge from
        https://origsvn.digium.com/svn/asterisk/branches/1.4

        ........
        r225484 | lmadsen | 2009-10-22 16:51:52 -0500 (Thu, 22 Oct 2009) | 11 lines

        Clean valgrind output by suppressing false errors.
        Update valgrind.txt documentation and add valgrind.supp file in order to
        allow those who are creating valgrind output to have less false errors in
        the logfile.

        (closes issue ASTERISK-14926)
        Reported by: atis
        Patches:
        valgrind.txt.diff uploaded by atis (license 242)
        asterisk2.supp uploaded by atis (license 242)
        Tested by: atis, amorsen
        ........
        ................

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

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

        Show
        Digium Subversion added a comment - Repository: asterisk Revision: 225486 A branches/1.6.0/contrib/valgrind.supp U branches/1.6.0/doc/valgrind.txt ------------------------------------------------------------------------ r225486 | lmadsen | 2009-10-22 16:57:50 -0500 (Thu, 22 Oct 2009) | 26 lines Merged revisions 225485 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ................ r225485 | lmadsen | 2009-10-22 16:52:30 -0500 (Thu, 22 Oct 2009) | 19 lines Merged revisions 225484 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r225484 | lmadsen | 2009-10-22 16:51:52 -0500 (Thu, 22 Oct 2009) | 11 lines Clean valgrind output by suppressing false errors. Update valgrind.txt documentation and add valgrind.supp file in order to allow those who are creating valgrind output to have less false errors in the logfile. (closes issue ASTERISK-14926 ) Reported by: atis Patches: valgrind.txt.diff uploaded by atis (license 242) asterisk2.supp uploaded by atis (license 242) Tested by: atis, amorsen ........ ................ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=225486
        Hide
        Digium Subversion added a comment -

        Repository: asterisk
        Revision: 225487

        A branches/1.6.1/contrib/valgrind.supp
        U branches/1.6.1/doc/valgrind.txt

        ------------------------------------------------------------------------
        r225487 | lmadsen | 2009-10-22 16:58:28 -0500 (Thu, 22 Oct 2009) | 26 lines

        Merged revisions 225485 via svnmerge from
        https://origsvn.digium.com/svn/asterisk/trunk

        ................
        r225485 | lmadsen | 2009-10-22 16:52:30 -0500 (Thu, 22 Oct 2009) | 19 lines

        Merged revisions 225484 via svnmerge from
        https://origsvn.digium.com/svn/asterisk/branches/1.4

        ........
        r225484 | lmadsen | 2009-10-22 16:51:52 -0500 (Thu, 22 Oct 2009) | 11 lines

        Clean valgrind output by suppressing false errors.
        Update valgrind.txt documentation and add valgrind.supp file in order to
        allow those who are creating valgrind output to have less false errors in
        the logfile.

        (closes issue ASTERISK-14926)
        Reported by: atis
        Patches:
        valgrind.txt.diff uploaded by atis (license 242)
        asterisk2.supp uploaded by atis (license 242)
        Tested by: atis, amorsen
        ........
        ................

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

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

        Show
        Digium Subversion added a comment - Repository: asterisk Revision: 225487 A branches/1.6.1/contrib/valgrind.supp U branches/1.6.1/doc/valgrind.txt ------------------------------------------------------------------------ r225487 | lmadsen | 2009-10-22 16:58:28 -0500 (Thu, 22 Oct 2009) | 26 lines Merged revisions 225485 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ................ r225485 | lmadsen | 2009-10-22 16:52:30 -0500 (Thu, 22 Oct 2009) | 19 lines Merged revisions 225484 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r225484 | lmadsen | 2009-10-22 16:51:52 -0500 (Thu, 22 Oct 2009) | 11 lines Clean valgrind output by suppressing false errors. Update valgrind.txt documentation and add valgrind.supp file in order to allow those who are creating valgrind output to have less false errors in the logfile. (closes issue ASTERISK-14926 ) Reported by: atis Patches: valgrind.txt.diff uploaded by atis (license 242) asterisk2.supp uploaded by atis (license 242) Tested by: atis, amorsen ........ ................ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=225487
        Hide
        Digium Subversion added a comment -

        Repository: asterisk
        Revision: 225488

        A branches/1.6.2/contrib/valgrind.supp
        U branches/1.6.2/doc/valgrind.txt

        ------------------------------------------------------------------------
        r225488 | lmadsen | 2009-10-22 16:58:50 -0500 (Thu, 22 Oct 2009) | 26 lines

        Merged revisions 225485 via svnmerge from
        https://origsvn.digium.com/svn/asterisk/trunk

        ................
        r225485 | lmadsen | 2009-10-22 16:52:30 -0500 (Thu, 22 Oct 2009) | 19 lines

        Merged revisions 225484 via svnmerge from
        https://origsvn.digium.com/svn/asterisk/branches/1.4

        ........
        r225484 | lmadsen | 2009-10-22 16:51:52 -0500 (Thu, 22 Oct 2009) | 11 lines

        Clean valgrind output by suppressing false errors.
        Update valgrind.txt documentation and add valgrind.supp file in order to
        allow those who are creating valgrind output to have less false errors in
        the logfile.

        (closes issue ASTERISK-14926)
        Reported by: atis
        Patches:
        valgrind.txt.diff uploaded by atis (license 242)
        asterisk2.supp uploaded by atis (license 242)
        Tested by: atis, amorsen
        ........
        ................

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

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

        Show
        Digium Subversion added a comment - Repository: asterisk Revision: 225488 A branches/1.6.2/contrib/valgrind.supp U branches/1.6.2/doc/valgrind.txt ------------------------------------------------------------------------ r225488 | lmadsen | 2009-10-22 16:58:50 -0500 (Thu, 22 Oct 2009) | 26 lines Merged revisions 225485 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ................ r225485 | lmadsen | 2009-10-22 16:52:30 -0500 (Thu, 22 Oct 2009) | 19 lines Merged revisions 225484 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r225484 | lmadsen | 2009-10-22 16:51:52 -0500 (Thu, 22 Oct 2009) | 11 lines Clean valgrind output by suppressing false errors. Update valgrind.txt documentation and add valgrind.supp file in order to allow those who are creating valgrind output to have less false errors in the logfile. (closes issue ASTERISK-14926 ) Reported by: atis Patches: valgrind.txt.diff uploaded by atis (license 242) asterisk2.supp uploaded by atis (license 242) Tested by: atis, amorsen ........ ................ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=225488

          People

          • Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development