Merge pull request #39856 from yokotaso

philwebb · philwebb · commit bd16fc4d26a1 · 2024-05-01T22:46:39.000-07:00
* pr/39856: Fix possible NullPointerException from getPermission() Closes gh-39856
diff --git a/spring-boot-project/spring-boot-tools/spring-boot-loader/src/main/java/org/springframework/boot/loader/net/protocol/jar/JarUrlConnection.java b/spring-boot-project/spring-boot-tools/spring-boot-loader/src/main/java/org/springframework/boot/loader/net/protocol/jar/JarUrlConnection.java
@@ -172,7 +172,7 @@ public Object getContent() throws IOException {
 
 	@Override
 	public Permission getPermission() throws IOException {
-		return this.jarFileConnection.getPermission();
+		return (this.jarFileConnection != null) ? this.jarFileConnection.getPermission() : null;
 	}
 
 	@Override

Original file line number	Diff line number	Diff line change
`@@ -172,7 +172,7 @@ public Object getContent() throws IOException {`
`172`	`172`
`173`	`173`	`@Override`
`174`	`174`	`public Permission getPermission() throws IOException {`
`175`		`- return this.jarFileConnection.getPermission();`
	`175`	`+ return (this.jarFileConnection != null) ? this.jarFileConnection.getPermission() : null;`
`176`	`176`	`}`
`177`	`177`
`178`	`178`	`@Override`