Commit Graph

1732 Commits

Author SHA1 Message Date
Rahul Frias
e5865ae3bc Do not convert the protection scheme to network byte order [DO NOT MERGE ANYWHERE]
am: 13a4fe8e4c  -s ours

Change-Id: I44db5d8401ba1f11624bc4b42fe7ded3d5586cb1
2016-08-22 21:50:02 +00:00
Rahul Frias
13a4fe8e4c Do not convert the protection scheme to network byte order [DO NOT MERGE ANYWHERE]
[ Merge of http://go/wvgerrit/19901 ]

Protections schemes are specified using a 4CC code {"cbc1", "cbcs",
"cenc", "cens"}. A host to network conversion was performed when the
PSSH was created and inserted into the license request. A reverse
conversion was performed when the code was extracted from the
license response.

These conversions are problematic if the PSSH is created externally and
passed into mediaDrm. To address this, the conversions have been removed
and allow protobuf to handle byte ordering. For backward compatibility
we allow codes in either ordering.

b/30713238

Change-Id: I721b375e446526e544856759cada76a1fa6c7be5
2016-08-22 20:49:24 +00:00
Rahul Frias
6cd9b50064 Merge \"Upgrade TLS version in integration tests\" into nyc-dev
am: 1ebaf6ad82

Change-Id: Ibcfe029d50fefeb6e00ca6b5ac2d704838a51604
2016-06-20 20:02:42 +00:00
Rahul Frias
1ebaf6ad82 Merge "Upgrade TLS version in integration tests" into nyc-dev 2016-06-20 19:55:27 +00:00
Rahul Frias
b552a133c2 Upgrade TLS version in integration tests
[ Merge of http://go/wvgerrit/18295 ]

SSL v3 is being disabled across Google infrastructure. Networking
code in integration tests used SSL v3 during HTTPS protocol negotiation.
Once this is disabled, it will cause integration test failures at
staging and UAT license servers.

With this change the client will use TLS 1.2. Insecure cipher suites
(TLS_RSA_WITH_RC4_128_MD5, TLS_RSA_WITH_RC4_128_SHA,
 TLS_ECDHE_ECDSA_WITH_RC4_128_SHA, TLS_ECDHE_RSA_WITH_RC4_128_SHA)
have been disabled as well.

b/29356581

Change-Id: I98a04d345fe83c48132b9d79986a21cc84827dc8
2016-06-17 22:28:56 -07:00
Jeff Tinker
98d0e9b614 Merge \"Rework WidevineSamplePlayer to use fragments\" into nyc-dev
am: 02d8671882

Change-Id: I8b2d18fb234b697712c33f51fa3be8b45b0f1e19
2016-06-17 21:37:47 +00:00
Jeff Tinker
02d8671882 Merge "Rework WidevineSamplePlayer to use fragments" into nyc-dev 2016-06-17 21:24:47 +00:00
Jeff Tinker
3c5ca90637 Update the expected android build version for NYC release
am: 2bd6131162

Change-Id: Ia5006b4f1c54896dfb3c0753d6139dc10f20c208
2016-06-17 18:19:57 +00:00
Jeff Tinker
2bd6131162 Update the expected android build version for NYC release
bug: 29270537
Change-Id: I2f85dd95e1a2b9eaa79299441b48716599211eea
2016-06-17 00:39:33 +00:00
Jeff Tinker
9f735d298a Rework WidevineSamplePlayer to use fragments
Previously it used TabActivity which has been deprecated
and no longer works.

bug: 29045104

Change-Id: I207f0208b6dba47adfa0ffe7485800d1561af617
2016-06-13 11:27:03 -07:00
Rahul Frias
1f7aa66d54 Report OEMCrypto security patch level
am: e33895f5de

* commit 'e33895f5deccdbb15d3ba028190e07864f326ab2':
  Report OEMCrypto security patch level

Change-Id: I377b697ebae8beee5679d71cf9233e15d95d65c3
2016-05-24 00:38:59 +00:00
Rahul Frias
e33895f5de Report OEMCrypto security patch level
[ Merge of http://go/wvgerrit/17959 ]

This will allow the license server to base licensing decisions on the
devices security module revision.

b/28882058

Change-Id: I574e7686bb305397946d2bfaff504cfae242e628
2016-05-23 10:50:07 -07:00
Fred Gylys-Colwell
b65344398c Remove pattern offset tests
am: 0b2223018d

* commit '0b2223018dd2901e1d5e68d2a5f238bba9ba79ed':
  Remove pattern offset tests

Change-Id: I1a9709a7dd092d13508fca20f871ab71a43a5af1
2016-05-19 21:53:04 +00:00
Fred Gylys-Colwell
0b2223018d Remove pattern offset tests
Merge from widevine repo of http://go/wvgerrit/17877

The final spec for CENC says that partial encrypted blocks are only
allowed for "cenc" mode, and not for "cens" mode.  Also, the
pattern_offset must be 0 for all calls to OEMCrypto_DecryptCENC.

Review doc changes http://go/wvdelta11 and http://go/oemcrypto

Notice that this CL affects documentation and test code only.  No
production code is affected.

b/28718266
b/28817376

Change-Id: I1d6a38a763a2b693ea0c26bc909e1c721b8facaf
2016-05-17 15:15:18 -07:00
Fred Gylys-Colwell
5a16e85da5 Merge "Push android unit tests to /data" into nyc-dev
am: 1f204c5cfa

* commit '1f204c5cfa1bf3aee53e9f284a92031c47c31cbd':
  Push android unit tests to /data

Change-Id: Idc020cf557649469bc4b4ddc66963c47211613ea
2016-05-03 15:51:32 +00:00
Fred Gylys-Colwell
1f204c5cfa Merge "Push android unit tests to /data" into nyc-dev 2016-05-03 15:48:04 +00:00
Fred Gylys-Colwell
a4ebdf80b5 Push android unit tests to /data
Merge of http://go/wvgerrit/17652

This CL changes the android/build_and_run_all_unit_tests.sh script to
push exectuables to /data if it can't push to /system/bin.

The script run_all_unit_tests.sh will check if test files are in /data
and run those first, if not, it runs the files in /system/bin.

This change is needed to test unit tests on devices where /system/bin
is a read only file system, even after running "adb remount".

This CL will help verify, but will not fix
b/27678092

Change-Id: Ia0fe8c2e68722f58c2626f22113abdbdc8c5e408
2016-05-02 15:16:14 -07:00
John "Juce" Bruce
3cd400644b Don\'t Validate Crypto Mode for Unencrypted Decrypt
am: 6d2822f

* commit '6d2822f6c615e957de2ad0af3fb1009e79a46fcc':
  Don't Validate Crypto Mode for Unencrypted Decrypt

Change-Id: Ie0b8a72fa42e373938c489572519135f2b57c25c
2016-04-27 23:46:53 +00:00
John "Juce" Bruce
6d2822f6c6 Don't Validate Crypto Mode for Unencrypted Decrypt
(This is a merge of http://go/wvgerrit/17661)

Because Widevine core always has a decrypt mode set, even on unencrypted
calls where we potentially do not know the crypto mode, it will reject
unencrypted decrypt calls for sessions that have been flagged as
AES-CBC, since the crypto mode defaults to AES-CTR.

The fix is to not validate the crypto mode on unencrypted decrypt calls,
as the data won't be getting decrypted anyway.

Bug: 28423928
Change-Id: If848834dd498ca96983a2b69d448b8d81d50e0a4
2016-04-27 12:20:18 -07:00
Rahul Frias
850cf0e08e Merge "Modify QueryStatusL3 test to allow for L3 devices" into nyc-dev
am: ecfd61e

* commit 'ecfd61e803f64e42d32eb000daf136e510f18a2e':
  Modify QueryStatusL3 test to allow for L3 devices

Change-Id: I38e8d9dba45baf91a3ce4886308ad9d01a3bd6e8
2016-04-15 23:22:19 +00:00
Rahul Frias
ecfd61e803 Merge "Modify QueryStatusL3 test to allow for L3 devices" into nyc-dev 2016-04-15 22:51:56 +00:00
Kyle Alexander
e495647635 Merge "Add display and logging for validating HDCP levels." into nyc-dev
am: 53ad22d

* commit '53ad22d970aee93479152aaa5ef294b9ea072025':
  Add display and logging for validating HDCP levels.

Change-Id: I3a547561afc91538837408041216eb7ae1865db3
2016-04-15 00:36:02 +00:00
Kyle Alexander
53ad22d970 Merge "Add display and logging for validating HDCP levels." into nyc-dev 2016-04-15 00:30:03 +00:00
Fred Gylys-Colwell
2049c6ca3b Specify generic encryption buffer sizes
am: 2717f29

* commit '2717f297079494068c7078ef4fc5723f27457070':
  Specify generic encryption buffer sizes

Change-Id: I61d6f64017bec8fdb453263841ea0c3f71f46ac7
2016-04-15 00:12:25 +00:00
Fred Gylys-Colwell
2717f29707 Specify generic encryption buffer sizes
Merge from widevine repo of http://go/wvgerrit/17463

This CL updates the unit tests to verify that
OEMCrypto_Generic_Encrypt and OEMCrypto_Generic_Decrypt can handle a
buffer size of at least 100k.  It also adds code to the
oemcrypto_dynamic_adapter so that buffer sizes that are larger than
100k are broken into chunks of 100k.

All Nexus devices targeted for N pass these tests.

b/27040752

Change-Id: Iaf5c65d2f0b69e60f03cc99732d1ecab60658049
2016-04-14 15:02:59 -07:00
Rahul Frias
c535436bab Modify QueryStatusL3 test to allow for L3 devices
[ Merge of http://go/wvgerrit/17501 ]

Widevine request_license_tests were written with the expectation that
devices would by default be enabled for security level 1 and fallback
to level 3 if needed. Some devices such as seed are L3 only and results
in failures in the QueryStatusL3 test. This CL corrects the test's
expectation.

b/26902214

Change-Id: Ie84554337cd3716d120f67e592888af0cde719c7
2016-04-13 10:30:07 -07:00
Kyle Alexander
27c8c3f150 Add display and logging for validating HDCP levels.
This adds display and logging values to help with debugging b/28024277.

Matching Widevine cl: go/wvgerrit/17446

Bug: 28024277
Change-Id: Ib25ebf37f885b60ae4ab41851df4af4e75b5aace
2016-04-11 09:48:27 -07:00
John "Juce" Bruce
0b8b2530f3 Merge "Restore WVCryptoPlugin Unit Tests" into nyc-dev
am: 91059f1

* commit '91059f1d812e1946d71dd4ce0418609e871af27a':
  Restore WVCryptoPlugin Unit Tests

Change-Id: I064f56545a2aed2225e1fa0c1411dc7baac71c08
2016-04-09 01:00:53 +00:00
John Bruce
91059f1d81 Merge "Restore WVCryptoPlugin Unit Tests" into nyc-dev 2016-04-08 18:22:17 +00:00
Jeff Tinker
ec0c0ff78a Merge "Fix StringIndexOutOfBoundsException in MediaDrmAPITest" into nyc-dev
am: 5dd0d2a

* commit '5dd0d2a5ceb7f72d21ac5e224c5b57c0b438c1f2':
  Fix StringIndexOutOfBoundsException in MediaDrmAPITest

Change-Id: I27ebda8cc4642eb0d44edfd0c31358aa052c1d63
2016-04-08 18:16:35 +00:00
Jeff Tinker
5dd0d2a5ce Merge "Fix StringIndexOutOfBoundsException in MediaDrmAPITest" into nyc-dev 2016-04-08 18:05:12 +00:00
Rahul Frias
089eb0f90c Load keys before sending OnKeyStatusChange notifications
am: 7cb83eb

* commit '7cb83eb02e92abb41349acf1e89d0d31a35f531f':
  Load keys before sending OnKeyStatusChange notifications

Change-Id: I851689274b2edc60edab4775aa490fa16d72a0c7
2016-04-08 17:38:01 +00:00
Jeff Tinker
06c45414b5 Fix StringIndexOutOfBoundsException in MediaDrmAPITest
The AES test was useful to verify the framework API changes, but
the decrypt test was not correct.  Removing it for now.

Merge of go/wvgerrit/17472

bug: 27973491
Change-Id: I942fda22072636ea7ce56b992e7d3a60077c0d27
2016-04-08 16:51:22 +00:00
Rahul Frias
7cb83eb02e Load keys before sending OnKeyStatusChange notifications
[ merge of http://go/wvgerrit/17454 ]

When processing a license or renewal, calls to Set/UpdateLicense
update the policy information. A side effect was introduced whereby
updating the policy may cause (expiration, session key state)
notifications to be sent to the listener. Due to the ordering,
the notifications would be sent before the keys were loaded/refreshed,
which caused issues when the notifications were immediately acted upon.
This has now been corrected.

b/27842970

Change-Id: Id81a71ff48edfa9ca0baafc43267995d5a3e80a6
2016-04-07 15:58:45 -07:00
John "Juce" Bruce
a73dc419fd Restore WVCryptoPlugin Unit Tests
(This is a merge of http://go/wvgerrit/17452)

When the Crypto API was updated, several unit tests were removed, as
they did not work with the new API. This patch restores those tests.

Note that this does not add NEW tests for the new functionality
enabled by the API changes. Those will be coming in a separate patch.
This patch merely restores the previous tests and their functionality.

Bug: 26901698
Change-Id: I3315408804bdc6871b58c3620e482bf6d61fe03b
2016-04-07 12:15:12 -07:00
Rahul Frias
af422336a6 Return ERROR_INSUFFICIENT_OUTPUT_PROTECTION when appropriate
am: b76eccf

* commit 'b76eccf32d291f7fce03ba82f56ede28b87e3a4e':
  Return ERROR_INSUFFICIENT_OUTPUT_PROTECTION when appropriate

Change-Id: Ifdb9428997757a1114199244699bc15a2eccf14d
2016-04-06 17:26:38 +00:00
Rahul Frias
b76eccf32d Return ERROR_INSUFFICIENT_OUTPUT_PROTECTION when appropriate
[ Merge from http://go/wvgerrit/17429 ]

This is to correct for when HDCP validation failure at the OEMCrypto level
ends up returning a CryptoException through MediaCrypto with error code
Unknown Error (-2998).

b/27849488

Change-Id: Ib3a9b1c3a17b9323f3d079a678cf88dedeec3b4b
2016-04-05 16:44:31 -07:00
John "Juce" Bruce
ff896a0a9b Merge "Move Widevine Unit Tests to Standard gMock" into nyc-dev
am: 19c36a6

* commit '19c36a64d61e180a1fb08b4694b67307af58c089':
  Move Widevine Unit Tests to Standard gMock

Change-Id: Ib115baa314284c94b323632a2f491fc13f0de27b
2016-04-04 23:23:17 +00:00
John Bruce
19c36a64d6 Merge "Move Widevine Unit Tests to Standard gMock" into nyc-dev 2016-04-04 23:13:23 +00:00
Jeff Tinker
a118a8c541 Merge "Fix offline playback using widevine sample player" into nyc-dev
am: 971d94e

* commit '971d94e410362192c932c41b9c296c6ead36b88c':
  Fix offline playback using widevine sample player

Change-Id: Ia2e35560e57b8e64215b1192e0c692e98ac4b09b
2016-04-04 21:40:48 +00:00
Jeff Tinker
971d94e410 Merge "Fix offline playback using widevine sample player" into nyc-dev 2016-04-04 21:27:19 +00:00
Jeff Tinker
ad8e58f6f7 Fix offline playback using widevine sample player
Previously the sample player relied on implementation
details of FileDescriptor.toString() to get the low
level int file descriptor, which changed in N release.
This updates the sample player to use a more reliable
method.

bug: 26422357
Change-Id: I73cc759542994a352eae646a3872ee1125445739
2016-04-04 12:13:53 -07:00
Fred Gylys-Colwell
aefda18f5b Update Mock Android.mk
am: c42f1e9

* commit 'c42f1e93729d0f949115d3e8eeb6fe13723f5aa0':
  Update Mock Android.mk

Change-Id: I33113993d5de38becf9bd8611709a5cd9faa2006
2016-04-01 23:32:49 +00:00
Fred Gylys-Colwell
c42f1e9372 Update Mock Android.mk
Merge from widevine repo of http://go/wvgerrit/17393

This CL changes the installation directory for oemcrypto reference
code. In particular, it uses the new build rules to specifies the library
should go in /vendor/lib instead of /system/lib or /vendor/lib64.

This library should not be used on a production device.

b/26425759

Change-Id: I3a58d0d1ddeb5601b0f2f97cc87c2d4a2a41f0d8
2016-04-01 15:57:56 -07:00
John "Juce" Bruce
0555a7c9de Move Widevine Unit Tests to Standard gMock
(In addition to being a merge of http://go/wvgerrit/17394, this commit
updates tests/Android.mk to no longer try to build the private gMock.)

Previously, we maintained our own version of gMock (named libwvgmock
to avoid naming conflicts) and depended on it for our unit tests. Now
that Android has gMock in the externals/ directory and vaage@ has
eliminated the need for us to customize gMock's arity, we can stop
depending on or maintaining our own copy.

Bug: 26907374
Change-Id: Ia01d6c02e2f28a642480d7ef178b9bc3dde6d306
2016-04-01 14:49:43 -07:00
Fred Gylys-Colwell
de8574beae Level 3 OEMCrypto with symbols removed
am: 0e929c2

* commit '0e929c2ec75be460b31f54cbc246be04c57ab9c5':
  Level 3 OEMCrypto with symbols removed

Change-Id: I64b8b58dfa7657bc031492ee17a057673cec048b
2016-03-31 02:10:59 +00:00
Fred Gylys-Colwell
0e929c2ec7 Level 3 OEMCrypto with symbols removed
Merge from widevine side of http://go/wvgerrit/17355

The previous build of this library had some filenames and linker
symbols left in that should not be leaked.  This build removes them.

Library versions:
level3/mips/libwvlevel3.a  Level3 Library 4465 Mar 30 2016 13:29:24
level3/arm/libwvlevel3.a  Level3 Library 4445 Mar 30 2016 13:23:54
level3/x86/libwvlevel3.a  Level3 Library 4464 Mar 30 2016 13:26:29

b/27831262

Change-Id: I7801c6f953af338f5e80e95aa27098ae4c7a6c93
2016-03-30 15:56:15 -07:00
Rahul Frias
e9f15858f0 Address failures in Widevine CDM extended duration tests
[ Merge of http://go/wvgerrit/17250 ]

This appears to be the result of a bad merge.

b/27657766

Change-Id: I890162595623f1c13973cedc17eb611fac7e5535
2016-03-24 13:38:00 -07:00
Fred Gylys-Colwell
9bf25bc67f Turn off mips64 library (second attempt)
Merge from widevine repo of http://go/wvgerrit/17223/3

This time I mean it.

b/27786231

Change-Id: I8d4d3acf7470c8b5878b90774f97b5fce1011ccf
2016-03-23 20:52:12 -07:00
Fred Gylys-Colwell
19cd8db256 Turn off mips64 library
Merge from widevine repo of http://go/wvgerrit/17223

This CL turns off the mips64 bit build for widevine.  This is because
the level 3 mips32 library does not build and run correctly for a 64
bit target.

b/27786231

Change-Id: I3388d9ebc80ae3cf477a8a54adfd1f22dfac4dda
2016-03-23 07:54:31 -07:00