#3448 closed defect (fixed)
Stack Overflow Detected in OMEdit
Reported by: | dersh | Owned by: | sjoelund.se |
---|---|---|---|
Priority: | high | Milestone: | 1.9.4 |
Component: | Frontend | Version: | trunk |
Keywords: | Cc: |
Description
If I try to run Modelica.Fluid.Examples.BranchingDynamicPipes in OMEdit I get the following:
[1] 11:20:08 Scripting Notification Stack overflow detected and was not caught. Send us a bug report at https://trac.openmodelica.org/OpenModelica/newticket Include the following trace: [bt] Execution path:
and nothing else happens (progress window doesn't open etc). I am running 1.9.4~dev that I just built, on a Mac, and a number of other models all seem to build and run correctly.
According to the recursive BuildModel Test page, this is a model that should be working currently. I see that there were a few recent changesets related to catching stack overflows. So, that probably relates.
Change History (8)
comment:1 Changed 9 years ago by dersh
comment:2 Changed 9 years ago by sjoelund.se
Seems to be a stack overflow in: omc_NFSCodeDependency_collectUsedElements2.
I will improve the stack overflow detection for OSX (sadly, it will be missing in 1.9.3).
comment:3 Changed 9 years ago by sjoelund.se
[code]Stack overflow detected and was not caught.
Send us a bug report at https://trac.openmodelica.org/OpenModelica/newticket
Include the following trace:
[bt] #1 libOpenModelicaRuntimeC.dylib(mmc_setStacktraceMessages_threadData)
[bt] #2 libOpenModelicaCompiler.dylib(NFSCodeDependency.collectUsedElement)
[bt] #3...1024 libOpenModelicaCompiler.dylib(NFSCodeDependency.collectUsedElements2)
[bt] #1025 [...]code
comment:4 Changed 9 years ago by sjoelund.se
- Component changed from Unknown to Frontend
- Milestone changed from Future to 1.9.4
- Owner changed from somebody to sjoelund.se
- Status changed from new to accepted
comment:5 Changed 9 years ago by sjoelund.se
- Resolution set to fixed
- Status changed from accepted to closed
I now fixed the stack overflow. It didn't crash OMEdit before, right? Just printed that message? Otherwise, there are more changes necessary.
comment:6 Changed 9 years ago by dersh
Yes, it didn't crash before, and yes it now seems to be fixed.
Thank you.
comment:7 Changed 8 years ago by sjoelund.se
- Milestone changed from 1.9.4 to 1.9.4-1.9.x
Milestone renamed
comment:8 Changed 8 years ago by sjoelund.se
- Milestone changed from 1.9.4-1.9.x to 1.9.4
Milestone renamed
I just downloaded the nightly 1.9.3 VM Linux image, and when I try it there it does run correctly.