[JENKINS-13271] fix NPE when parsing changes containing a deleted per…

…force user
jenkinsci · Mar 29, 2012 · 2a92a46 · 2a92a46
1 parent 2a659c6
commit 2a92a46
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/src/main/java/hudson/plugins/perforce/PerforceSCM.java b/src/main/java/hudson/plugins/perforce/PerforceSCM.java
@@ -617,6 +617,12 @@ private static void retrieveUserInformation(Depot depot, List<Changelist> change
             }catch(Exception e){
                 throw new PerforceException("Problem getting user information for " + user,e);
             }
+            //If there is no such user in perforce, then ignore and keep going.
+            if(pu == null){
+                LOGGER.warning("Perforce User ("+user+") does not exist.");
+                continue;
+            }
+
             User author = User.get(user);
             // Need to store the actual perforce user id for later retrieval
             // because Jenkins does not support all the same characters that