This only affects the arguments for this override. Chromium changes: https://chromium.googlesource.com/chromium/src/+/8ff5b0d783a0415593ecf400b2c829463289d25a commit 8ff5b0d783a0415593ecf400b2c829463289d25a Author: Elly <ellyjones@chromium.org> Date: Tue Jun 24 17:52:27 2025 -0700 crx_file: migrate to crypto/keypair and crypto/sign The new APIs are a bit more ergonomic, have fewer failure cases, and are not deprecated. This change also fixes some unsafe buffer usage (of argv) in crx3_build_action. Bug: 372283556 Change-Id: I8220ff7499385001f7ba83e8eb638ee7dd987541 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6663109 Commit-Queue: Elly FJ <ellyjones@chromium.org> Auto-Submit: Elly FJ <ellyjones@chromium.org> Reviewed-by: Devlin Cronin <rdevlin.cronin@chromium.org> Cr-Commit-Position: refs/heads/main@{#1478295}
14 lines
659 B
Diff
14 lines
659 B
Diff
diff --git a/components/crx_file/crx_creator.cc b/components/crx_file/crx_creator.cc
|
|
index cb90c22f3566a581946553a3cf07b6ad7e90bf87..db558ae734723ce6705c4ed328b01c0a7ca0d968 100644
|
|
--- a/components/crx_file/crx_creator.cc
|
|
+++ b/components/crx_file/crx_creator.cc
|
|
@@ -70,7 +70,7 @@ CreatorResult SignArchiveAndCreateHeader(
|
|
|
|
// Assemble SignedData section.
|
|
SignedData signed_header_data;
|
|
- signed_header_data.set_crx_id(GetCrxId(public_key));
|
|
+ signed_header_data.set_crx_id(GetCrxId_BraveImpl(public_key, header));
|
|
const std::string signed_header_data_str =
|
|
signed_header_data.SerializeAsString();
|
|
const auto signed_header_size_octets =
|