The function machine_get_loadparm() is supposed to produce as C-string, that is a null-terminated one, but it does not. ElectricFence can detect this problem if the loadparm machine property is used.
Let us make the returned string a null-terminated one. Signed-off-by: Halil Pasic <[email protected]> Fixes: 7104bae9de ("hw/s390x: provide loadparm property for the machine") --- hw/s390x/s390-virtio-ccw.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/hw/s390x/s390-virtio-ccw.c b/hw/s390x/s390-virtio-ccw.c index 8cc2f25d8a..e0e4a69ac8 100644 --- a/hw/s390x/s390-virtio-ccw.c +++ b/hw/s390x/s390-virtio-ccw.c @@ -701,8 +701,12 @@ bool hpage_1m_allowed(void) static char *machine_get_loadparm(Object *obj, Error **errp) { S390CcwMachineState *ms = S390_CCW_MACHINE(obj); + char *loadparm_str; - return g_memdup(ms->loadparm, sizeof(ms->loadparm)); + /* make a null-terminated string */ + loadparm_str = g_memdup(ms->loadparm, sizeof(ms->loadparm) + 1); + loadparm_str[sizeof(ms->loadparm)] = 0; + return loadparm_str; } static void machine_set_loadparm(Object *obj, const char *val, Error **errp) base-commit: 53ce7b47b5bf47db067b81c18c786ed7b792d031 -- 2.17.1
