[release-1.14] Fix possible nil pointer dereference in event-dispatch…

…er (#7997) * Fix possible nil pointer dereference in event-dispatcher * readErr -> err --------- Co-authored-by: Christoph Stäbler <[email protected]>
knative · Jun 12, 2024 · 84a331f · 84a331f
1 parent 3b3d77f
commit 84a331f
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/pkg/kncloudevents/event_dispatcher.go b/pkg/kncloudevents/event_dispatcher.go
@@ -325,11 +325,11 @@ func (d *Dispatcher) executeRequest(ctx context.Context, target duckv1.Addressab
 	dispatchInfo.ResponseHeader = response.Header
 
 	body := new(bytes.Buffer)
-	_, readErr := body.ReadFrom(response.Body)
+	_, err = body.ReadFrom(response.Body)
 
 	if isFailure(response.StatusCode) {
 		// Read response body into dispatchInfo for failures
-		if readErr != nil && readErr != io.EOF {
+		if err != nil && err != io.EOF {
 			dispatchInfo.ResponseBody = []byte(fmt.Sprintf("dispatch resulted in status \"%s\". Could not read response body: error: %s", response.Status, err.Error()))
 		} else {
 			dispatchInfo.ResponseBody = body.Bytes()
@@ -341,7 +341,7 @@ func (d *Dispatcher) executeRequest(ctx context.Context, target duckv1.Addressab
 	}
 
 	var responseMessageBody []byte
-	if readErr != nil && readErr != io.EOF {
+	if err != nil && err != io.EOF {
 		responseMessageBody = []byte(fmt.Sprintf("Failed to read response body: %s", err.Error()))
 	} else {
 		responseMessageBody = body.Bytes()