From 5f0d7bbbe8f670bdfdd83ff0ff6ac2dc515aed14 Mon Sep 17 00:00:00 2001 From: Tim Schumacher Date: Wed, 15 Jun 2022 12:39:09 +0200 Subject: [PATCH] megvii: Return a char pointer for some stubs instead `mg_facepp[3]` and `mg_facepp[4]` seem to return version strings, which occasionally get logged if the return value is non-zero. Return a dummy string for those functions to avoid crashing in strlen. Change-Id: Icc5d2ab2f72c480f03e652e3849aceaf4cfb498b --- megvii/megvii.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/megvii/megvii.c b/megvii/megvii.c index eaf0584..f0fe1b0 100644 --- a/megvii/megvii.c +++ b/megvii/megvii.c @@ -8,12 +8,16 @@ static int stub_fail() { return -1; } +static char *stub_fail_str() { + return "stub"; +} + void *mg_facepp[] = { &stub_fail, &stub_fail, &stub_fail, - &stub_fail, - &stub_fail, + &stub_fail_str, + &stub_fail_str, &stub_fail, &stub_fail, &stub_fail,