Fix pass by reference bug on the module side for windows error exploit

[bwatters-r7]

This fixes a bug found by @cdelafuente-r7 where we're using a pass-by-reference for strings from the mkdir method to the cleanup methods. This means that if we alter the string used to create the directory later in the module, the cleanup method tries to delete whatever is in the string at the end of the module, rather than what was in it when we called mkdir. To fix it in this module, I just added a local method called clone_mkdir where we clone the string before passing it into mkdir.

We're working on a fix that will negate the need for this here: #18403

[cdelafuente-r7]

Thanks @bwatters-r7 for this fix! It looks good to me. I retested the module and verified the directories were properly cleaned up. Thanks!