Opened 10 years ago
Closed 9 years ago
#3249 closed defect (fixed)
Simulation crash
| Reported by: | Adrian Pop | Owned by: | Martin Sjölund |
|---|---|---|---|
| Priority: | high | Milestone: | 1.9.3 |
| Component: | Run-time | Version: | trunk |
| Keywords: | Cc: |
Description
The test: testsuite/simulation/modelica/others/Random2.mos
fails on other gccs than 4.4, for example:
https://test.openmodelica.org/hudson/job/OpenModelica_TEST_GCC_4.9/lastCompletedBuild/testReport/(root)/simulation_modelica_others/Random2_mos/
base_array.c: array dimensions sizes are NULL! Random.TestRandom: util/real_array.c:98: copy_real_array_data: Assertion `base_array_ok(dest)' failed. Aborted (core dumped)
Attachments (1)
Change History (7)
by , 10 years ago
| Attachment: | valgrind-trace.txt added |
|---|
comment:1 by , 10 years ago
comment:2 by , 10 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → assigned |
comment:3 by , 10 years ago
It seems we pass tmp1 without allocating anything in it. When passing a reference to an array, we should probably allocate it first (unless it is a : dimension). Or do a copy_alloc always (which is probably slower). Or create the array pointing to the memory of $PrandomSeed and pass that.
comment:4 by , 10 years ago
| Status: | assigned → accepted |
|---|
comment:5 by , 9 years ago
| Owner: | changed from to |
|---|
comment:6 by , 9 years ago
| Milestone: | Future → 1.9.3 |
|---|---|
| Resolution: | → fixed |
| Status: | accepted → closed |
Fixed in 3c606c7/OMCompiler
Note:
See TracTickets
for help on using tickets.
Added the valgrind-trace.txt so that we have some idea where to look.