test: more coverage

Signed-off-by: Shiwei Zhang <shizh@microsoft.com>
veraison · Feb 20, 2023 · ad6c208 · ad6c208
1 parent e648281
commit ad6c208
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/cbor.go b/cbor.go
@@ -122,9 +122,8 @@ func deterministicBinaryString(data cbor.RawMessage) (cbor.RawMessage, error) {
 	}
 
 	// slow path: convert by re-encoding
+	// error checking is not required since `data` has been validataed
 	var s []byte
-	if err := decModeWithTagsForbidden.Unmarshal(data, &s); err != nil {
-		return nil, err
-	}
+	_ = decModeWithTagsForbidden.Unmarshal(data, &s)
 	return encMode.Marshal(s)
 }