[PATCH] pixman-renderer: don't forget to initialize the debug fields

Scott Moreau oreaus at gmail.com
Fri Mar 8 11:44:40 PST 2013


On Fri, Mar 8, 2013 at 12:35 PM, Philipp Brüschweiler <blei42 at gmail.com>wrote:

> Fixes https://bugs.freedesktop.org/show_bug.cgi?id=61930
> ---
>  src/pixman-renderer.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/src/pixman-renderer.c b/src/pixman-renderer.c
> index 9571f6f..9dbe9f0 100644
> --- a/src/pixman-renderer.c
> +++ b/src/pixman-renderer.c
> @@ -485,6 +485,8 @@ pixman_renderer_init(struct weston_compositor *ec)
>         if (renderer == NULL)
>                 return -1;
>
> +       renderer->repaint_debug = 0;
> +       renderer->debug_color = NULL;
>

Alternatively you could change malloc to calloc so the entire struct is
initialized.


>         renderer->base.read_pixels = pixman_renderer_read_pixels;
>         renderer->base.repaint_output = pixman_renderer_repaint_output;
>         renderer->base.flush_damage = pixman_renderer_flush_damage;
> --
> 1.8.1.5
>
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/wayland-devel/attachments/20130308/2adf8d77/attachment.html>


More information about the wayland-devel mailing list