Click here to Skip to main content
11,584,460 members (63,779 online)
Click here to Skip to main content

Tagged as

Deleting folders is a risky business

, 30 Jan 2012 CPOL 15.7K 12
Rate this:
Please Sign up or sign in to vote.
Deleting folder programmatically is extremely dangerous. It can erase your system.
Deleting folder programmatically is extremely dangerous. It is enough for somebody else to change a configuration file or constant variable including the target folder's name to, say C:\, and that's it: the workstation is paralyzed!

Following suggested methods are to prevent such a situation. Their aim is to wrap the low level, possibly recursive file system operation with the necessary validation and clear facade - look twice, delete once.

Delete temporary directory created by current user (non recursive!)


    /// <summary>
    /// carefully remove directory and its files
    /// verify the directory is located under the %temp%
    /// and it is not e.g. C:\
    /// </summary>
    /// <param name="dir"></param>

    public static void DeleteTempDirShallow(string dir)
    {
        // check if it is an invalid directory path,
        // e.g. a disk drive or just a bad string

        if (! Directory.Exists(dir)) return;
 
        DirectoryInfo userTempDirInfo = new DirectoryInfo(Path.GetTempPath());
        DirectoryInfo dirInfo = new DirectoryInfo(dir);
 
        if (dirInfo.FullName.Contains(userTempDirInfo.FullName))
        {
            foreach (FileInfo file in dirInfo.GetFiles())
                file.Delete();
 
            dirInfo.Delete(); // just clean up the empty dir
        }
    }

Delete temporary directory created by current user (recursive!)


    public static void DeleteTempDirRecursive(string dir)
    {
        // check if it is an invalid directory path,
        // e.g. a disk drive or just a bad string
        if (! Directory.Exists(dir)) return;
 
        DirectoryInfo userTempDirInfo = new DirectoryInfo(Path.GetTempPath());
        DirectoryInfo dirInfo = new DirectoryInfo(dir);
 
        if (dirInfo.FullName.Contains(userTempDirInfo.FullName))
        {
            dirInfo.Delete(recursive: true);
        }
    }

Do I try to delete any root or system folder?


    static List<string> systemDirs = new List<string>
    {
        Environment.GetFolderPath(Environment.SpecialFolder.CommonProgramFiles),
        Environment.GetFolderPath(Environment.SpecialFolder.CommonProgramFilesX86),
        Environment.GetFolderPath(Environment.SpecialFolder.CommonPrograms),
        Environment.GetFolderPath(Environment.SpecialFolder.CommonStartMenu),
        Environment.GetFolderPath(Environment.SpecialFolder.CommonStartup),
        Environment.GetFolderPath(Environment.SpecialFolder.Fonts),
        Environment.GetFolderPath(Environment.SpecialFolder.NetworkShortcuts),
        Environment.GetFolderPath(Environment.SpecialFolder.PrinterShortcuts),
        Environment.GetFolderPath(Environment.SpecialFolder.ProgramFiles),
        Environment.GetFolderPath(Environment.SpecialFolder.ProgramFilesX86),
        Environment.GetFolderPath(Environment.SpecialFolder.Resources),
        Environment.GetFolderPath(Environment.SpecialFolder.StartMenu),
        Environment.GetFolderPath(Environment.SpecialFolder.Startup),
        Environment.GetFolderPath(Environment.SpecialFolder.System),
        Environment.GetFolderPath(Environment.SpecialFolder.SystemX86),
        Environment.GetFolderPath(Environment.SpecialFolder.Templates),
        Environment.GetFolderPath(Environment.SpecialFolder.UserProfile),
        Environment.GetFolderPath(Environment.SpecialFolder.Windows),
    };
 
    /// <summary>
    /// check if the argument dir is a disk drive,
    /// is a system (sub-)folder
    /// and should not be erased
    /// </summary>
    /// <param name="dir"></param>
    /// <returns></returns>
    public static bool IsSystemOrRootDir(string dir)
    {
        // check if it is an invalid directory path,
        // or just a bad string
        if (! Directory.Exists(dir)) return true;
 
        DirectoryInfo dirInfo = new DirectoryInfo(dir);
 
        // is it a root (disk drive)? 
        if (dirInfo.Parent == null) return true;
 
        bool result = false;
        systemDirs.ForEach(sysDir => { result = result ||
                                       dirInfo.FullName.Contains(sysDir); } );
        return result;
    }

References

License

This article, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Share

About the Author

Michael Gr
United States United States
No Biography provided

You may also be interested in...

Comments and Discussions

 
GeneralMy vote of 5 Pin
hypermellow11-Feb-13 22:53
memberhypermellow11-Feb-13 22:53 
GeneralMy vote of 4 Pin
jfriedman13-May-12 3:57
memberjfriedman13-May-12 3:57 
GeneralRe: My vote of 4 Pin
DelphiCoder31-Aug-14 12:36
memberDelphiCoder31-Aug-14 12:36 
GeneralRe: Thank You! Pin
Michael Gr30-Jan-12 2:10
memberMichael Gr30-Jan-12 2:10 
GeneralRe: OK , I updated my Vote, +5 Pin
Rajesh Anuhya30-Jan-12 1:55
memberRajesh Anuhya30-Jan-12 1:55 
Generalnice !! Pin
Abhishek Unnikrishnan12-Feb-12 5:45
memberAbhishek Unnikrishnan12-Feb-12 5:45 
GeneralReason for my vote of 5 I wasn't aware of this in 24 years o... Pin
javaevangelist7-Feb-12 8:18
memberjavaevangelist7-Feb-12 8:18 
GeneralReason for my vote of 5 Very nice Pin
TigerHajtos6-Feb-12 21:24
memberTigerHajtos6-Feb-12 21:24 
GeneralReason for my vote of 5 I have to admit, I have never (thank... Pin
Marek Roszak6-Feb-12 20:55
memberMarek Roszak6-Feb-12 20:55 
GeneralNever thought about this.. Thankyou. Have 5 Pin
Pankaj Chamria30-Jan-12 2:55
memberPankaj Chamria30-Jan-12 2:55 
GeneralRe: Thanks Pankaj. Pin
Michael Gr30-Jan-12 22:34
memberMichael Gr30-Jan-12 22:34 
GeneralCopied from Here:http://codereview.stackexchange.com/questio... Pin
Rajesh Anuhya30-Jan-12 1:18
memberRajesh Anuhya30-Jan-12 1:18 
GeneralReason for my vote of 1 Copied from Here : http://codereview... Pin
Rajesh Anuhya30-Jan-12 1:18
memberRajesh Anuhya30-Jan-12 1:18 
GeneralRe: Dear Rajesh. Actually I have published the article first at ... Pin
quiet.step30-Jan-12 1:38
memberquiet.step30-Jan-12 1:38 
GeneralReason for my vote of 5 Very nice! Pin
Pablo Aliskevicius29-Jan-12 19:35
memberPablo Aliskevicius29-Jan-12 19:35 
GeneralRe: Thanks. Pin
Michael Gr30-Jan-12 2:11
memberMichael Gr30-Jan-12 2:11 

General General    News News    Suggestion Suggestion    Question Question    Bug Bug    Answer Answer    Joke Joke    Rant Rant    Admin Admin   

Use Ctrl+Left/Right to switch messages, Ctrl+Up/Down to switch threads, Ctrl+Shift+Left/Right to switch pages.

| Advertise | Privacy | Terms of Use | Mobile
Web04 | 2.8.150603.1 | Last Updated 30 Jan 2012
Article Copyright 2012 by Michael Gr
Everything else Copyright © CodeProject, 1999-2015
Layout: fixed | fluid