Merge pull request #1695 from CastagnaIT/next_cleanups

CastagnaIT · web-flow · commit 952fb0506091 · 2024-10-15T12:39:46.000+02:00
[Decrypters][cleanups] Const init data, session id as string
diff --git a/src/Session.cpp b/src/Session.cpp
@@ -82,7 +82,7 @@ void SESSION::CSession::DisposeSampleDecrypter()
   {
     for (auto& cdmSession : m_cdmSessions)
     {
-      cdmSession.m_cdmSessionStr = nullptr;
+      cdmSession.m_sessionId.clear();
       cdmSession.m_cencSingleSampleDecrypter = nullptr;
     }
   }
@@ -293,8 +293,8 @@ bool SESSION::CSession::PreInitializeDRM(std::string& challengeB64,
                           m_decrypter->CreateSingleSampleDecrypter(initData, decKid, "", true,
                                                                    CryptoMode::AES_CTR)) != nullptr)
   {
-    session.m_cdmSessionStr = session.m_cencSingleSampleDecrypter->GetSessionId();
-    sessionId = session.m_cdmSessionStr;
+    session.m_sessionId = session.m_cencSingleSampleDecrypter->GetSessionId();
+    sessionId = session.m_sessionId;
     challengeB64 = m_decrypter->GetChallengeB64Data(session.m_cencSingleSampleDecrypter);
   }
   else
@@ -466,7 +466,7 @@ bool SESSION::CSession::InitializeDRM(bool addDefaultKID /* = false */)
         m_decrypter->GetCapabilities(session.m_cencSingleSampleDecrypter, defaultKid,
                                      sessionPsshset.media_, session.m_decrypterCaps);
 
-        session.m_cdmSessionStr = session.m_cencSingleSampleDecrypter->GetSessionId();
+        session.m_sessionId = session.m_cencSingleSampleDecrypter->GetSessionId();
 
         if (session.m_decrypterCaps.flags & DRM::DecrypterCapabilites::SSD_INVALID)
         {
@@ -927,14 +927,14 @@ bool SESSION::CSession::IsCDMSessionSecurePath(size_t index)
           DRM::DecrypterCapabilites::SSD_SECURE_PATH) != 0;
 }
 
-const char* SESSION::CSession::GetCDMSession(unsigned int index)
+std::string SESSION::CSession::GetCDMSession(unsigned int index)
 {
   if (index >= m_cdmSessions.size())
   {
     LOG::LogF(LOGERROR, "No CDM session at index %u", index);
-    return nullptr;
+    return {};
   }
-  return m_cdmSessions[index].m_cdmSessionStr;
+  return m_cdmSessions[index].m_sessionId;
 }
 
 uint64_t SESSION::CSession::PTSToElapsed(uint64_t pts)
@@ -1294,7 +1294,7 @@ std::shared_ptr<Adaptive_CencSingleSampleDecrypter> SESSION::CSession::GetSingle
   for (std::vector<CCdmSession>::iterator b(m_cdmSessions.begin() + 1), e(m_cdmSessions.end());
        b != e; ++b)
   {
-    if (b->m_cdmSessionStr && sessionId == b->m_cdmSessionStr)
+    if (sessionId == b->m_sessionId)
       return b->m_cencSingleSampleDecrypter;
   }
   return nullptr;
diff --git a/src/Session.h b/src/Session.h
@@ -125,7 +125,7 @@ class ATTR_DLL_LOCAL CSession : public adaptive::AdaptiveStreamObserver
    *  \param index The index (psshSet number) of the cdm session
    *  \return The session string
    */
-  const char* GetCDMSession(unsigned int index);
+  std::string GetCDMSession(unsigned int index);
 
   /*! \brief Get the media type mask
    *  \return The media type mask
@@ -353,7 +353,7 @@ class ATTR_DLL_LOCAL CSession : public adaptive::AdaptiveStreamObserver
   {
     DRM::DecrypterCapabilites m_decrypterCaps;
     std::shared_ptr<Adaptive_CencSingleSampleDecrypter> m_cencSingleSampleDecrypter;
-    const char* m_cdmSessionStr{nullptr};
+    std::string m_sessionId;
   };
   std::vector<CCdmSession> m_cdmSessions;
 
diff --git a/src/common/AdaptiveDecrypter.h b/src/common/AdaptiveDecrypter.h
@@ -11,6 +11,7 @@
 #include "utils/CryptoUtils.h"
 
 #include <stdexcept>
+#include <string>
 #include <string_view>
 #include <vector>
 
@@ -54,5 +55,5 @@ class Adaptive_CencSingleSampleDecrypter : public AP4_CencSingleSampleDecrypter
 
   virtual AP4_UI32 AddPool() { return 0; }
   virtual void RemovePool(AP4_UI32 poolId) {}
-  virtual const char* GetSessionId() { return nullptr; }
+  virtual std::string GetSessionId() { return {}; }
 };
diff --git a/src/decrypters/IDecrypter.h b/src/decrypters/IDecrypter.h
@@ -125,7 +125,7 @@ class IDecrypter
    * \return The single sample decrypter if successfully created
    */
   virtual std::shared_ptr<Adaptive_CencSingleSampleDecrypter> CreateSingleSampleDecrypter(
-      std::vector<uint8_t>& initData,
+      const std::vector<uint8_t>& initData,
       const std::vector<uint8_t>& defaultKeyId,
       std::string_view licenseUrl,
       bool skipSessionMessage,
diff --git a/src/decrypters/clearkey/ClearKeyDecrypter.cpp b/src/decrypters/clearkey/ClearKeyDecrypter.cpp
@@ -31,7 +31,7 @@ bool CClearKeyDecrypter::OpenDRMSystem(const DRM::Config& config)
 }
 
 std::shared_ptr<Adaptive_CencSingleSampleDecrypter> CClearKeyDecrypter::CreateSingleSampleDecrypter(
-    std::vector<uint8_t>& initData,
+    const std::vector<uint8_t>& initData,
     const std::vector<uint8_t>& defaultkeyid,
     std::string_view licenseUrl,
     bool skipSessionMessage,
diff --git a/src/decrypters/clearkey/ClearKeyDecrypter.h b/src/decrypters/clearkey/ClearKeyDecrypter.h
@@ -19,7 +19,7 @@ class CClearKeyDecrypter : public IDecrypter
   virtual std::vector<std::string_view> SelectKeySystems(std::string_view keySystem) override;
   virtual bool OpenDRMSystem(const DRM::Config& config) override;
   virtual std::shared_ptr<Adaptive_CencSingleSampleDecrypter> CreateSingleSampleDecrypter(
-      std::vector<uint8_t>& initData,
+      const std::vector<uint8_t>& initData,
       const std::vector<uint8_t>& defaultkeyid,
       std::string_view licenseUrl,
       bool skipSessionMessage,
diff --git a/src/decrypters/widevine/WVCencSingleSampleDecrypter.cpp b/src/decrypters/widevine/WVCencSingleSampleDecrypter.cpp
@@ -42,7 +42,7 @@ void CWVCencSingleSampleDecrypter::SetSession(const std::string sessionId,
 
 CWVCencSingleSampleDecrypter::CWVCencSingleSampleDecrypter(
     IWVCdmAdapter<media::CdmAdapter>* cdmAdapter,
-    std::vector<uint8_t>& pssh,
+    const std::vector<uint8_t>& pssh,
     const std::vector<uint8_t>& defaultKeyId,
     bool skipSessionMessage,
     CryptoMode cryptoMode)
@@ -202,9 +202,9 @@ void CWVCencSingleSampleDecrypter::GetCapabilities(const std::vector<uint8_t>& k
   }
 }
 
-const char* CWVCencSingleSampleDecrypter::GetSessionId()
+std::string CWVCencSingleSampleDecrypter::GetSessionId()
 {
-  return m_strSession.empty() ? nullptr : m_strSession.c_str();
+  return m_strSession;
 }
 
 void CWVCencSingleSampleDecrypter::CloseSessionId()
diff --git a/src/decrypters/widevine/WVCencSingleSampleDecrypter.h b/src/decrypters/widevine/WVCencSingleSampleDecrypter.h
@@ -30,7 +30,7 @@ class ATTR_DLL_LOCAL CWVCencSingleSampleDecrypter : public Adaptive_CencSingleSa
 {
 public:
   CWVCencSingleSampleDecrypter(IWVCdmAdapter<media::CdmAdapter>* cdmAdapter,
-                               std::vector<uint8_t>& pssh,
+                               const std::vector<uint8_t>& pssh,
                                const std::vector<uint8_t>& defaultKeyId,
                                bool skipSessionMessage,
                                CryptoMode cryptoMode);
@@ -39,7 +39,7 @@ class ATTR_DLL_LOCAL CWVCencSingleSampleDecrypter : public Adaptive_CencSingleSa
   void GetCapabilities(const std::vector<uint8_t>& keyId,
                        uint32_t media,
                        DecrypterCapabilites& caps);
-  virtual const char* GetSessionId() override;
+  virtual std::string GetSessionId() override;
   void CloseSessionId();
   AP4_DataBuffer GetChallengeData();
 
diff --git a/src/decrypters/widevine/WVDecrypter.cpp b/src/decrypters/widevine/WVDecrypter.cpp
@@ -96,15 +96,15 @@ bool CWVDecrypter::OpenDRMSystem(const DRM::Config& config)
 }
 
 std::shared_ptr<Adaptive_CencSingleSampleDecrypter> CWVDecrypter::CreateSingleSampleDecrypter(
-    std::vector<uint8_t>& initData,
+    const std::vector<uint8_t>& initData,
     const std::vector<uint8_t>& defaultKeyId,
     std::string_view licenseUrl,
     bool skipSessionMessage,
     CryptoMode cryptoMode)
 {
   auto decrypter = std::make_shared<CWVCencSingleSampleDecrypter>(
       m_WVCdmAdapter.get(), initData, defaultKeyId, skipSessionMessage, cryptoMode);
-  if (!decrypter->GetSessionId())
+  if (decrypter->GetSessionId().empty())
   {
     return nullptr;
   }
diff --git a/src/decrypters/widevine/WVDecrypter.h b/src/decrypters/widevine/WVDecrypter.h
@@ -22,7 +22,7 @@ class ATTR_DLL_LOCAL CWVDecrypter : public DRM::IDecrypter
   virtual std::vector<std::string_view> SelectKeySystems(std::string_view keySystem) override;
   virtual bool OpenDRMSystem(const DRM::Config& config) override;
   virtual std::shared_ptr<Adaptive_CencSingleSampleDecrypter> CreateSingleSampleDecrypter(
-      std::vector<uint8_t>& initData,
+      const std::vector<uint8_t>& initData,
       const std::vector<uint8_t>& defaultKeyId,
       std::string_view licenseUrl,
       bool skipSessionMessage,
diff --git a/src/decrypters/widevineandroid/WVCencSingleSampleDecrypter.cpp b/src/decrypters/widevineandroid/WVCencSingleSampleDecrypter.cpp
@@ -28,7 +28,7 @@ using namespace UTILS;
 
 CWVCencSingleSampleDecrypterA::CWVCencSingleSampleDecrypterA(
     IWVCdmAdapter<jni::CJNIMediaDrm>* cdmAdapter,
-    std::vector<uint8_t>& pssh,
+    const std::vector<uint8_t>& pssh,
     const std::vector<uint8_t>& defaultKeyId)
   : m_cdmAdapter(cdmAdapter),
     m_pssh(pssh),
@@ -159,9 +159,9 @@ CWVCencSingleSampleDecrypterA::~CWVCencSingleSampleDecrypterA()
   m_cdmAdapter->DetachObserver(this);
 }
 
-const char* CWVCencSingleSampleDecrypterA::GetSessionId()
+std::string CWVCencSingleSampleDecrypterA::GetSessionId()
 {
-  return m_sessionId.c_str();
+  return m_sessionId;
 }
 
 std::vector<uint8_t> CWVCencSingleSampleDecrypterA::GetChallengeData()
diff --git a/src/decrypters/widevineandroid/WVCencSingleSampleDecrypter.h b/src/decrypters/widevineandroid/WVCencSingleSampleDecrypter.h
@@ -25,13 +25,13 @@ class ATTR_DLL_LOCAL CWVCencSingleSampleDecrypterA : public Adaptive_CencSingleS
 {
 public:
   CWVCencSingleSampleDecrypterA(IWVCdmAdapter<jni::CJNIMediaDrm>* cdmAdapter,
-                                std::vector<uint8_t>& pssh,
+                                const std::vector<uint8_t>& pssh,
                                 const std::vector<uint8_t>& defaultKeyId);
   virtual ~CWVCencSingleSampleDecrypterA();
 
   bool StartSession(bool skipSessionMessage) { return KeyUpdateRequest(true, skipSessionMessage); };
 
-  virtual const char* GetSessionId() override;
+  virtual std::string GetSessionId() override;
   std::vector<uint8_t> GetChallengeData();
   virtual bool HasLicenseKey(const std::vector<uint8_t>& keyId);
 
diff --git a/src/decrypters/widevineandroid/WVDecrypter.cpp b/src/decrypters/widevineandroid/WVDecrypter.cpp
@@ -90,7 +90,7 @@ bool CWVDecrypterA::OpenDRMSystem(const DRM::Config& config)
 }
 
 std::shared_ptr<Adaptive_CencSingleSampleDecrypter> CWVDecrypterA::CreateSingleSampleDecrypter(
-    std::vector<uint8_t>& initData,
+    const std::vector<uint8_t>& initData,
     const std::vector<uint8_t>& defaultKeyId,
     std::string_view licenseUrl,
     bool skipSessionMessage,
@@ -99,7 +99,7 @@ std::shared_ptr<Adaptive_CencSingleSampleDecrypter> CWVDecrypterA::CreateSingleS
   std::shared_ptr<CWVCencSingleSampleDecrypterA> decrypter =
       std::make_shared<CWVCencSingleSampleDecrypterA>(m_WVCdmAdapter.get(), initData, defaultKeyId);
 
-  if (!(*decrypter->GetSessionId() && decrypter->StartSession(skipSessionMessage)))
+  if (!(!decrypter->GetSessionId().empty() && decrypter->StartSession(skipSessionMessage)))
   {
     return nullptr;
   }
diff --git a/src/decrypters/widevineandroid/WVDecrypter.h b/src/decrypters/widevineandroid/WVDecrypter.h
@@ -49,7 +49,7 @@ class ATTR_DLL_LOCAL CWVDecrypterA : public DRM::IDecrypter
   virtual bool OpenDRMSystem(const DRM::Config& config) override;
 
   virtual std::shared_ptr<Adaptive_CencSingleSampleDecrypter> CreateSingleSampleDecrypter(
-      std::vector<uint8_t>& initData,
+      const std::vector<uint8_t>& initData,
       const std::vector<uint8_t>& defaultKeyId,
       std::string_view licenseUrl,
       bool skipSessionMessage,
diff --git a/src/main.cpp b/src/main.cpp
@@ -306,8 +306,7 @@ bool CInputStreamAdaptive::OpenStream(int streamid)
     const std::string keySystem = CSrvBroker::GetKodiProps().GetDrmKeySystem();
     cryptoSession.SetKeySystem(m_session->GetCryptoKeySystem(keySystem));
 
-    const char* sessionId(m_session->GetCDMSession(cdmSessionIndex));
-    cryptoSession.SetSessionId(sessionId);
+    cryptoSession.SetSessionId(m_session->GetCDMSession(cdmSessionIndex));
 
     if (m_session->GetDecrypterCaps(cdmSessionIndex).flags &
         DRM::DecrypterCapabilites::SSD_SUPPORTS_DECODING)
@@ -581,7 +580,7 @@ bool CVideoCodecAdaptive::Open(const kodi::addon::VideoCodecInitdata& initData)
   }
   m_name += ".decoder";
 
-  std::string sessionId(initData.GetCryptoSession().GetSessionId());
+  std::string sessionId = initData.GetCryptoSession().GetSessionId();
   std::shared_ptr<Adaptive_CencSingleSampleDecrypter> ssd(m_session->GetSingleSampleDecrypter(sessionId));
 
   return m_session->GetDecrypter()->OpenVideoDecoder(

Original file line number	Diff line number	Diff line change
`@@ -82,7 +82,7 @@ void SESSION::CSession::DisposeSampleDecrypter()`
`82`	`82`	`{`
`83`	`83`	`for (auto& cdmSession : m_cdmSessions)`
`84`	`84`	`{`
`85`		`- cdmSession.m_cdmSessionStr = nullptr;`
	`85`	`+ cdmSession.m_sessionId.clear();`
`86`	`86`	`cdmSession.m_cencSingleSampleDecrypter = nullptr;`
`87`	`87`	`}`
`88`	`88`	`}`
`@@ -293,8 +293,8 @@ bool SESSION::CSession::PreInitializeDRM(std::string& challengeB64,`
`293`	`293`	`m_decrypter->CreateSingleSampleDecrypter(initData, decKid, "", true,`
`294`	`294`	`CryptoMode::AES_CTR)) != nullptr)`
`295`	`295`	`{`
`296`		`- session.m_cdmSessionStr = session.m_cencSingleSampleDecrypter->GetSessionId();`
`297`		`- sessionId = session.m_cdmSessionStr;`
	`296`	`+ session.m_sessionId = session.m_cencSingleSampleDecrypter->GetSessionId();`
	`297`	`+ sessionId = session.m_sessionId;`
`298`	`298`	`challengeB64 = m_decrypter->GetChallengeB64Data(session.m_cencSingleSampleDecrypter);`
`299`	`299`	`}`
`300`	`300`	`else`
`@@ -466,7 +466,7 @@ bool SESSION::CSession::InitializeDRM(bool addDefaultKID /* = false */)`
`466`	`466`	`m_decrypter->GetCapabilities(session.m_cencSingleSampleDecrypter, defaultKid,`
`467`	`467`	`sessionPsshset.media_, session.m_decrypterCaps);`
`468`	`468`
`469`		`- session.m_cdmSessionStr = session.m_cencSingleSampleDecrypter->GetSessionId();`
	`469`	`+ session.m_sessionId = session.m_cencSingleSampleDecrypter->GetSessionId();`
`470`	`470`
`471`	`471`	`if (session.m_decrypterCaps.flags & DRM::DecrypterCapabilites::SSD_INVALID)`
`472`	`472`	`{`
`@@ -927,14 +927,14 @@ bool SESSION::CSession::IsCDMSessionSecurePath(size_t index)`
`927`	`927`	`DRM::DecrypterCapabilites::SSD_SECURE_PATH) != 0;`
`928`	`928`	`}`
`929`	`929`
`930`		`-const char* SESSION::CSession::GetCDMSession(unsigned int index)`
	`930`	`+std::string SESSION::CSession::GetCDMSession(unsigned int index)`
`931`	`931`	`{`
`932`	`932`	`if (index >= m_cdmSessions.size())`
`933`	`933`	`{`
`934`	`934`	`LOG::LogF(LOGERROR, "No CDM session at index %u", index);`
`935`		`- return nullptr;`
	`935`	`+ return {};`
`936`	`936`	`}`
`937`		`- return m_cdmSessions[index].m_cdmSessionStr;`
	`937`	`+ return m_cdmSessions[index].m_sessionId;`
`938`	`938`	`}`
`939`	`939`
`940`	`940`	`uint64_t SESSION::CSession::PTSToElapsed(uint64_t pts)`
`@@ -1294,7 +1294,7 @@ std::shared_ptr<Adaptive_CencSingleSampleDecrypter> SESSION::CSession::GetSingle`
`1294`	`1294`	`for (std::vector<CCdmSession>::iterator b(m_cdmSessions.begin() + 1), e(m_cdmSessions.end());`
`1295`	`1295`	`b != e; ++b)`
`1296`	`1296`	`{`
`1297`		`- if (b->m_cdmSessionStr && sessionId == b->m_cdmSessionStr)`
	`1297`	`+ if (sessionId == b->m_sessionId)`
`1298`	`1298`	`return b->m_cencSingleSampleDecrypter;`
`1299`	`1299`	`}`
`1300`	`1300`	`return nullptr;`
Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ bool CClearKeyDecrypter::OpenDRMSystem(const DRM::Config& config)`
`31`	`31`	`}`
`32`	`32`
`33`	`33`	`std::shared_ptr<Adaptive_CencSingleSampleDecrypter> CClearKeyDecrypter::CreateSingleSampleDecrypter(`
`34`		`- std::vector<uint8_t>& initData,`
	`34`	`+ const std::vector<uint8_t>& initData,`
`35`	`35`	`const std::vector<uint8_t>& defaultkeyid,`
`36`	`36`	`std::string_view licenseUrl,`
`37`	`37`	`bool skipSessionMessage,`
Original file line number	Diff line number	Diff line change
`@@ -42,7 +42,7 @@ void CWVCencSingleSampleDecrypter::SetSession(const std::string sessionId,`
`42`	`42`
`43`	`43`	`CWVCencSingleSampleDecrypter::CWVCencSingleSampleDecrypter(`
`44`	`44`	`IWVCdmAdapter<media::CdmAdapter>* cdmAdapter,`
`45`		`- std::vector<uint8_t>& pssh,`
	`45`	`+ const std::vector<uint8_t>& pssh,`
`46`	`46`	`const std::vector<uint8_t>& defaultKeyId,`
`47`	`47`	`bool skipSessionMessage,`
`48`	`48`	`CryptoMode cryptoMode)`
`@@ -202,9 +202,9 @@ void CWVCencSingleSampleDecrypter::GetCapabilities(const std::vector<uint8_t>& k`
`202`	`202`	`}`
`203`	`203`	`}`
`204`	`204`
`205`		`-const char* CWVCencSingleSampleDecrypter::GetSessionId()`
	`205`	`+std::string CWVCencSingleSampleDecrypter::GetSessionId()`
`206`	`206`	`{`
`207`		`- return m_strSession.empty() ? nullptr : m_strSession.c_str();`
	`207`	`+ return m_strSession;`
`208`	`208`	`}`
`209`	`209`
`210`	`210`	`void CWVCencSingleSampleDecrypter::CloseSessionId()`
Original file line number	Diff line number	Diff line change
`@@ -96,15 +96,15 @@ bool CWVDecrypter::OpenDRMSystem(const DRM::Config& config)`
`96`	`96`	`}`
`97`	`97`
`98`	`98`	`std::shared_ptr<Adaptive_CencSingleSampleDecrypter> CWVDecrypter::CreateSingleSampleDecrypter(`
`99`		`- std::vector<uint8_t>& initData,`
	`99`	`+ const std::vector<uint8_t>& initData,`
`100`	`100`	`const std::vector<uint8_t>& defaultKeyId,`
`101`	`101`	`std::string_view licenseUrl,`
`102`	`102`	`bool skipSessionMessage,`
`103`	`103`	`CryptoMode cryptoMode)`
`104`	`104`	`{`
`105`	`105`	`auto decrypter = std::make_shared<CWVCencSingleSampleDecrypter>(`
`106`	`106`	`m_WVCdmAdapter.get(), initData, defaultKeyId, skipSessionMessage, cryptoMode);`
`107`		`- if (!decrypter->GetSessionId())`
	`107`	`+ if (decrypter->GetSessionId().empty())`
`108`	`108`	`{`
`109`	`109`	`return nullptr;`
`110`	`110`	`}`
Original file line number	Diff line number	Diff line change
`@@ -28,7 +28,7 @@ using namespace UTILS;`
`28`	`28`
`29`	`29`	`CWVCencSingleSampleDecrypterA::CWVCencSingleSampleDecrypterA(`
`30`	`30`	`IWVCdmAdapter<jni::CJNIMediaDrm>* cdmAdapter,`
`31`		`- std::vector<uint8_t>& pssh,`
	`31`	`+ const std::vector<uint8_t>& pssh,`
`32`	`32`	`const std::vector<uint8_t>& defaultKeyId)`
`33`	`33`	`: m_cdmAdapter(cdmAdapter),`
`34`	`34`	`m_pssh(pssh),`
`@@ -159,9 +159,9 @@ CWVCencSingleSampleDecrypterA::~CWVCencSingleSampleDecrypterA()`
`159`	`159`	`m_cdmAdapter->DetachObserver(this);`
`160`	`160`	`}`
`161`	`161`
`162`		`-const char* CWVCencSingleSampleDecrypterA::GetSessionId()`
	`162`	`+std::string CWVCencSingleSampleDecrypterA::GetSessionId()`
`163`	`163`	`{`
`164`		`- return m_sessionId.c_str();`
	`164`	`+ return m_sessionId;`
`165`	`165`	`}`
`166`	`166`
`167`	`167`	`std::vector<uint8_t> CWVCencSingleSampleDecrypterA::GetChallengeData()`
Original file line number	Diff line number	Diff line change
`@@ -90,7 +90,7 @@ bool CWVDecrypterA::OpenDRMSystem(const DRM::Config& config)`
`90`	`90`	`}`
`91`	`91`
`92`	`92`	`std::shared_ptr<Adaptive_CencSingleSampleDecrypter> CWVDecrypterA::CreateSingleSampleDecrypter(`
`93`		`- std::vector<uint8_t>& initData,`
	`93`	`+ const std::vector<uint8_t>& initData,`
`94`	`94`	`const std::vector<uint8_t>& defaultKeyId,`
`95`	`95`	`std::string_view licenseUrl,`
`96`	`96`	`bool skipSessionMessage,`
`@@ -99,7 +99,7 @@ std::shared_ptr<Adaptive_CencSingleSampleDecrypter> CWVDecrypterA::CreateSingleS`
`99`	`99`	`std::shared_ptr<CWVCencSingleSampleDecrypterA> decrypter =`
`100`	`100`	`std::make_shared<CWVCencSingleSampleDecrypterA>(m_WVCdmAdapter.get(), initData, defaultKeyId);`
`101`	`101`
`102`		`- if (!(*decrypter->GetSessionId() && decrypter->StartSession(skipSessionMessage)))`
	`102`	`+ if (!(!decrypter->GetSessionId().empty() && decrypter->StartSession(skipSessionMessage)))`
`103`	`103`	`{`
`104`	`104`	`return nullptr;`
`105`	`105`	`}`