diff options
| author | 2020-05-19 23:18:03 +0900 | |
|---|---|---|
| committer | 2020-05-19 23:19:59 +0900 | |
| commit | a1887f326c9d46f34beea466c231e899a1163351 (patch) | |
| tree | dcef79022ac4a37578d1c110d814413801677dac /tools/releasetools/test_apex_utils.py | |
| parent | ca6e50fa2c554f7c7e491448bd13e3d7d7b019fa (diff) | |
sign_apex.py respects hash algorithm
When sign_apex.py re-signs an apex, the hash algorithm is extracted and
it is used when re-packaging the file.
Bug: 155771970
Test: releasetools_py3_test doesn't regress
(some tests were failed, but they failed even without this change:
test_ApexApkSigner_apkKeyNotPresent (test_apex_utils.ApexUtilsTest) ... ERROR
test_ApexApkSigner_noApkPresent (test_apex_utils.ApexUtilsTest) ... ERROR
test_ApexApkSigner_noAssetDir (test_apex_utils.ApexUtilsTest) ... ERROR
test_ApexApkSigner_signApk (test_apex_utils.ApexUtilsTest) ... ERROR)
Change-Id: I996a33e0208d3dd3a04a31b8dafb27ef995297f2
Diffstat (limited to 'tools/releasetools/test_apex_utils.py')
| -rw-r--r-- | tools/releasetools/test_apex_utils.py | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/tools/releasetools/test_apex_utils.py b/tools/releasetools/test_apex_utils.py index e19bc90b32..7b4a4b0c71 100644 --- a/tools/releasetools/test_apex_utils.py +++ b/tools/releasetools/test_apex_utils.py @@ -50,11 +50,12 @@ class ApexUtilsTest(test_utils.ReleaseToolsTestCase): payload_file = self._GetTestPayload() apex_utils.SignApexPayload( 'avbtool', payload_file, self.payload_key, 'testkey', 'SHA256_RSA2048', - self.SALT, no_hashtree=True) + self.SALT, 'sha256', no_hashtree=True) payload_info = apex_utils.ParseApexPayloadInfo('avbtool', payload_file) self.assertEqual('SHA256_RSA2048', payload_info['Algorithm']) self.assertEqual(self.SALT, payload_info['Salt']) self.assertEqual('testkey', payload_info['apex.key']) + self.assertEqual('sha256', payload_info['Hash Algorithm']) self.assertEqual('0 bytes', payload_info['Tree Size']) @test_utils.SkipIfExternalToolsUnavailable() @@ -62,7 +63,7 @@ class ApexUtilsTest(test_utils.ReleaseToolsTestCase): payload_file = self._GetTestPayload() apex_utils.SignApexPayload( 'avbtool', payload_file, self.payload_key, 'testkey', 'SHA256_RSA2048', - self.SALT, no_hashtree=True) + self.SALT, 'sha256', no_hashtree=True) apex_utils.VerifyApexPayload( 'avbtool', payload_file, self.payload_key, True) @@ -71,7 +72,7 @@ class ApexUtilsTest(test_utils.ReleaseToolsTestCase): payload_file = self._GetTestPayload() apex_utils.SignApexPayload( 'avbtool', payload_file, self.payload_key, 'testkey', 'SHA256_RSA2048', - self.SALT, no_hashtree=False) + self.SALT, 'sha256', no_hashtree=False) apex_utils.VerifyApexPayload('avbtool', payload_file, self.payload_key) payload_info = apex_utils.ParseApexPayloadInfo('avbtool', payload_file) self.assertEqual('4096 bytes', payload_info['Tree Size']) @@ -81,7 +82,7 @@ class ApexUtilsTest(test_utils.ReleaseToolsTestCase): payload_file = self._GetTestPayload() apex_utils.SignApexPayload( 'avbtool', payload_file, self.payload_key, 'testkey', 'SHA256_RSA2048', - self.SALT, no_hashtree=True) + self.SALT, 'sha256', no_hashtree=True) apex_utils.VerifyApexPayload('avbtool', payload_file, self.payload_key, no_hashtree=True) payload_info = apex_utils.ParseApexPayloadInfo('avbtool', payload_file) @@ -98,7 +99,7 @@ class ApexUtilsTest(test_utils.ReleaseToolsTestCase): 'avbtool', payload_file, self.payload_key, - 'testkey', 'SHA256_RSA2048', self.SALT, + 'testkey', 'SHA256_RSA2048', self.SALT, 'sha256', True, payload_signer_args) apex_utils.VerifyApexPayload( @@ -115,6 +116,7 @@ class ApexUtilsTest(test_utils.ReleaseToolsTestCase): 'testkey', 'SHA256_RSA2048', self.SALT, + 'sha256', no_hashtree=True) @test_utils.SkipIfExternalToolsUnavailable() @@ -122,7 +124,7 @@ class ApexUtilsTest(test_utils.ReleaseToolsTestCase): payload_file = self._GetTestPayload() apex_utils.SignApexPayload( 'avbtool', payload_file, self.payload_key, 'testkey', 'SHA256_RSA2048', - self.SALT, True) + self.SALT, 'sha256', True) apex_utils.VerifyApexPayload( 'avbtool', payload_file, self.payload_key, True) self.assertRaises( |