[PATCH weston] editor: Add missing free() and display_destroy() in main

Quentin Glidic sardemff7+wayland at sardemff7.net
Thu Mar 30 23:36:47 UTC 2017


On 3/29/17 6:16 PM, Raúl Peñacoba wrote:
> Signed-off-by: Raúl Peñacoba <raul.mikaop.zelda at gmail.com>

Good:
Reviewed-by: Quentin Glidic <sardemff7+git at sardemff7.net>

And pushed:
bd8dc0a2..745e647f  master -> master

Thanks,

> ---
>   clients/editor.c | 3 +++
>   1 file changed, 3 insertions(+)
> 
> diff --git a/clients/editor.c b/clients/editor.c
> index a0cc97af..b63c5628 100644
> --- a/clients/editor.c
> +++ b/clients/editor.c
> @@ -1607,6 +1607,7 @@ main(int argc, char *argv[])
>   	editor.display = display_create(&argc, argv);
>   	if (editor.display == NULL) {
>   		fprintf(stderr, "failed to create display: %m\n");
> +		free(text_buffer);
>   		return -1;
>   	}
>   
> @@ -1615,6 +1616,8 @@ main(int argc, char *argv[])
>   
>   	if (editor.text_input_manager == NULL) {
>   		fprintf(stderr, "No text input manager global\n");
> +		display_destroy(editor.display);
> +		free(text_buffer);
>   		return -1;
>   	}
>   
> 


-- 

Quentin “Sardem FF7” Glidic


More information about the wayland-devel mailing list