Changeset 788 for branches/eraser6/Eraser/Program.cs

Timestamp:

12/11/2008 12:15:36 PM (4 years ago)

Author:

lowjoel

Message:

Show a helpful message whenever Eraser tries to communicate with another instance of higher privileges (e.g., when a non-elevated process is communicating with an elevated process)

File:

• branches/eraser6/Eraser/Program.cs (modified) (2 diffs)

branches/eraser6/Eraser/Program.cs

@@ -319 +319 @@
 
                     byte[] buffer = new byte[commandLineStr.Length];
-                    int count = Encoding.UTF8.GetBytes(commandLineStr.ToString(),0,
+                    int count = Encoding.UTF8.GetBytes(commandLineStr.ToString(), 0,
                         commandLineStr.Length, buffer, 0);
                     client.Write(buffer, 0, buffer.Length);
+                }
+                catch (UnauthorizedAccessException)
+                {
+                    //We can't connect to the pipe because the other instance of Eraser
+                    //is running with higher privileges than this instance. Tell the
+                    //user this is the case and show him how to resolve the issue.
+                    MessageBox.Show(S._("Another instance of Eraser is already running but it is " +
+                        "running with higher privileges than this instance of Eraser.\n\n" +
+                        "Eraser will now exit."), S._("Eraser"), MessageBoxButtons.OK,
+                        MessageBoxIcon.Information);
                 }
                 catch (TimeoutException)
@@ -1096 +1106 @@
 
             //Send the task out.
-            using (Program.eraserClient = new RemoteExecutorClient())
-            {
-                if (!((RemoteExecutorClient)Program.eraserClient).Connect())
-                {
-                    //The client cannot connect to the server. This probably means
-                    //that the server process isn't running. Start an instance.
-                    Process eraserInstance = Process.Start(
-                        Assembly.GetExecutingAssembly().Location, "--quiet");
-                    eraserInstance.WaitForInputIdle();
-
+            try
+            {
+                using (Program.eraserClient = new RemoteExecutorClient())
+                {
                     if (!((RemoteExecutorClient)Program.eraserClient).Connect())
-                        throw new Exception("Eraser cannot connect to the running " +
-                            "instance for erasures.");
-                }
-
-                Program.eraserClient.Run();
-                Program.eraserClient.AddTask(ref task);
+                    {
+                        //The client cannot connect to the server. This probably means
+                        //that the server process isn't running. Start an instance.
+                        Process eraserInstance = Process.Start(
+                            Assembly.GetExecutingAssembly().Location, "--quiet");
+                        eraserInstance.WaitForInputIdle();
+
+                        if (!((RemoteExecutorClient)Program.eraserClient).Connect())
+                            throw new Exception("Eraser cannot connect to the running " +
+                                "instance for erasures.");
+                    }
+
+                    Program.eraserClient.Run();
+                    Program.eraserClient.AddTask(ref task);
+                }
+            }
+            catch (UnauthorizedAccessException e)
+            {
+                //We can't connect to the pipe because the other instance of Eraser
+                //is running with higher privileges than this instance.
+                throw new UnauthorizedAccessException("Another instance of Eraser " +
+                    "is already running but it is running with higher privileges than " +
+                    "this instance of Eraser. Tasks cannot be added in this manner.\n\n" +
+                    "Close the running instance of Eraser and start it again without " +
+                    "administrator privileges, or run the command again as an administrator",
+                    e);
             }
         }