+int tpm_buf_init(struct tpm_buf *buf, u16 tag, u32 ordinal)
+{
+       int rc;
+
+       rc = __tpm_buf_init(buf);


Assuming that functions like tpm_buf_init() are the top-level API being defined in this patch, shouldn't it check if buf is valid before passing into the internal functions like __tpm_buf_init(buf) (maybe WARN()/BUG_ON()?).  Or does __tpm_buf_init(buf) do this check?

Thanks,
Jay

Reply via email to