Opened 9 years ago

Closed 9 years ago

Last modified 8 years ago

#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

I just downloaded the nightly 1.9.3 VM Linux image, and when I try it there it does run correctly.

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

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 [...]
Last edited 9 years ago by sjoelund.se (previous) (diff)

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

Note: See TracTickets for help on using tickets.