Hmm, yea that's not a bad idea.  Let me look into the privilege thing a little tomorrow.  In the meantime, do we properly handle the error case anyway (at least it should print a log and not have unexpected behavior).  Creating the symlink could fail on other platforms too, not just windows <br><div class="gmail_quote">On Thu, May 7, 2015 at 5:28 PM Oleksiy Vyalov <<a href="mailto:ovyalov@google.com">ovyalov@google.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">In <a href="http://reviews.llvm.org/D9587#168485" target="_blank">http://reviews.llvm.org/D9587#168485</a>, @zturner wrote:<br>
<br>
> Even with this check, creating the symbolic link might still fail for other reasons related to the filesystem and not related to security.  Would it be better to just try to create it no matter what and log the error if it fails?<br>
<br>
<br>
I believe calling EnableCreateSymLinkPrivilege might be beneficial anyway since as I can tell from running "whoami /priv" that SE_CREATE_SYMBOLIC_LINK_NAME is disabled by default . I'm somewhat concerned that we may call FileSystem::Symlink when we 100% confident that it will fail. As an alternative, we may just call FileSystem::Symlink anyway but always ignoring its result...<br>
<br>
Please let me know your opinion.<br>
<br>
<br>
<a href="http://reviews.llvm.org/D9587" target="_blank">http://reviews.llvm.org/D9587</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
<br>
<br>
</blockquote></div>