From: Richard Maciel <[email protected]>

Related to Coverity CID 10291

The aforementioned function dereferenced the auth parameter without
checking for NULL first.

Signed-off-by: Richard Maciel <[email protected]>
---
 src/tcs/tcsi_caps_tpm.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/tcs/tcsi_caps_tpm.c b/src/tcs/tcsi_caps_tpm.c
index 9f05c4e..b600fe0 100644
--- a/src/tcs/tcsi_caps_tpm.c
+++ b/src/tcs/tcsi_caps_tpm.c
@@ -113,7 +113,8 @@ TCSP_SetCapability_Internal(TCS_CONTEXT_HANDLE hContext,    
/* in */
        if ((result = ctx_verify_context(hContext)))
                goto done;
 
-       if ((result = auth_mgr_check(hContext, &pOwnerAuth->AuthHandle)))
+       if ((pOwnerAuth != NULL) &&
+               (result = auth_mgr_check(hContext, &pOwnerAuth->AuthHandle)))
                goto done;
 
        if ((result = tpm_rqu_build(TPM_ORD_SetCapability, &offset, txBlob, 
capArea, subCapSize,
-- 
1.8.5.3


------------------------------------------------------------------------------
Put Bad Developers to Shame
Dominate Development with Jenkins Continuous Integration
Continuously Automate Build, Test & Deployment 
Start a new project now. Try Jenkins in the cloud.
http://p.sf.net/sfu/13600_Cloudbees
_______________________________________________
TrouSerS-tech mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/trousers-tech

Reply via email to