tree 76e9fbbfdba08a56e134d3e5449be78804e2fe88
parent 3065261d511de00aece4ec976f5a8de8290b8e5b
author Aaron Green <aarongreen@pigweed.infra.roller.google.com> 1705605228 +0000
committer CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com> 1705605228 +0000

[roll third_party/pigweed] pw_protobuf: Fix out-of-range read

A failed `varint::Read` led to a nested decoder's position to not be
updated when the stream was (partially) advanced. This led to destructor
reading past the end of the stream when `CloseNestedDecoder` was
invoked.

To fix this, `varint::Read` now always returns the number of bytes read
and `StreamDecoder` always updates its position, even on error.

Original-Bug: b/314803709
Original-Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/188095

https://pigweed.googlesource.com/pigweed/pigweed
third_party/pigweed Rolled-Commits: 580207197edbf60..6dbf91509a7c697
Roller-URL: https://ci.chromium.org/b/8758537598793759473
GitWatcher: ignore
CQ-Do-Not-Cancel-Tryjobs: true
Change-Id: I245ef3df8635bbb6343f304aa5a76db05ce8da35
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/experimental/+/188330
Commit-Queue: Pigweed Roller <pigweed-roller@pigweed-service-accounts.iam.gserviceaccount.com>
Bot-Commit: Pigweed Roller <pigweed-roller@pigweed-service-accounts.iam.gserviceaccount.com>