From 3c2b50ecfe1c58f64b32d30a3f96e647671f7d03 Mon Sep 17 00:00:00 2001
From: Jesse Glick <jglick@cloudbees.com>
Date: Wed, 1 May 2013 20:12:59 -0400
Subject: [PATCH] [FIXED JENKINS-17681] Do not try to call resolveSymlink on a
 nonexistent link as you will just get an IOException, breaking peephole
 permalinks in some Windows environments.

---
 changelog.html                                         |  3 +++
 core/src/main/java/hudson/Util.java                    |  4 ++++
 .../src/main/java/jenkins/model/PeepholePermalink.java | 10 +++++++++-
 3 files changed, 16 insertions(+), 1 deletion(-)

diff --git a/changelog.html b/changelog.html
index ca690e085b..8e738d6328 100644
--- a/changelog.html
+++ b/changelog.html
@@ -58,6 +58,9 @@ Upcoming changes</a>
   <li class=bug>
     Display Name is not shown.
      (<a href="https://issues.jenkins-ci.org/browse/JENKINS-17715">issue 17715</a>)
+  <li class='major bug'>
+    Symlink handling problem with build permalinks on Windows.
+    (<a href="https://issues.jenkins-ci.org/browse/JENKINS-17681">issue 17681</a>)
   <li class=bug>
     List views missing a required field were unloadable.
     (<a href="https://issues.jenkins-ci.org/browse/JENKINS-15309">issue 15309</a>)
diff --git a/core/src/main/java/hudson/Util.java b/core/src/main/java/hudson/Util.java
index 1365bca2be..511d9eefa8 100644
--- a/core/src/main/java/hudson/Util.java
+++ b/core/src/main/java/hudson/Util.java
@@ -1200,6 +1200,10 @@ public class Util {
             } catch (ClassNotFoundException x3) {
                 assert false : x3; // should be Java 7+ here
             }
+            if (x2.getClass().getName().equals("java.nio.file.FileSystemException")) {
+                // Thrown ("Incorrect function.") on JDK 7u21 in Windows 2012 when called on a non-symlink, rather than NotLinkException, contrary to documentation. Maybe only when not on NTFS?
+                return null;
+            }
             if (x2 instanceof IOException) {
                 throw (IOException) x2;
             }
diff --git a/core/src/main/java/jenkins/model/PeepholePermalink.java b/core/src/main/java/jenkins/model/PeepholePermalink.java
index 2e4677a134..397d035743 100644
--- a/core/src/main/java/jenkins/model/PeepholePermalink.java
+++ b/core/src/main/java/jenkins/model/PeepholePermalink.java
@@ -17,6 +17,7 @@ import javax.annotation.Nullable;
 import java.io.File;
 import java.io.IOException;
 import java.io.StringWriter;
+import java.util.Arrays;
 import java.util.logging.Level;
 import java.util.logging.Logger;
 
@@ -143,6 +144,12 @@ public abstract class PeepholePermalink extends Permalink implements Predicate<R
         }
     }
 
+    // File.exists returns false for a link with a missing target, so for Java 6 compatibility we have to use this circuitous method to see if it was created.
+    private static boolean exists(File link) {
+        File[] kids = link.getParentFile().listFiles();
+        return kids != null && Arrays.asList(kids).contains(link);
+    }
+
     static String readSymlink(File cache) throws IOException, InterruptedException {
         String target = Util.resolveSymlink(cache);
         if (target==null && cache.exists()) {
@@ -158,7 +165,8 @@ public abstract class PeepholePermalink extends Permalink implements Predicate<R
         File tmp = new File(cache.getPath()+".tmp");
         try {
             Util.createSymlink(tmp.getParentFile(),target,tmp.getName(),listener);
-            if (Util.resolveSymlink(tmp)==null) {
+            // Avoid calling resolveSymlink on a nonexistent file as it will probably throw an IOException:
+            if (!exists(tmp) || Util.resolveSymlink(tmp)==null) {
                 // symlink not supported. use a regular file
                 AtomicFileWriter cw = new AtomicFileWriter(cache);
                 try {