commit 8c71b3ecb355d4967171236033ef14c270cb0d81
author Gilles Peskine <Gilles.Peskine@arm.com> Wed Dec 19 17:37:02 2018 +0100
committer Gilles Peskine <Gilles.Peskine@arm.com> Wed Dec 19 17:37:02 2018 +0100
tree 509877eb04a6e9f9015fd44dcdb778aeafbfbcd3
parent d54b97503b42198e3b7d9fe5d7cba82ebb03469e

pk_parse*keyfile: explicitly validate path=NULL

Don't rely on the check in pk_load_file, that's fragile.

library/pkparse.c

diff --git a/library/pkparse.c b/library/pkparse.c
index 127f9b8..ae210bc 100644
--- a/library/pkparse.c
+++ b/library/pkparse.c
@@ -135,6 +135,7 @@
     unsigned char *buf;
 
     PK_VALIDATE_RET( ctx != NULL );
+    PK_VALIDATE_RET( path != NULL );
 
     if( ( ret = mbedtls_pk_load_file( path, &buf, &n ) ) != 0 )
         return( ret );
@@ -161,6 +162,7 @@
     unsigned char *buf;
 
     PK_VALIDATE_RET( ctx != NULL );
+    PK_VALIDATE_RET( path != NULL );
 
     if( ( ret = mbedtls_pk_load_file( path, &buf, &n ) ) != 0 )
         return( ret );