x86/sgx: Return -EINVAL on a zero length buffer in sgx_ioc_enclave_add_pages()
authorJarkko Sakkinen <jarkko@kernel.org>
Thu, 3 Dec 2020 18:35:27 +0000 (20:35 +0200)
committerBorislav Petkov <bp@suse.de>
Thu, 3 Dec 2020 18:54:40 +0000 (19:54 +0100)
commita4b9c48b96517ff4780b22a784e7537eac5dc21b
tree1e78a3ec15ba27f70d1adeaac27d2271592b109f
parentbab8c183d1d452f5fdc059aef2f0788bd2986231
x86/sgx: Return -EINVAL on a zero length buffer in sgx_ioc_enclave_add_pages()

The sgx_enclave_add_pages.length field is documented as

 * @length:     length of the data (multiple of the page size)

Fail with -EINVAL, when the caller gives a zero length buffer of data
to be added as pages to an enclave. Right now 'ret' is returned as
uninitialized in that case.

 [ bp: Flesh out commit message. ]

Fixes: c6d26d370767 ("x86/sgx: Add SGX_IOC_ENCLAVE_ADD_PAGES")
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Jarkko Sakkinen <jarkko@kernel.org>
Signed-off-by: Borislav Petkov <bp@suse.de>
Link: https://lore.kernel.org/linux-sgx/X8ehQssnslm194ld@mwanda/
Link: https://lkml.kernel.org/r/20201203183527.139317-1-jarkko@kernel.org
arch/x86/kernel/cpu/sgx/ioctl.c