[PATCH v3] compositor-x11: fix title overflow in x11_backend_create_output
Daniel Stone
daniel at fooishbar.org
Thu Jun 23 21:09:26 UTC 2016
Hi,
On 7 June 2016 at 03:29, Bryce Harrington <bryce at osg.samsung.com> wrote:
> On Sun, Jun 05, 2016 at 07:01:11PM +0200, Benoit Gschwind wrote:
>> sprintf can overflow the fixed length title which is char[32]. This
>> patch change title to dynamically allocated char array using asprintf or
>> strdup. If one of them fail we leave returning NULL to indicate the
>> failure.
>>
>> Signed-of-by: Benoit Gschwind<gschwind at gnu-log.net>
>
> Yeah I had noticed this myself earlier and wondered if it should be
> dynamically allocated. This looks like it would be a possible candidate
> for the stable branch if we do a .1.
>
> Reviewed-by: Bryce Harrington <bryce at osg.samsung.com>
Pushed with reviews, thanks!
0f1cac5..5c2f20e upstream -> master
Cheers,
Daniel
More information about the wayland-devel
mailing list