[DynInst_API:] [dyninst/dyninst] d6ab42: AddressTranslateSysV::adjustForAddrSpaceWrap: Fix ...


Date: Tue, 09 May 2023 10:30:57 -0700
From: Tim Haines <noreply@xxxxxxxxxx>
Subject: [DynInst_API:] [dyninst/dyninst] d6ab42: AddressTranslateSysV::adjustForAddrSpaceWrap: Fix ...
  Branch: refs/heads/master
  Home:   https://github.com/dyninst/dyninst
  Commit: d6ab42a4da20f7d9a5a58c00a4835d337a95fef5
      https://github.com/dyninst/dyninst/commit/d6ab42a4da20f7d9a5a58c00a4835d337a95fef5
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2023-05-09 (Tue, 09 May 2023)

  Changed paths:
    M common/src/addrtranslate-sysv.C

  Log Message:
  -----------
  AddressTranslateSysV::adjustForAddrSpaceWrap: Fix C file descriptor leak (#1427)

* AddressTranslateSysV::adjustForAddrSpaceWrap: Fix C file descriptor leak

Found using cppcheck:

common/src/addrtranslate-sysv.C:1046:58: error: Resource leak: fd [resourceLeak]
   if (read(fd, &e_hdr, sizeof(e_hdr)) != sizeof(e_hdr)) return base;
                                                         ^
common/src/addrtranslate-sysv.C:1048:28: error: Resource leak: fd [resourceLeak]
   if (e_hdr.e_phoff == 0) return base;


[← Prev in Thread] Current Thread [Next in Thread→]
  • [DynInst_API:] [dyninst/dyninst] d6ab42: AddressTranslateSysV::adjustForAddrSpaceWrap: Fix ..., Tim Haines <=