delete tree function Planned maintenance scheduled April 23, 2019 at 23:30 UTC (7:30pm US/Eastern) Announcing the arrival of Valued Associate #679: Cesar Manara Unicorn Meta Zoo #1: Why another podcast?Recursive hashing functionRecursive search to delete n'th child in treeGlob-Like Search in VBAWalk a directory tree recursivelyRecursive file copy functionSanitizing functionRecursively delete empty folders in LuaSecurely Delete All Drive DataReading Files in a File Manager Plugin for WYSIWYG EditorsWordPress plugin that accepts a zip file, moves it to a specific folder, then unzips it

How to name indistinguishable henchmen in a screenplay?

When does a function NOT have an antiderivative?

.bashrc alias for a command with fixed second parameter

Table formatting with tabularx?

Where did Ptolemy compare the Earth to the distance of fixed stars?

Is the time—manner—place ordering of adverbials an oversimplification?

Did pre-Columbian Americans know the spherical shape of the Earth?

Does a random sequence of vectors span a Hilbert space?

How to make triangles with rounded sides and corners? (squircle with 3 sides)

Understanding piped commands in GNU/Linux

Diophantine equation 3^a+1=3^b+5^c

Why complex landing gears are used instead of simple, reliable and light weight muscle wire or shape memory alloys?

How do I say "this must not happen"?

One-one communication

Any stored/leased 737s that could substitute for grounded MAXs?

Sally's older brother

Is this Kuo-toa homebrew race balanced?

What did Turing mean when saying that "machines cannot give rise to surprises" is due to a fallacy?

Vertical ranges of Column Plots in 12

Weaponising the Grasp-at-a-Distance spell

Is it OK to use the testing sample to compare algorithms?

Should man-made satellites feature an intelligent inverted "cow catcher"?

How can I prevent/balance waiting and turtling as a response to cooldown mechanics

How to infer difference of population proportion between two groups when proportion is small?



delete tree function



Planned maintenance scheduled April 23, 2019 at 23:30 UTC (7:30pm US/Eastern)
Announcing the arrival of Valued Associate #679: Cesar Manara
Unicorn Meta Zoo #1: Why another podcast?Recursive hashing functionRecursive search to delete n'th child in treeGlob-Like Search in VBAWalk a directory tree recursivelyRecursive file copy functionSanitizing functionRecursively delete empty folders in LuaSecurely Delete All Drive DataReading Files in a File Manager Plugin for WYSIWYG EditorsWordPress plugin that accepts a zip file, moves it to a specific folder, then unzips it



.everyoneloves__top-leaderboard:empty,.everyoneloves__mid-leaderboard:empty,.everyoneloves__bot-mid-leaderboard:empty margin-bottom:0;








0












$begingroup$


I made this function to delete a directory with all it's contents recursively, will it work as expected ? is it safe ? I don't want to wake up one day with /home contents is gone :D



public static function delTree($dir) 
if(!is_dir($dir))return false;;

$files = scandir($dir);if(!$files)return false;
$files = array_diff($files, array('.','..'));

foreach ($files as $file)
(is_dir("$dir/$file")) ? SELF::delTree("$dir/$file") : unlink("$dir/$file");


return rmdir($dir);



Note: I use this function internally, meaning there is no client parameters like directory names is taken from the client before I call it, so there is no chance for traversal attacks, and I check the base path with another function before I call it, for example to delete a client folder I do something like this



$clientsFolderPath = $_SERVER['DOCUMENT_ROOT'] . "/../clients"
$clientFolderPath = "$clientsFolderPath/$clientId";

$realBase = realpath($clientsFolderPath);
$realClientDir = realpath($clientFolderPath);
if ( $realBase || !$realClientDir || strpos($realClientDir, $realBase) !== 0 )
//error, log , and exit;
else
ExtendedSystemModel::delTree($clientFolderPath);









share









$endgroup$


















    0












    $begingroup$


    I made this function to delete a directory with all it's contents recursively, will it work as expected ? is it safe ? I don't want to wake up one day with /home contents is gone :D



    public static function delTree($dir) 
    if(!is_dir($dir))return false;;

    $files = scandir($dir);if(!$files)return false;
    $files = array_diff($files, array('.','..'));

    foreach ($files as $file)
    (is_dir("$dir/$file")) ? SELF::delTree("$dir/$file") : unlink("$dir/$file");


    return rmdir($dir);



    Note: I use this function internally, meaning there is no client parameters like directory names is taken from the client before I call it, so there is no chance for traversal attacks, and I check the base path with another function before I call it, for example to delete a client folder I do something like this



    $clientsFolderPath = $_SERVER['DOCUMENT_ROOT'] . "/../clients"
    $clientFolderPath = "$clientsFolderPath/$clientId";

    $realBase = realpath($clientsFolderPath);
    $realClientDir = realpath($clientFolderPath);
    if ( $realBase || !$realClientDir || strpos($realClientDir, $realBase) !== 0 )
    //error, log , and exit;
    else
    ExtendedSystemModel::delTree($clientFolderPath);









    share









    $endgroup$














      0












      0








      0





      $begingroup$


      I made this function to delete a directory with all it's contents recursively, will it work as expected ? is it safe ? I don't want to wake up one day with /home contents is gone :D



      public static function delTree($dir) 
      if(!is_dir($dir))return false;;

      $files = scandir($dir);if(!$files)return false;
      $files = array_diff($files, array('.','..'));

      foreach ($files as $file)
      (is_dir("$dir/$file")) ? SELF::delTree("$dir/$file") : unlink("$dir/$file");


      return rmdir($dir);



      Note: I use this function internally, meaning there is no client parameters like directory names is taken from the client before I call it, so there is no chance for traversal attacks, and I check the base path with another function before I call it, for example to delete a client folder I do something like this



      $clientsFolderPath = $_SERVER['DOCUMENT_ROOT'] . "/../clients"
      $clientFolderPath = "$clientsFolderPath/$clientId";

      $realBase = realpath($clientsFolderPath);
      $realClientDir = realpath($clientFolderPath);
      if ( $realBase || !$realClientDir || strpos($realClientDir, $realBase) !== 0 )
      //error, log , and exit;
      else
      ExtendedSystemModel::delTree($clientFolderPath);









      share









      $endgroup$




      I made this function to delete a directory with all it's contents recursively, will it work as expected ? is it safe ? I don't want to wake up one day with /home contents is gone :D



      public static function delTree($dir) 
      if(!is_dir($dir))return false;;

      $files = scandir($dir);if(!$files)return false;
      $files = array_diff($files, array('.','..'));

      foreach ($files as $file)
      (is_dir("$dir/$file")) ? SELF::delTree("$dir/$file") : unlink("$dir/$file");


      return rmdir($dir);



      Note: I use this function internally, meaning there is no client parameters like directory names is taken from the client before I call it, so there is no chance for traversal attacks, and I check the base path with another function before I call it, for example to delete a client folder I do something like this



      $clientsFolderPath = $_SERVER['DOCUMENT_ROOT'] . "/../clients"
      $clientFolderPath = "$clientsFolderPath/$clientId";

      $realBase = realpath($clientsFolderPath);
      $realClientDir = realpath($clientFolderPath);
      if ( $realBase || !$realClientDir || strpos($realClientDir, $realBase) !== 0 )
      //error, log , and exit;
      else
      ExtendedSystemModel::delTree($clientFolderPath);







      php recursion security file-system





      share












      share










      share



      share










      asked 3 mins ago









      Accountant مAccountant م

      216111




      216111




















          0






          active

          oldest

          votes












          Your Answer






          StackExchange.ifUsing("editor", function ()
          StackExchange.using("externalEditor", function ()
          StackExchange.using("snippets", function ()
          StackExchange.snippets.init();
          );
          );
          , "code-snippets");

          StackExchange.ready(function()
          var channelOptions =
          tags: "".split(" "),
          id: "196"
          ;
          initTagRenderer("".split(" "), "".split(" "), channelOptions);

          StackExchange.using("externalEditor", function()
          // Have to fire editor after snippets, if snippets enabled
          if (StackExchange.settings.snippets.snippetsEnabled)
          StackExchange.using("snippets", function()
          createEditor();
          );

          else
          createEditor();

          );

          function createEditor()
          StackExchange.prepareEditor(
          heartbeatType: 'answer',
          autoActivateHeartbeat: false,
          convertImagesToLinks: false,
          noModals: true,
          showLowRepImageUploadWarning: true,
          reputationToPostImages: null,
          bindNavPrevention: true,
          postfix: "",
          imageUploader:
          brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
          contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
          allowUrls: true
          ,
          onDemand: true,
          discardSelector: ".discard-answer"
          ,immediatelyShowMarkdownHelp:true
          );



          );













          draft saved

          draft discarded


















          StackExchange.ready(
          function ()
          StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fcodereview.stackexchange.com%2fquestions%2f217863%2fdelete-tree-function%23new-answer', 'question_page');

          );

          Post as a guest















          Required, but never shown

























          0






          active

          oldest

          votes








          0






          active

          oldest

          votes









          active

          oldest

          votes






          active

          oldest

          votes















          draft saved

          draft discarded
















































          Thanks for contributing an answer to Code Review Stack Exchange!


          • Please be sure to answer the question. Provide details and share your research!

          But avoid


          • Asking for help, clarification, or responding to other answers.

          • Making statements based on opinion; back them up with references or personal experience.

          Use MathJax to format equations. MathJax reference.


          To learn more, see our tips on writing great answers.




          draft saved


          draft discarded














          StackExchange.ready(
          function ()
          StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fcodereview.stackexchange.com%2fquestions%2f217863%2fdelete-tree-function%23new-answer', 'question_page');

          );

          Post as a guest















          Required, but never shown





















































          Required, but never shown














          Required, but never shown












          Required, but never shown







          Required, but never shown

































          Required, but never shown














          Required, but never shown












          Required, but never shown







          Required, but never shown







          Popular posts from this blog

          名間水力發電廠 目录 沿革 設施 鄰近設施 註釋 外部連結 导航菜单23°50′10″N 120°42′41″E / 23.83611°N 120.71139°E / 23.83611; 120.7113923°50′10″N 120°42′41″E / 23.83611°N 120.71139°E / 23.83611; 120.71139計畫概要原始内容臺灣第一座BOT 模式開發的水力發電廠-名間水力電廠名間水力發電廠 水利署首件BOT案原始内容《小檔案》名間電廠 首座BOT水力發電廠原始内容名間電廠BOT - 經濟部水利署中區水資源局

          Prove that NP is closed under karp reduction?Space(n) not closed under Karp reductions - what about NTime(n)?Class P is closed under rotation?Prove or disprove that $NL$ is closed under polynomial many-one reductions$mathbfNC_2$ is closed under log-space reductionOn Karp reductionwhen can I know if a class (complexity) is closed under reduction (cook/karp)Check if class $PSPACE$ is closed under polyonomially space reductionIs NPSPACE also closed under polynomial-time reduction and under log-space reduction?Prove PSPACE is closed under complement?Prove PSPACE is closed under union?

          Is my guitar’s action too high? Announcing the arrival of Valued Associate #679: Cesar Manara Planned maintenance scheduled April 23, 2019 at 23:30 UTC (7:30pm US/Eastern)Strings too stiff on a recently purchased acoustic guitar | Cort AD880CEIs the action of my guitar really high?Μy little finger is too weak to play guitarWith guitar, how long should I give my fingers to strengthen / callous?When playing a fret the guitar sounds mutedPlaying (Barre) chords up the guitar neckI think my guitar strings are wound too tight and I can't play barre chordsF barre chord on an SG guitarHow to find to the right strings of a barre chord by feel?High action on higher fret on my steel acoustic guitar