New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix undefined behavior in ewmhSupported() #901

Closed
wants to merge 1 commit into
base: 2.3.x
from

Conversation

Projects
None yet
3 participants
@binary1248
Member

binary1248 commented May 28, 2015

Fixes #892

I'll let @eXpl0it3r decide whether this gets into 2.3.1 or 2.3.2. 馃槢

Fix undefined behavior in ewmhSupported() caused by some window manag鈥
鈥rs not null-terminating strings. Fixes #892
@eXpl0it3r

This comment has been minimized.

Show comment
Hide comment
@eXpl0it3r

eXpl0it3r May 29, 2015

Member

I guess we can easily include this in 2.3.1.

@kode9 can you give this a shot?

Member

eXpl0it3r commented May 29, 2015

I guess we can easily include this in 2.3.1.

@kode9 can you give this a shot?

@eXpl0it3r eXpl0it3r added this to the 2.3.1 milestone May 29, 2015

@kode9

This comment has been minimized.

Show comment
Hide comment
@kode9

kode9 May 30, 2015

Contributor

Seems fine, tested it with clang and gcc as well.

Contributor

kode9 commented May 30, 2015

Seems fine, tested it with clang and gcc as well.

@eXpl0it3r

This comment has been minimized.

Show comment
Hide comment
@eXpl0it3r

eXpl0it3r Jun 4, 2015

Member

Merged in cadc4d8 on branch 2.3.x

Member

eXpl0it3r commented Jun 4, 2015

Merged in cadc4d8 on branch 2.3.x

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment