Mutex initialization inside a C# method always returns positive createdNew

Question

I have created simple MutexManager:

public static class MutexManager
{
    private static string mutexName
    {
        get
        {
            return "MyAppName" + System.Security.Principal.WindowsIdentity.GetCurrent().User.AccountDomainSid;
        }
    }

    public static bool CreateApplicationMutex()
    {

        bool createdNew;
        var mutex = new Mutex(false, mutexName, out createdNew);                       

        return createdNew;
    }
}

The problem is that CreateApplicationMutex always returns true on new application instance startup. As long as I had exactly same code in app.cs everything was correct, but after I moved it to MutexManager createdNew is always true. What am I doing wrong?

Should work provided that mutexName returns the same value in both instances. Make sure that you are starting the application as the same user. — mm8, Oct 22 '18 at 09:32

score 1 · Answer 1 · edited Jun 20 '20 at 09:12

1

The following works as expected for me, and returns false on second instance

public static class MutexManager
{
   private static string mutexName => "MyAppName" + System.Security.Principal.WindowsIdentity.GetCurrent()
                                                            .User?.AccountDomainSid;
   public static bool CreateApplicationMutex()
   {
      new Mutex(false, mutexName, out var createdNew);

      return createdNew;
   }
}

private static void Main(string[] args)
{
   Console.WriteLine(MutexManager.CreateApplicationMutex());
   Console.ReadKey();
}

Output

true
false

Make sure you debug your app, and check the mutex name

Update

Winforms

MessageBox.Show(
   MutexManager.CreateApplicationMutex()
               .ToString());
Application.EnableVisualStyles();
Application.SetCompatibleTextRenderingDefault(false);
Application.Run(new Form1());

WPF

public partial class App : Application
{
   protected override void OnStartup(StartupEventArgs e)
   {
      MessageBox.Show(
         MutexManager.CreateApplicationMutex()
                     .ToString());
      base.OnStartup(e);
   }
}

Once again it works as expected, and cant be reproduced

edited Jun 20 '20 at 09:12

Community

1
1

answered Oct 22 '18 at 09:01

TheGeneral

79,002
9
103
141

It looks to work fine in console app but same code doesnt work as expected when I am invoking this method in ApplicationStart (my WPF app startpoint). It work if I move this code from MutexManager to ApplicationStart but I would like to keep it there. – Bartek Chyży Oct 22 '18 at 09:20
@BartekChyży tested in a winforms and wpf, and it works between all 3 in any combination, and cant reproduce your results – TheGeneral Oct 22 '18 at 09:28
Thank for your time. I will let you know if I found the solution. – Bartek Chyży Oct 22 '18 at 09:32
I had to wrap it in dispatcher invocation but I don't fully understand why. – Bartek Chyży Oct 22 '18 at 09:47

score 1 · Answer 2 · answered Aug 01 '20 at 23:18

I had a similar problem, the reason was life time of mutex variable. Following code works fine by me in debug version, but always returns true for created_new in release version:

using System;

static class Program
{
    [STAThread]
    static void Main(string[] args)
    {
        string mutex_name = "MyApplicationName_SingleInstanceMutex";
        bool created_new = false;
        System.Threading.Mutex mutex = new System.Threading.Mutex(false, mutex_name, out created_new);
        if (!created_new)
        {
            System.Windows.Forms.MessageBox.Show("Application is already started!");
            return;
        }
        
        /* Create & run mainForm here */
    }
}

Making mutex static solves the problem:

using System;

static class Program
{
    static System.Threading.Mutex mutex = null;

    [STAThread]
    static void Main(string[] args)
    {
        string mutex_name = "MyApplicationName_SingleInstanceMutex";
        bool created_new = false;
        mutex = new System.Threading.Mutex(false, mutex_name, out created_new);
        if (!created_new)
        {
            System.Windows.Forms.MessageBox.Show("Application is already started!");
            return;
        }
        
        /* Create & run mainForm here */
    }
}

Mutex initialization inside a C# method always returns positive createdNew

2 Answers2

Update