Fix Bug: Make sure direct memory is freed properly

caixunshi · spencergibb · commit aa8851751b2c · 2024-03-21T15:13:49.000-04:00
diff --git a/spring-cloud-gateway-server/src/main/java/org/springframework/cloud/gateway/filter/RemoveCachedBodyFilter.java b/spring-cloud-gateway-server/src/main/java/org/springframework/cloud/gateway/filter/RemoveCachedBodyFilter.java
@@ -40,7 +40,8 @@ public Mono<Void> filter(ServerWebExchange exchange, GatewayFilterChain chain) {
 					if (log.isTraceEnabled()) {
 						log.trace("releasing cached body in exchange attribute");
 					}
-					dataBuffer.release();
+					// ensure proper release
+					while(!dataBuffer.release()) {}
 				}
 			}
 		});

Original file line number	Diff line number	Diff line change
`@@ -40,7 +40,8 @@ public Mono<Void> filter(ServerWebExchange exchange, GatewayFilterChain chain) {`
`40`	`40`	`if (log.isTraceEnabled()) {`
`41`	`41`	`log.trace("releasing cached body in exchange attribute");`
`42`	`42`	`}`
`43`		`- dataBuffer.release();`
	`43`	`+ // ensure proper release`
	`44`	`+ while(!dataBuffer.release()) {}`
`44`	`45`	`}`
`45`	`46`	`}`
`46`	`47`	`});`