make sure virial array is initialized in the constructor to avoid uninitialized access

Author: akohlmey

src/bond.cpp

@@ -36,12 +36,13 @@ int Bond::instance_total = 0;
    a particular bond style can override this
 ------------------------------------------------------------------------- */
 
-Bond::Bond(LAMMPS *_lmp) : Pointers(_lmp)
+Bond::Bond(LAMMPS *_lmp) :
+    Pointers(_lmp), virial{0.0, 0.0, 0.0, 0.0, 0.0, 0.0}, eatom(nullptr), vatom(nullptr),
+    setflag(nullptr), svector(nullptr)
 {
   instance_me = instance_total++;
 
   energy = 0.0;
-  virial[0] = virial[1] = virial[2] = virial[3] = virial[4] = virial[5] = 0.0;
   writedata = 1;
   reinitflag = 1;
 
@@ -53,12 +54,8 @@ Bond::Bond(LAMMPS *_lmp) : Pointers(_lmp)
   partial_flag = 0;
 
   single_extra = 0;
-  svector = nullptr;
 
   maxeatom = maxvatom = 0;
-  eatom = nullptr;
-  vatom = nullptr;
-  setflag = nullptr;
 
   execution_space = Host;
   datamask_read = ALL_MASK;

src/pair.cpp

@@ -53,14 +53,15 @@ int Pair::instance_total = 0;
 /* ---------------------------------------------------------------------- */
 
 Pair::Pair(LAMMPS *lmp) :
-    Pointers(lmp), eatom(nullptr), vatom(nullptr), cvatom(nullptr), cutsq(nullptr),
-    setflag(nullptr), cutghost(nullptr), rtable(nullptr), drtable(nullptr), ftable(nullptr),
-    dftable(nullptr), ctable(nullptr), dctable(nullptr), etable(nullptr), detable(nullptr),
-    ptable(nullptr), dptable(nullptr), vtable(nullptr), dvtable(nullptr), rdisptable(nullptr),
-    drdisptable(nullptr), fdisptable(nullptr), dfdisptable(nullptr), edisptable(nullptr),
-    dedisptable(nullptr), pvector(nullptr), svector(nullptr), list(nullptr), listhalf(nullptr),
-    listfull(nullptr), list_tally_compute(nullptr), elements(nullptr), elem1param(nullptr),
-    elem2param(nullptr), elem3param(nullptr), map(nullptr)
+    Pointers(lmp), virial{0.0, 0.0, 0.0, 0.0, 0.0, 0.0}, eatom(nullptr), vatom(nullptr),
+    cvatom(nullptr), cutsq(nullptr), setflag(nullptr), cutghost(nullptr), rtable(nullptr),
+    drtable(nullptr), ftable(nullptr), dftable(nullptr), ctable(nullptr), dctable(nullptr),
+    etable(nullptr), detable(nullptr), ptable(nullptr), dptable(nullptr), vtable(nullptr),
+    dvtable(nullptr), rdisptable(nullptr), drdisptable(nullptr), fdisptable(nullptr),
+    dfdisptable(nullptr), edisptable(nullptr), dedisptable(nullptr), pvector(nullptr),
+    svector(nullptr), list(nullptr), listhalf(nullptr), listfull(nullptr),
+    list_tally_compute(nullptr), elements(nullptr), elem1param(nullptr), elem2param(nullptr),
+    elem3param(nullptr), map(nullptr)
 {
   instance_me = instance_total++;