diff mbox series

[U-Boot,1/1] test/py: avb: fix test_avb_persistent_values fail

Message ID 20190506090731.7603-1-igor.opaniuk@toradex.com
State Accepted
Commit 34501d6713d2ba9d6a8e8c7a1f2942cc16ef9043
Delegated to: Tom Rini
Headers show
Series [U-Boot,1/1] test/py: avb: fix test_avb_persistent_values fail | expand

Commit Message

Igor Opaniuk May 6, 2019, 9:07 a.m. UTC 
From: Igor Opaniuk <igor.opaniuk@gmail.com>

Fix test_avb_persistent_values() pytest, which was failing because of
wrong size value provided from tee sandbox driver.

Reported-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
Signed-off-by: Igor Opaniuk <igor.opaniuk@gmail.com>
---
 drivers/tee/sandbox.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Tom Rini May 10, 2019, 11:11 a.m. UTC | #1 
On Mon, May 06, 2019 at 12:07:31PM +0300, Igor Opaniuk wrote:

> From: Igor Opaniuk <igor.opaniuk@gmail.com>
> 
> Fix test_avb_persistent_values() pytest, which was failing because of
> wrong size value provided from tee sandbox driver.
> 
> Reported-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
> Signed-off-by: Igor Opaniuk <igor.opaniuk@gmail.com>

Applied to u-boot/master, thanks!
diff mbox series

Patch

diff --git a/drivers/tee/sandbox.c b/drivers/tee/sandbox.c
index a136bc9609..2f3355c7b7 100644
--- a/drivers/tee/sandbox.c
+++ b/drivers/tee/sandbox.c
@@ -178,7 +178,7 @@  static u32 ta_avb_invoke_func(struct udevice *dev, u32 func, uint num_params,
 		if (!ep)
 			return TEE_ERROR_ITEM_NOT_FOUND;
 
-		value_sz = strlen(ep->data);
+		value_sz = strlen(ep->data) + 1;
 		memcpy(value, ep->data, value_sz);
 
 		return TEE_SUCCESS;