[M148] [XKB] Harden the layout name parser. [chromium/src : refs/branch-heads/7778]

1 view
Skip to first unread message

chrome-cherry-picker@chops-service-accounts.iam.gserviceaccount.com (Gerrit)

unread,
Apr 21, 2026, 9:16:50 PM (8 days ago) Apr 21
to Mike West, android-bu...@system.gserviceaccount.com, extension...@chromium.org, ozone-...@chromium.org, chromium-a...@chromium.org

chrome-che...@chops-service-accounts.iam.gserviceaccount.com voted Auto-Submit+1

Auto-Submit+1
Open in Gerrit

Related details

Attention set is empty
Submit Requirements:
  • requirement satisfiedCode-Owners
  • requirement is not satisfiedCode-Review
  • requirement satisfiedLint
  • requirement is not satisfiedReview-Enforcement
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: chromium/src
Gerrit-Branch: refs/branch-heads/7778
Gerrit-Change-Id: I22864f9deb130a697da2c768a76fa63c8d9429fe
Gerrit-Change-Number: 7784454
Gerrit-PatchSet: 2
Gerrit-CC: Mike West <mk...@chromium.org>
Gerrit-Comment-Date: Wed, 22 Apr 2026 01:16:43 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
satisfied_requirement
unsatisfied_requirement
open
diffy

rubber-stamper@appspot.gserviceaccount.com (Gerrit)

unread,
Apr 21, 2026, 9:17:12 PM (8 days ago) Apr 21
to chrome-che...@chops-service-accounts.iam.gserviceaccount.com, Mike West, android-bu...@system.gserviceaccount.com, extension...@chromium.org, ozone-...@chromium.org, chromium-a...@chromium.org

Related details

Attention set is empty
Submit Requirements:
  • requirement satisfiedCode-Owners
  • requirement satisfiedCode-Review
  • requirement satisfiedLint
  • requirement satisfiedReview-Enforcement
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: chromium/src
Gerrit-Branch: refs/branch-heads/7778
Gerrit-Change-Id: I22864f9deb130a697da2c768a76fa63c8d9429fe
Gerrit-Change-Number: 7784454
Gerrit-PatchSet: 2
Gerrit-Comment-Date: Wed, 22 Apr 2026 01:17:06 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
satisfied_requirement
open
diffy

Chromium LUCI CQ (Gerrit)

unread,
Apr 21, 2026, 10:19:55 PM (8 days ago) Apr 21
to chrome-che...@chops-service-accounts.iam.gserviceaccount.com, Mike West, rubber-...@appspot.gserviceaccount.com, android-bu...@system.gserviceaccount.com, extension...@chromium.org, ozone-...@chromium.org, chromium-a...@chromium.org

Chromium LUCI CQ submitted the change

Change information

Commit message:
[M148] [XKB] Harden the layout name parser.

Original change's description:
> [XKB] Harden the layout name parser.
>
> This CL adds validation to the input component layout name parser and
> the XKB layout name parser in order to ensure that they match reasonable
> expectations. Since they're used as filenames in `libxkbcommon`, this
> might prevent unexpected path traversal.
>
> Bug: 499130207
> Change-Id: I22864f9deb130a697da2c768a76fa63c8d9429fe
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7735525
> Commit-Queue: Mike West <mk...@chromium.org>
> Reviewed-by: Finnur Thorarinsson <fin...@chromium.org>
> Reviewed-by: Sean O'Brien <seob...@chromium.org>
> Cr-Commit-Position: refs/heads/main@{#1612696}

(cherry picked from commit 73daddbbeb66872bfa5352783bd4422614b55eee)
Bug: 501628566,499130207
Change-Id: I22864f9deb130a697da2c768a76fa63c8d9429fe
Cr-Commit-Position: refs/branch-heads/7778@{#1328}
Cr-Branched-From: 77f495ee216d4c3cc784d33658bad4778c0680ee-refs/heads/main@{#1610480}
Files:
  • M extensions/common/BUILD.gn
  • M extensions/common/manifest_handlers/input_components_handler.cc
  • A extensions/common/manifest_handlers/input_components_handler_unittest.cc
  • M ui/events/ozone/layout/xkb/xkb_keyboard_layout_engine.cc
  • M ui/events/ozone/layout/xkb/xkb_keyboard_layout_engine_unittest.cc
Change size: M
Delta: 5 files changed, 143 insertions(+), 5 deletions(-)
Branch: refs/branch-heads/7778
Submit Requirements:
Open in Gerrit
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: merged
Gerrit-Project: chromium/src
Gerrit-Branch: refs/branch-heads/7778
Gerrit-Change-Id: I22864f9deb130a697da2c768a76fa63c8d9429fe
Gerrit-Change-Number: 7784454
open
diffy
satisfied_requirement
Reply all
Reply to author
Forward
0 new messages