tree 81b884155d50118470004836574e51bcfa0ae0a0
parent 6124f8b496ed212d40e5c0c41cbfdc712bad53c3
author Alexei Frolov <frolv@pigweed.infra.roller.google.com> 1708477763 +0000
committer CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com> 1708477763 +0000

[roll third_party/pigweed] pw_transfer: Fix integration test START packet issue

The proxy in pw_transfer's integration tests were receiving multiple
START packets, resulting in incorrect behavior in tests. This occurred
due to an assumption that all RPC packets contained transfer chunks and
attempting to decode them. An empty RPC payload resulted in a default
(zero-valued) chunk, which was interpreted as a START chunk for legacy
protocol compatibility.

This fixes the issue by only considering RPC packets which actually
have a payload.

Original-Bug: 322497491
Original-Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/192755

https://pigweed.googlesource.com/pigweed/pigweed
third_party/pigweed Rolled-Commits: 36b6a7ddb0c28b6..b83c8ed0ce86ff9
Roller-URL: https://ci.chromium.org/b/8755525388255161329
GitWatcher: ignore
CQ-Do-Not-Cancel-Tryjobs: true
Change-Id: Ia2583798466a30b2c3be61dfe6bb0872bcc2d90a
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/experimental/+/193321
Commit-Queue: Pigweed Roller <pigweed-roller@pigweed-service-accounts.iam.gserviceaccount.com>
Bot-Commit: Pigweed Roller <pigweed-roller@pigweed-service-accounts.iam.gserviceaccount.com>