You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
According to my printf debugging, nMarker_Emissivity is 2 and Wall_Emissivity is {"left", "right"} (matching the cfg file), but CRadP1Solver::BC_Marshak calls GetWall_Emissivity with the marker string "upper".
Looks like BC_Marshak is used for all boundary types of the radiation solver, maybe we need to return 0 emissivity for markers that are not found in the list.
Describe the bug
The serial regression test
p1rad
, which runs SU2_CFD on TestCases/radiation/p1model/configp1.cfg, has a heap-buffer-overflow inCConfig::GetWall_Emissivity
according to Clang's address sanitizer.According to my printf debugging,
nMarker_Emissivity
is2
andWall_Emissivity
is{"left", "right"}
(matching the cfg file), butCRadP1Solver::BC_Marshak
callsGetWall_Emissivity
with the marker string"upper"
.Bug report checklist
Desktop (please complete the following information):
Ubuntu 22.04 with Clang++ 16.0.6, MPI disabled, SU2 at current
develop
branch, commit f9f7a01.The text was updated successfully, but these errors were encountered: