14

I'm trying to run a batch file, as another user, from my web app. For some reason, the batch file hangs! I can see "cmd.exe" running in the task manager, but it just sits there forever, unable to be killed, and the batch file is not running. Here's my code:

SecureString password = new SecureString();
foreach (char c in "mypassword".ToCharArray())
    password.AppendChar(c);

ProcessStartInfo psi = new ProcessStartInfo();
psi.WorkingDirectory = @"c:\build";
psi.FileName = Environment.SystemDirectory + @"\cmd.exe";
psi.Arguments = "/q /c build.cmd";
psi.UseShellExecute = false;
psi.UserName = "builder";
psi.Password = password;
Process.Start(psi);

If you didn't guess, this batch file builds my application (a different application than the one that is executing this command).

The Process.Start(psi); line returns immediately, as it should, but the batch file just seems to hang, without executing. Any ideas?

EDIT: See my answer below for the contents of the batch file.

  • The output.txt never gets created.

I added these lines:

psi.RedirectStandardOutput = true;
Process p = Process.Start(psi);
String outp = p.StandardOutput.ReadLine();

and stepped through them in debug mode. The code hangs on the ReadLine(). I'm stumped!

1
  • Can you post the code for your batch file, and have you tried emitting echoes from your batch file so you could see that it is starting?
    – Jon Limjap
    Aug 29, 2008 at 8:40

9 Answers 9

5

I believe I've found the answer. It seems that Microsoft, in all their infinite wisdom, has blocked batch files from being executed by IIS in Windows Server 2003. Brenden Tompkins has a work-around here:

http://codebetter.com/blogs/brendan.tompkins/archive/2004/05/13/13484.aspx

That won't work for me, because my batch file uses IF and GOTO, but it would definitely work for simple batch files.

3

Why not just do all the work in C# instead of using batch files?

I was bored so i wrote this real quick, it's just an outline of how I would do it since I don't know what the command line switches do or the file paths.

using System;
using System.IO;
using System.Text;
using System.Security;
using System.Diagnostics;

namespace asdf
{
    class StackoverflowQuestion
    {
        private const string MSBUILD = @"path\to\msbuild.exe";
        private const string BMAIL = @"path\to\bmail.exe";
        private const string WORKING_DIR = @"path\to\working_directory";

        private string stdout;
        private Process p;

        public void DoWork()
        {
            // build project
            StartProcess(MSBUILD, "myproject.csproj /t:Build", true);
        }

        public void StartProcess(string file, string args, bool redirectStdout)
        {
            SecureString password = new SecureString();
            foreach (char c in "mypassword".ToCharArray())
                password.AppendChar(c);

            ProcessStartInfo psi = new ProcessStartInfo();
            p = new Process();
            psi.WindowStyle = ProcessWindowStyle.Hidden;
            psi.WorkingDirectory = WORKING_DIR;
            psi.FileName = file;
            psi.UseShellExecute = false;
            psi.RedirectStandardOutput = redirectStdout;
            psi.UserName = "builder";
            psi.Password = password;
            p.StartInfo = psi;
            p.EnableRaisingEvents = true;
            p.Exited += new EventHandler(p_Exited);
            p.Start();

            if (redirectStdout)
            {
                stdout = p.StandardOutput.ReadToEnd();
            }
        }

        void p_Exited(object sender, EventArgs e)
        {
            if (p.ExitCode != 0)
            {
                // failed
                StringBuilder args = new StringBuilder();
                args.Append("-s k2smtpout.secureserver.net ");
                args.Append("-f [email protected] ");
                args.Append("-t [email protected] ");
                args.Append("-a \"Build failed.\" ");
                args.AppendFormat("-m {0} -h", stdout);

                // send email
                StartProcess(BMAIL, args.ToString(), false);
            }
        }
    }
}
2

Without seeing the build.cmd it's hard to tell what is going on, however, you should build the path using Path.Combine(arg1, arg2); It's the correct way to build a path.

Path.Combine( Environment.SystemDirectory, "cmd.exe" );

I don't remember now but don't you have to set UseShellExecute = true ?

1

Another possibility to "debug" it is to use standardoutput and then read from it:

psi.RedirectStandardOutput = True;
Process proc = Process.Start(psi);
String whatever = proc.StandardOutput.ReadLine();
1

In order to "see" what's going on, I'd suggest you transform the process into something more interactive (turn off Echo off) and put some "prints" to see if anything is actually happening. What is in the output.txt file after you run this?

Does the bmail actually executes?

Put some prints after/before to see what's going on.

Also add "@" to the arguments, just in case:

psi.Arguments = @"/q /c build.cmd";

It has to be something very simple :)

0

My guess would be that the build.cmd is waiting for some sort of user-interaction/reply. If you log the output of the command with the "> logfile.txt" operator at the end, it might help you find the problem.

0

Here's the contents of build.cmd:

@echo off
set path=C:\WINDOWS\Microsoft.NET\Framework\v2.0.50727;%path%

msbuild myproject.csproj /t:Build > output.txt
IF NOT ERRORLEVEL 1 goto :end

:error
bmail -s k2smtpout.secureserver.net -f [email protected] -t [email protected] -a "Build failed." -m output.txt -h

:end
del output.txt

As you can see, I'm careful not to output anything. It all goes to a file that gets emailed to me if the build happens to fail. I've actually been running this file as a scheduled task nightly for quite a while now. I'm trying to build a web app that allows me to run it on demand.

Thanks for everyone's help so far! The Path.Combine tip was particularly useful.

0

I think cmd.exe hangs if the parameters are incorrect.

If the batch executes correctly then I would just shell execute it like this instead.

ProcessStartInfo psi = new ProcessStartInfo();
Process p = new Process();
psi.WindowStyle = ProcessWindowStyle.Hidden;
psi.WorkingDirectory = @"c:\build";
psi.FileName = @"C:\build\build.cmd";
psi.UseShellExecute = true;
psi.UserName = "builder";
psi.Password = password;
p.StartInfo = psi;
p.Start();

Also it could be that cmd.exe just can't find build.cmd so why not give the full path to the file?

0

What are the endlines of you batch? If the code hangs on ReadLine, then the problem might be that it's unable to read the batch file…

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Not the answer you're looking for? Browse other questions tagged or ask your own question.