startnine/start9

Pressing Start on the Taskbar causes a Crash

dominichayesferen opened this issue · 15 comments

dominichayesferen commented

Version: Just pulled it off of this Repo

Pretty much self-explanatory, clicked Start on New Taskbar = Crash

reflectronic commented

oh 🅱️rap

reflectronic commented

Hmmm works for me. I'll implement some sort of a crash logging soon:tm: so we can look into exactly what is causing this

dominichayesferen commented

UPDATE 2: It does it on Normal Windows 7 Boot:
InstalledTransientModules[0].HandleCommand(command);
............................................ ^ System.ArgumentOutOfRangeException

dominichayesferen commented

"Index was out of range. Must be non-negative and less than the size of collection."
File: IModule.cs

reflectronic commented

Hmmm... do you have both of the modules installed in %appdata%\Start9\Modules?

dominichayesferen commented

Oh... that might be the issue. 🤦‍♂️

dominichayesferen commented

Erm, is that Start9/Modules/*, or module files in the Modules folder?

dominichayesferen commented

...with them as Modules/TestFixedModule and Modules/TestTransientModule, it still throws the exception

reflectronic commented

image
So you have these files?

dominichayesferen commented

TestFixed: Yes
TestTransient: No

reflectronic commented

Ahh that's the problem. The start menu is TestTransient

dominichayesferen commented

Can I have a copy of TestTransient over Discord DMs? It isn't in DLL and PDB file form here

dominichayesferen commented

Fixed over TeamViewer (so others would have to ask Reflectron about how he fixed my one)...

reflectronic commented

Alright, we looked into the issue and it seems like it's a simple fix:

Firstly, feren did not have the TestTransientModule [this is the Start Menu]. Thus the InstalledTransientModules collection was empty and ArgumentOutOfRangeException was raised.

This fixed that issue - however, another issue cropped up. Code Access Security (CAS) policy does not trust assemblies downloaded from the internet, and when they are loaded NotSupportedException is thrown. To remedy this, an app.exe.config file must be included with the following contents:

<configuration>  
   <runtime>  
      <loadFromRemoteSources enabled="true"/>  
   </runtime>  
</configuration>

This allows assemblies from remote sources (downloaded from the net, etc.) to be loaded. I will update the build settings for the Start9 project to include an app.exe.config enabling loadFromRemoteSources in a future commit.