Make sure we do not crash if session is null

This was seen in testing when openssl tries to duplicate a context and operation state fails (common). The old context is left with a null session, but later on openssl tries to use it again. Signed-off-by: Simo Sorce <[email protected]>
simo5 · Jun 12, 2024 · e9c9202 · e9c9202
1 parent d3cd256
commit e9c9202
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/src/session.c b/src/session.c
@@ -386,11 +386,17 @@ static void session_free(P11PROV_SESSION *session)
 
 CK_SESSION_HANDLE p11prov_session_handle(P11PROV_SESSION *session)
 {
+    if (!session) {
+        return CK_INVALID_HANDLE;
+    }
     return session->session;
 }
 
 CK_SLOT_ID p11prov_session_slotid(P11PROV_SESSION *session)
 {
+    if (!session) {
+        return CK_UNAVAILABLE_INFORMATION;
+    }
     return session->slotid;
 }