From 64af8eb468c55a92580b89612ffc7af15a2f08fa Mon Sep 17 00:00:00 2001 From: Claude Date: Thu, 2 Apr 2026 09:35:35 +0100 Subject: [PATCH] ax(ueps): remove superfluous payloadBytes intermediate variable payload was declared via var at the top of ReadAndVerify; the inner payloadBytes := io.ReadAll + payload = payloadBytes pattern introduced an unnecessary name that added no semantic value. Assign directly to the outer payload variable instead. Co-Authored-By: Charon --- pkg/ueps/reader.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/pkg/ueps/reader.go b/pkg/ueps/reader.go index f29b4f7..dfdf3cd 100644 --- a/pkg/ueps/reader.go +++ b/pkg/ueps/reader.go @@ -44,11 +44,11 @@ func ReadAndVerify(reader *bufio.Reader, sharedSecret []byte) (*ParsedPacket, er if tagByte == TagPayload { // Payload is length-prefixless; caller frames the stream. // HMAC covers signedData (header TLVs) + raw payload bytes, not the 0xFF tag. - payloadBytes, err := io.ReadAll(reader) + var err error + payload, err = io.ReadAll(reader) if err != nil { return nil, err } - payload = payloadBytes break }