Add missing frame release to Http2ClientStreamTransportState.

benjaminp · ejona86 · commit dc316f7fd974 · 2025-02-10T21:40:05.000-08:00
If a data frame is received before headers, processing of the frame is abandoned. The frame must be released in that case.
diff --git a/core/src/main/java/io/grpc/internal/Http2ClientStreamTransportState.java b/core/src/main/java/io/grpc/internal/Http2ClientStreamTransportState.java
@@ -140,6 +140,7 @@ protected void transportDataReceived(ReadableBuffer frame, boolean endOfStream)
       }
     } else {
       if (!headersReceived) {
+        frame.close();
         http2ProcessingFailed(
             Status.INTERNAL.withDescription("headers not received before payload"),
             false,

Original file line number	Diff line number	Diff line change
`@@ -140,6 +140,7 @@ protected void transportDataReceived(ReadableBuffer frame, boolean endOfStream)`
`140`	`140`	`}`
`141`	`141`	`} else {`
`142`	`142`	`if (!headersReceived) {`
	`143`	`+ frame.close();`
`143`	`144`	`http2ProcessingFailed(`
`144`	`145`	`Status.INTERNAL.withDescription("headers not received before payload"),`
`145`	`146`	`false,`