ICODE-ADC/lammps
4
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

set lmp pointer to NULL after delete to avoid using it or deleting it a second time

Browse Source
This commit is contained in:
Axel Kohlmeyer
2025-01-29 23:32:52 -05:00
parent 06bdb7b637
commit da1bbec0be
6 changed files with 12 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
unittest/force-styles/test_angle_style.cpp
View File

@ -53,12 +53,13 @@ using ::testing::StartsWith;
using namespace LAMMPS_NS; using namespace LAMMPS_NS;
void cleanup_lammps(LAMMPS *lmp, const TestConfig &cfg) void cleanup_lammps(LAMMPS *&lmp, const TestConfig &cfg)
{ {
platform::unlink(cfg.basename + ".restart"); platform::unlink(cfg.basename + ".restart");
platform::unlink(cfg.basename + ".data"); platform::unlink(cfg.basename + ".data");
platform::unlink(cfg.basename + "-coeffs.in"); platform::unlink(cfg.basename + "-coeffs.in");
delete lmp; delete lmp;
lmp = nullptr;
} }
LAMMPS *init_lammps(LAMMPS::argv &args, const TestConfig &cfg, const bool newton) LAMMPS *init_lammps(LAMMPS::argv &args, const TestConfig &cfg, const bool newton)

3
unittest/force-styles/test_bond_style.cpp
View File

@ -53,12 +53,13 @@ using ::testing::StartsWith;
using namespace LAMMPS_NS; using namespace LAMMPS_NS;
void cleanup_lammps(LAMMPS *lmp, const TestConfig &cfg) void cleanup_lammps(LAMMPS *&lmp, const TestConfig &cfg)
{ {
platform::unlink(cfg.basename + ".restart"); platform::unlink(cfg.basename + ".restart");
platform::unlink(cfg.basename + ".data"); platform::unlink(cfg.basename + ".data");
platform::unlink(cfg.basename + "-coeffs.in"); platform::unlink(cfg.basename + "-coeffs.in");
delete lmp; delete lmp;
lmp = nullptr;
} }
LAMMPS *init_lammps(LAMMPS::argv &args, const TestConfig &cfg, const bool newton) LAMMPS *init_lammps(LAMMPS::argv &args, const TestConfig &cfg, const bool newton)

3
unittest/force-styles/test_dihedral_style.cpp
View File

@ -53,12 +53,13 @@ using ::testing::StartsWith;
using namespace LAMMPS_NS; using namespace LAMMPS_NS;
void cleanup_lammps(LAMMPS *lmp, const TestConfig &cfg) void cleanup_lammps(LAMMPS *&lmp, const TestConfig &cfg)
{ {
platform::unlink(cfg.basename + ".restart"); platform::unlink(cfg.basename + ".restart");
platform::unlink(cfg.basename + ".data"); platform::unlink(cfg.basename + ".data");
platform::unlink(cfg.basename + "-coeffs.in"); platform::unlink(cfg.basename + "-coeffs.in");
delete lmp; delete lmp;
lmp = nullptr;
} }
LAMMPS *init_lammps(LAMMPS::argv &args, const TestConfig &cfg, const bool newton) LAMMPS *init_lammps(LAMMPS::argv &args, const TestConfig &cfg, const bool newton)

3
unittest/force-styles/test_fix_timestep.cpp
View File

@ -56,10 +56,11 @@ using ::testing::StartsWith;
using namespace LAMMPS_NS; using namespace LAMMPS_NS;
void cleanup_lammps(LAMMPS *lmp, const TestConfig &cfg) void cleanup_lammps(LAMMPS *&lmp, const TestConfig &cfg)
{ {
platform::unlink(cfg.basename + ".restart"); platform::unlink(cfg.basename + ".restart");
delete lmp; delete lmp;
lmp = nullptr;
} }
LAMMPS *init_lammps(LAMMPS::argv &args, const TestConfig &cfg, const bool use_respa) LAMMPS *init_lammps(LAMMPS::argv &args, const TestConfig &cfg, const bool use_respa)

3
unittest/force-styles/test_improper_style.cpp
View File

@ -53,12 +53,13 @@ using ::testing::StartsWith;
using namespace LAMMPS_NS; using namespace LAMMPS_NS;
void cleanup_lammps(LAMMPS *lmp, const TestConfig &cfg) void cleanup_lammps(LAMMPS *&lmp, const TestConfig &cfg)
{ {
platform::unlink(cfg.basename + ".restart"); platform::unlink(cfg.basename + ".restart");
platform::unlink(cfg.basename + ".data"); platform::unlink(cfg.basename + ".data");
platform::unlink(cfg.basename + "-coeffs.in"); platform::unlink(cfg.basename + "-coeffs.in");
delete lmp; delete lmp;
lmp = nullptr;
} }
LAMMPS *init_lammps(LAMMPS::argv &args, const TestConfig &cfg, const bool newton) LAMMPS *init_lammps(LAMMPS::argv &args, const TestConfig &cfg, const bool newton)

3
unittest/force-styles/test_pair_style.cpp
View File

@ -54,12 +54,13 @@ using ::testing::StartsWith;
using namespace LAMMPS_NS; using namespace LAMMPS_NS;
void cleanup_lammps(LAMMPS *lmp, const TestConfig &cfg) void cleanup_lammps(LAMMPS *&lmp, const TestConfig &cfg)
{ {
platform::unlink(cfg.basename + ".restart"); platform::unlink(cfg.basename + ".restart");
platform::unlink(cfg.basename + ".data"); platform::unlink(cfg.basename + ".data");
platform::unlink(cfg.basename + "-coeffs.in"); platform::unlink(cfg.basename + "-coeffs.in");
delete lmp; delete lmp;
lmp = nullptr;
} }
LAMMPS *init_lammps(LAMMPS::argv &args, const TestConfig &cfg, const bool newton) LAMMPS *init_lammps(LAMMPS::argv &args, const TestConfig &cfg, const bool newton)