[DynInst_API:] [dyninst/dyninst] faf8a7: Local var leak in Symtab::addSymbol


Date: Thu, 11 May 2023 07:37:28 -0700
From: Tim Haines <noreply@xxxxxxxxxx>
Subject: [DynInst_API:] [dyninst/dyninst] faf8a7: Local var leak in Symtab::addSymbol
  Branch: refs/heads/fix_resource_leaks
  Home:   https://github.com/dyninst/dyninst
  Commit: faf8a70bc630c44dc56ba41bf1a23ce8a5f8f8fa
      https://github.com/dyninst/dyninst/commit/faf8a70bc630c44dc56ba41bf1a23ce8a5f8f8fa
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2023-05-11 (Thu, 11 May 2023)

  Changed paths:
    M symtabAPI/src/Symtab-edit.C

  Log Message:
  -----------
  Local var leak in Symtab::addSymbol

This was found using cppcheck's memleak.


  Commit: 5bcc1517456fcad57443c4619e261f2dc6809f57
      https://github.com/dyninst/dyninst/commit/5bcc1517456fcad57443c4619e261f2dc6809f57
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2023-05-11 (Thu, 11 May 2023)

  Changed paths:
    M proccontrol/src/memcache.C

  Log Message:
  -----------
  memCache::doOperation

Not technically a leak, but cppcheck can't see through the 'push_back'.


  Commit: 9943de2f169068d424a6101a19fef4de0bc922bb
      https://github.com/dyninst/dyninst/commit/9943de2f169068d424a6101a19fef4de0bc922bb
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2023-05-11 (Thu, 11 May 2023)

  Changed paths:
    M dyninstAPI/src/unix.C

  Log Message:
  -----------
  PCProcess::hasPassedMain

Found using cppcheck's danglingTemporaryLifetime.


  Commit: de254c5b903a3b5f1ff94a3a470502dc121c89c4
      https://github.com/dyninst/dyninst/commit/de254c5b903a3b5f1ff94a3a470502dc121c89c4
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2023-05-11 (Thu, 11 May 2023)

  Changed paths:
    M dyninstAPI/src/parse-power.C

  Log Message:
  -----------
  parse_func::calcParentFunc

Found using cppcheck's danglingTemporaryLifetime.


  Commit: 6ba0f27da1448ab5292bf3180e835456cc99f9d5
      https://github.com/dyninst/dyninst/commit/6ba0f27da1448ab5292bf3180e835456cc99f9d5
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2023-05-11 (Thu, 11 May 2023)

  Changed paths:
    M proccontrol/src/irpc.C

  Log Message:
  -----------
  int_iRPC::setBinarySize

Found using cppcheck's publicAllocationError.


Compare: https://github.com/dyninst/dyninst/compare/faf8a70bc630%5E...6ba0f27da144
[← Prev in Thread] Current Thread [Next in Thread→]
  • [DynInst_API:] [dyninst/dyninst] faf8a7: Local var leak in Symtab::addSymbol, Tim Haines <=