vecgeom: Do not read destructed memory

[smuzaffar]

Apply changes from https://gitlab.cern.ch/VecGeom/VecGeom/-/merge_requests/1262

FYI @civanch
We do get these warnings for GCC 14 builds https://cmssdt.cern.ch/SDT/cgi-bin/buildlogs/el8_amd64_gcc14/CMSSW_15_1_X_2025-02-24-1900/BigProducts/Simulation

  vecgeom/v1.2.10-73ad1151de5c4a9ee0003bb695ac82c0/include/VecGeom/base/VariableSizeObj.h:182:32: warning: 'MEM[(struct VariableData_t &)_6 + 4].fSelfAlloc' may be used uninitialized [-Wmaybe-uninitialized]
   182 |     if (obj->GetVariableData().fSelfAlloc) delete[](char *) obj;
      |                                ^
In function 'ReleaseInstance',
    inlined from '__dt_base ' at vecgeom/v1.2.10-73ad1151de5c4a9ee0003bb695ac82c0/include/VecGeom/volumes/TessellatedStruct.h:229:43,
    inlined from '__dt_base ' at vecgeom/v1.2.10-73ad1151de5c4a9ee0003bb695ac82c0/include/VecGeom/volumes/UnplacedTessellated.h:38:7,
    inlined from '__dt_base ' at geant4/11.2.2-cb96bd8c2ff4d6e5ba67de349ddad4cb/geant4.11.2.2/source/geometry/management/include/G4UAdapter.hh:264:1,
    inlined from '__dt_base ' at geant4/11.2.2-cb96bd8c2ff4d6e5ba67de349ddad4cb/geant4.11.2.2/source/geometry/solids/specific/src/G4UTessellatedSolid.cc:76:1:
  vecgeom/v1.2.10-73ad1151de5c4a9ee0003bb695ac82c0/include/VecGeom/base/VariableSizeObj.h:182:32: warning: 'MEM[(struct VariableData_t &)_57 + 4].fSelfAlloc' may be used uninitialized [-Wmaybe-uninitialized]
   182 |     if (obj->GetVariableData().fSelfAlloc) delete[](char *) obj;

[cmsbuild]

A new Pull Request was created by @smuzaffar for branch IB/CMSSW_15_1_X/master.

@cmsbuild, @iarspider, @smuzaffar can you please review it and eventually sign? Thanks.
@antoniovilela, @mandrenguyen, @rappoccio, @sextonkennedy you are the release manager for this.
cms-bot commands are listed here

[cmsbuild]

cms-bot internal usage

[smuzaffar]

please test

[cmsbuild]

Pull request #9719 was updated.

[smuzaffar]

please test

[civanch]

@smuzaffar , this fix needs to be integrated. If I understand correctly it is essential for phase2.

[cmsbuild]

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-4cac46/44636/summary.html
COMMIT: cb58ba9
CMSSW: CMSSW_15_1_X_2025-02-25-1100/el8_amd64_gcc12
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmsdist/9719/44636/install.sh to create a dev area with all the needed externals and cmssw changes.

The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:

• @cms-sw Update tag for DQM-Integration to V00-09-00 #9718

You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-4cac46/44636/git-recent-commits.json
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-4cac46/44636/git-merge-result

Comparison Summary

Summary:

• You potentially added 18 lines to the logs
• Reco comparison results: 4 differences found in the comparisons
• DQMHistoTests: Total files compared: 49
• DQMHistoTests: Total histograms compared: 3920300
• DQMHistoTests: Total failures: 21
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 3920259
• DQMHistoTests: Total skipped: 20
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.0 KiB( 48 files compared)
• Checked 214 log files, 184 edm output root files, 49 DQM output files
• TriggerResults: no differences found

[smuzaffar]

+externals

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next IB/CMSSW_15_1_X/master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @rappoccio, @sextonkennedy, @antoniovilela, @mandrenguyen (and backports should be raised in the release meeting by the corresponding L2)