On 24 June 2011 15:54, <stefano.stabell...@eu.citrix.com> wrote: > + if (!in->c.ds) { > + char *vfb = xenstore_read_str(NULL, "device/vfb"); > + if (vfb == NULL) { > + /* there is no vfb, run vkbd on its own */ > + in->c.ds = get_displaystate(); > + } else { > + free(vfb);
xenstore_read_str() returns a pointer from a qemu_strdup() so strictly this should be a qemu_free(), I think? -- PMM