Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
[JENKINS-21631] - Don't fail with NPE if the PeepholePermalink cache …
…contains the non-numeric value.

Actually, the change just prevents symptoms. There should be an error somewhere else.

Signed-off-by: Oleg Nenashev <o.v.nenashev@gmail.com>
(cherry picked from commit 5df3f74)
  • Loading branch information
oleg-nenashev authored and olivergondza committed Jul 3, 2014
1 parent 56f1ee4 commit e79f66b
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions core/src/main/java/jenkins/model/PeepholePermalink.java
Expand Up @@ -98,6 +98,9 @@ protected File getPermalinkFile(Job<?,?> job) {
} catch (InterruptedException e) {
LOGGER.log(Level.WARNING, "Failed to read permalink cache:" + f, e);
// if we fail to read the cache, fall back to the re-computation
} catch (NumberFormatException e) {
LOGGER.log(Level.WARNING, "Failed to parse the build number in the permalink cache:" + f, e);
// if we fail to read the cache, fall back to the re-computation
} catch (IOException e) {
// this happens when the symlink doesn't exist
// (and it cannot be distinguished from the case when the actual I/O error happened
Expand Down

0 comments on commit e79f66b

Please sign in to comment.