Fix use after free in DiskFilesOrDirectories

[teemperor]

Summary

We copy the local variable Resolved into Storage to keep it around. However, we then still let the SearchDir ref point to Resolved which then is used to access the already freed memory later on. With this patch we point to Storage which doesn't get deleted after the current scope exits.

Discovered by memory sanitizer in the CompletionTest.DirCompletionUsername test.

source/Commands/CommandCompletions.cpp

if (FirstSep == llvm::StringRef::npos) {
       matches.AppendString(CompletionBuffer);
       return 1;
     }
 
     // We want to keep the form the user typed, so we special case this to
     // search in the fully resolved directory, but CompletionBuffer keeps the
     // unmodified form that the user typed.
     Storage = Resolved;
-    SearchDir = Resolved;
+    SearchDir = Storage;
   } else {
     SearchDir = path::parent_path(CompletionBuffer);
   }
 
   size_t FullPrefixLen = CompletionBuffer.size();
 
   PartialItem = path::filename(CompletionBuffer);
   if (PartialItem == ".")