On 27 July 2013 00:21, Bill Spitzak <[email protected]> wrote: >> + device = output->device; >> fbdev_output_destroy(base); > > > Are you sure this does not free the memory that is now at *device?
Maybe I missed something - can you point me at the code which makes you think it does? My reading of the code indicates the string is a const char * passed into fbdev_ouput_create and saved into that device struct. It is not strdup()ed into there. This patch is to avoid a dereference into a block of memory freed in fbdev_output_destroy() Rob _______________________________________________ wayland-devel mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/wayland-devel
