[DynInst_API:] [dyninst/dyninst] 68fc3d: Protect against self-assignment in copy constructo...


Date: Tue, 09 May 2023 22:14:54 -0700
From: Tim Haines <noreply@xxxxxxxxxx>
Subject: [DynInst_API:] [dyninst/dyninst] 68fc3d: Protect against self-assignment in copy constructo...
  Branch: refs/heads/master
  Home:   https://github.com/dyninst/dyninst
  Commit: 68fc3d8a5f1527dcaa66c9455369c62e2b4fa54c
      https://github.com/dyninst/dyninst/commit/68fc3d8a5f1527dcaa66c9455369c62e2b4fa54c
  Author: Tim Haines <thaines.astro@xxxxxxxxx>
  Date:   2023-05-10 (Wed, 10 May 2023)

  Changed paths:
    M common/src/Edge.C
    M common/src/Node.C

  Log Message:
  -----------
  Protect against self-assignment in copy constructors (#1430)

* Protect against self-assignment in copy constructors

These were found by cppcheck:

common/src/Node.C:198:29: warning: 'operator=' should check for assignment to self to avoid problems with dynamic memory. [operatorEqToSelf]
NodeIterator &NodeIterator::operator=(const NodeIterator &rhs) {
                            ^
common/src/Edge.C:108:29: warning: 'operator=' should check for assignment to self to avoid problems with dynamic memory. [operatorEqToSelf]
EdgeIterator &EdgeIterator::operator=(const EdgeIterator &rhs) {


[← Prev in Thread] Current Thread [Next in Thread→]
  • [DynInst_API:] [dyninst/dyninst] 68fc3d: Protect against self-assignment in copy constructo..., Tim Haines <=