The temperature property accessors decode the channel index out of the property name and report a failure with error_setg_errno(). sscanf() does not set errno when it matches fewer items than expected, so the message is suffixed with whatever unrelated error was left behind by an earlier call. The bounds check right below already uses plain error_setg() for the same condition.
Signed-off-by: Emmanuel Blot <[email protected]> --- hw/sensor/tmp421.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/hw/sensor/tmp421.c b/hw/sensor/tmp421.c index 127edd0ba5..fde41b2323 100644 --- a/hw/sensor/tmp421.c +++ b/hw/sensor/tmp421.c @@ -117,7 +117,7 @@ static void tmp421_get_temperature(Object *obj, Visitor *v, const char *name, int tempid; if (sscanf(name, "temperature%d", &tempid) != 1) { - error_setg_errno(errp, errno, "error reading %s", name); + error_setg(errp, "error reading %s", name); return; } @@ -154,7 +154,7 @@ static void tmp421_set_temperature(Object *obj, Visitor *v, const char *name, } if (sscanf(name, "temperature%d", &tempid) != 1) { - error_setg_errno(errp, errno, "error reading %s", name); + error_setg(errp, "error reading %s", name); return; } -- 2.53.0-Meta
