From eaa26399e03913d2cfde52fae71a47f5df1eaa8d Mon Sep 17 00:00:00 2001 From: rlaphoenix Date: Wed, 28 Sep 2022 07:54:09 +0100 Subject: [PATCH] Cdm: Reduce maximum concurrent sessions to 16 It seems 16 is the more common limit on moderm OEM Crypto API systems (at least L1). It's also a more reasonable limit. This also encourages people to .close() their session more. It also makes it quicker to notice if a codebase is forgetting to do a .close() call somewhere as you will reach the limit faster and easier now. In normal use cases, a limit of 16 sessions will not be a problem as long as the sessions are being closed correctly. --- pywidevine/cdm.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pywidevine/cdm.py b/pywidevine/cdm.py index 065e95c..c37112a 100644 --- a/pywidevine/cdm.py +++ b/pywidevine/cdm.py @@ -62,7 +62,7 @@ class Cdm: root_cert = DrmCertificate() root_cert.ParseFromString(root_signed_cert.drm_certificate) - MAX_NUM_OF_SESSIONS = 50 # most common limit + MAX_NUM_OF_SESSIONS = 16 def __init__( self,