OMEdit crashes on startup since rev.25700

[zbigniew.wolanski@…]

OMEdit crashes on startup in revision 25700 and all consecutive revs.
Last tested revision with defect: 25747.
Last working revision: 25670.
Tested on 3 PCs: two Windows 7, one Windows 8.1
Crash reports was sent from each incident.

[Adeel Asghar]

We haven't got any crash report. It seems like crash reports are broken as well :). I will check it.
Can you post the following files?
%TEMP%/omeditcommunication.log
%TEMP%/omeditcommands.mos
%TEMP%/openmodelica.stacktrace.OMEdit

[Zbigniew.Wolanski@…]

These attachments are from one of the Windows 7 PCs exhibiting the crash

[Adrian Pop]

I looked a bit into this.
We do: omdevPath = Util.makeValueOrDefault(System.readEnv,"OMDEV","");
and if OMDEV is not present I guess we get a crash. And yes, via cmd.exe

> cd \Path\to\OMEdit.exe
> set OMDEV=
> .\OMEdit

will reproduce the issue.
We have

extern const char* System_readEnv(const char *envname)
{
  char *envvalue = getenv(envname);
  if (envvalue == NULL) MMC_THROW();
  return strcpy(ModelicaAllocateString(strlen(envvalue)),envvalue);
}

which means that if the env variable does not exist we do a throw and die because we don't do
MMC_TRY_TOP_INTERNAL for omc_Main_setWindowsPaths.

Adeel, can you wrap the call to omc_Main_setWindowsPaths within a try catch block?
Then try to see if it works by un-setting OMDEV variable.

[Adeel Asghar]

Fixed in r25813.

[Dietmar Winkler]

It doesn't make sense to keep closed ticket in the "Future" milestone that were simply forgotten to assign to the correct milestone in the past.

[Martin Sjölund]

Removing the pre1.9.4 milestone in favor of 1.9.4.