fix multiple instances sometimes being able to be opened

Author: tomysshadow

FlashpointSecurePlayer/FlashpointSecurePlayer.cs

@@ -20,21 +20,22 @@ public partial class FlashpointSecurePlayer : Form {
         private const string APPLICATION_MUTEX_NAME = "Flashpoint Secure Player";
         private const string FLASHPOINT_LAUNCHER_PARENT_PROCESS_EXE_FILE_NAME = "cmd.exe";
         private const string FLASHPOINT_LAUNCHER_PROCESS_NAME = "flashpoint";
+        private Mutex ApplicationMutex = null;
         private static SemaphoreSlim ModificationsSemaphoreSlim = new SemaphoreSlim(1, 1);
         private readonly RunAsAdministrator RunAsAdministrator;
         private readonly ModeTemplates ModeTemplates;
         private readonly EnvironmentVariables EnvironmentVariables;
         private readonly DownloadsBefore DownloadsBefore;
         private readonly RegistryBackups RegistryBackup;
         private readonly SingleInstance SingleInstance;
-        string ModificationsName = ACTIVE_EXE_CONFIGURATION_NAME;
-        bool RunAsAdministratorModification = false;
-        List<string> DownloadsBeforeModificationNames = null;
-        bool ActiveX = false;
-        string Server = null;
-        string Software = null;
-        ProcessStartInfo SoftwareProcessStartInfo = null;
-        public delegate void ErrorDelegate(string text);
+        private string ModificationsName = ACTIVE_EXE_CONFIGURATION_NAME;
+        private bool RunAsAdministratorModification = false;
+        private List<string> DownloadsBeforeModificationNames = null;
+        private bool ActiveX = false;
+        private string Server = null;
+        private string Software = null;
+        private ProcessStartInfo SoftwareProcessStartInfo = null;
+        private delegate void ErrorDelegate(string text);
 
         public FlashpointSecurePlayer() {
             InitializeComponent();
@@ -515,7 +516,7 @@ private async void FlashpointSecurePlayer_Load(object sender, EventArgs e) {
             // default to false in case of error
             bool createdNew = false;
             // signals the Mutex if it has not been
-            Mutex flashpointSecurePlayerMutex = new Mutex(true, APPLICATION_MUTEX_NAME, out createdNew);
+            ApplicationMutex = new Mutex(true, APPLICATION_MUTEX_NAME, out createdNew);
 
             if (!createdNew) {
                 // multiple instances open, blow up immediately

FlashpointSecurePlayer/Properties/AssemblyInfo.cs

@@ -33,7 +33,7 @@
 // You can specify all the values or you can default the Build and Revision Numbers
 // by using the '*' as shown below:
 // [assembly: AssemblyVersion("1.0.*")]
-[assembly: AssemblyVersion("1.0.5.0")]
-[assembly: AssemblyFileVersion("1.0.5.0")]
+[assembly: AssemblyVersion("1.0.6.0")]
+[assembly: AssemblyFileVersion("1.0.6.0")]
 [assembly: NeutralResourcesLanguage("en")]
 

FlashpointSecurePlayer/README.md

@@ -1,4 +1,4 @@
-# Flashpoint Secure Player 1.0.5
+# Flashpoint Secure Player 1.0.6
 This player attempts to solve common compatibility or portability issues posed by browser plugins on Windows for the purpose of playback in BlueMaxima's Flashpoint.
 
 It is compatible with Windows 7, Windows 8, Windows 8.1 and Windows 10, and requires .NET Framework 4.5. If you are on Windows 8.1 or Windows 10, or if you are on Windows 7/8 and have updates enabled, you already have .NET Framework 4.5. Otherwise, you may [download .NET Framework 4.5.](http://www.microsoft.com/en-us/download/details.aspx?id=30653)

FlashpointSecurePlayer/Server.cs

@@ -15,7 +15,7 @@
 namespace FlashpointSecurePlayer {
     public partial class Server : Form {
         private CustomSecurityManager CustomSecurityManager;
-        public Uri WebBrowserURL = null;
+        private Uri WebBrowserURL = null;
 
         public Server(Uri WebBrowserURL) {
             InitializeComponent();