StackOverflowException with Process in C#

Question

I have a process, which runs in a console app. It runs forever.

After a couple of days the app crashes with a StackOverflowException.

The essence of the app is where I spin up a Process with FFMpeg.exe and creates a sceenshot of a video stream. It works very good but only for a few days at the time.

I am pretty sure it has to do with the disposal of the FFMpeg or some internal Process stuff.

Here is the code

using ( Process ffmpegProcess = new Process() ) {

    //arguments for running ffmpeg
    ffmpegProcess.StartInfo.UseShellExecute = false;
    ffmpegProcess.StartInfo.CreateNoWindow = true;
    ffmpegProcess.StartInfo.RedirectStandardOutput = true;

    //specific for our screenshots
    ffmpegProcess.StartInfo.FileName = string.Concat( Environment.CurrentDirectory, Path.DirectorySeparatorChar, ffmpegProgramName );

    try {
        //todo: log this stopwatch somewhere perhaps
        processWatch.Start();

        //set arguments every time we want to create a new screen shot
        ffmpegProcess.StartInfo.Arguments = string.Format( @"-y -i {0}{1} -threads 0 -ss 00:00:01.000 -f image2 -s 620x349 -vframes 1 ../../web/{2}.jpg", server, streamPath, slug );
        ffmpegProcess.Start();
        ffmpegProcess.WaitForExit( 500 );

        Console.WriteLine( slug );
        Console.WriteLine( processWatch.Elapsed );

        processWatch.Reset();
        runCount++;
        cacheIndexer++;

        //lets see how many spins we've had!
        Console.WriteLine( string.Format( "SERVER CACHE INDEX : {0}", cacheIndexer ) );
        Console.WriteLine( string.Format( "RUN : {0}", runCount ) );
        Console.WriteLine( Environment.NewLine );

    } catch ( Exception ex ) {
        //Console.WriteLine( "Ex " + ex );
    }
}

The loop looks like this.

    public void RecurseTask() {
        /*
        You can try one of these, but you will se CPU usage go crazy and perhaps concurrency errors due IO blocking

        Parallel.ForEach( _videoStreamSlugs, ( e ) => _videoStreamScreenShots.GrabScreenShot( e ) );

        foreach ( var slug in _videoStreamSlugs ) {
            Task.Run( () => _videoStreamScreenShots.GrabScreenShot( slug ) );
        }
        */

        //we want to grab screen shots for every slug in out slug list!
        foreach ( var slug in _videoStreamSlugs ) {
            _videoStreamScreenShots.GrabScreenShot( slug );
        }

        //sleep for a little while
        Thread.Sleep( _recurseInterval );

        //A heavy clean up!
        //We do this, trying to avoid a stackoverflow excecption in the recursive method
        //Please inspect this if problems arise
        GC.Collect();

        //lets grab over again
        RecurseTask();
    }

I added a GC.Collect out of curiosity to see if it made a difference.

I am not doing a Windows Service.

Answer 1

Inside RecurseTask you always call RecurseTask, obviously, when long running it will throw a StackOverflowException, you can try to change to

public void RecurseTask() {
    while(true)
    {
        /*
        You can try one of these, but you will se CPU usage go crazy and perhaps concurrency errors due IO blocking

        Parallel.ForEach( _videoStreamSlugs, ( e ) => _videoStreamScreenShots.GrabScreenShot( e ) );

        foreach ( var slug in _videoStreamSlugs ) {
            Task.Run( () => _videoStreamScreenShots.GrabScreenShot( slug ) );
        }
        */

        //we want to grab screen shots for every slug in out slug list!
        foreach ( var slug in _videoStreamSlugs ) {
            _videoStreamScreenShots.GrabScreenShot( slug );
        }

        //sleep for a little while
        Thread.Sleep( _recurseInterval );

        //A heavy clean up!
        //We do this, trying to avoid a stackoverflow excecption in the recursive method
        //Please inspect this if problems arise
        //GC.Collect(); Not needed

        //lets grab over again
    }
}