wayland: use getenv() according to spec

From the NOTES section of getenv(3):
 The string pointed to by the return value of getenv() may be statically
 allocated, and can be modified by a subsequent call to getenv()...

So, previously xcursor_theme could be modified by libc to contain
xcursor_size. This has been fixed by getting xcursor_theme after
computing the value of xcursor_size.
This commit is contained in:
Pranjal Kole 2022-01-29 16:18:19 +05:30
parent 2cc0a450b4
commit fa396cdc33

View file

@ -1630,7 +1630,6 @@ wayl_reload_xcursor_theme(struct seat *seat, int new_scale)
seat->pointer.cursor = NULL;
}
const char *xcursor_theme = getenv("XCURSOR_THEME");
int xcursor_size = 24;
{
@ -1642,6 +1641,8 @@ wayl_reload_xcursor_theme(struct seat *seat, int new_scale)
}
}
const char *xcursor_theme = getenv("XCURSOR_THEME");
LOG_INFO("cursor theme: %s, size: %u, scale: %d",
xcursor_theme, xcursor_size, new_scale);