| commit | 97613d6ea39ae99f2d29a708b1550dae5ef37bde | [log] [tgz] |
|---|---|---|
| author | Wyatt Hepler <hepler@pigweed.infra.roller.google.com> | Wed May 07 12:50:05 2025 -0700 |
| committer | CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com> | Wed May 07 12:50:05 2025 -0700 |
| tree | 28d4864ca5a6db620bf83cc24b969afdd7757736 | |
| parent | 7f22cb1f2a2d51d545cd2f11797fbf0a7cfb769d [diff] |
roll: third_party/pigweed pw_tokenizer: Never silently detokenize collisions in Python To avoid misrepresenting messages, nested messages should only be decoded if they are unambiguous -- only one successful way to decode them. The Python nested detokenization code was accidentally checking .matches() (the list of possible matches) instead of .ok(). This evaluated true for any positive number of matches, which could silently result in an ambiguous decode. This commit updates nested Base64 decoding to only decode unambiguous matches. Original-Bug: b/416030128 Original-Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/289452 Original-Revision: 8bbf38974855143e72c88b5ac43f2c65f04caabd Rolled-Repo: https://pigweed.googlesource.com/pigweed/pigweed Rolled-Commits: 5c5e892302c067..8bbf3897485514 Roll-Count: 1 Roller-URL: https://cr-buildbucket.appspot.com/build/8715501811561615681 GitWatcher: ignore CQ-Do-Not-Cancel-Tryjobs: true Change-Id: If00cb8f63f20a7fe9bc771d6f5e508dcd65f8c78 Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/experimental/+/289553 Bot-Commit: Pigweed Roller <pigweed-roller@pigweed-service-accounts.iam.gserviceaccount.com> Commit-Queue: Pigweed Roller <pigweed-roller@pigweed-service-accounts.iam.gserviceaccount.com> Lint: Lint 🤖 <android-build-ayeaye@system.gserviceaccount.com>
diff --git a/third_party/pigweed b/third_party/pigweed index 5c5e892..8bbf389 160000 --- a/third_party/pigweed +++ b/third_party/pigweed
@@ -1 +1 @@ -Subproject commit 5c5e892302c0675b6ffbe030e7884fb12ecf113a +Subproject commit 8bbf38974855143e72c88b5ac43f2c65f04caabd