mirror of
https://github.com/X11Libre/xserver.git
synced 2026-04-14 17:18:09 +00:00
os: handle memory allocation failure in set_font_authorizations()
Reported in #1817: xwayland-24.1.6/redhat-linux-build/../os/utils.c:1108:9: warning[-Wanalyzer-possible-null-dereference]: dereference of possibly-NULL ‘result’ xwayland-24.1.6/redhat-linux-build/../os/utils.c:1108:9: danger: ‘malloc((long unsigned int)len + 18)’ could be NULL: unchecked value from (3) Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com> Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/2163>
This commit is contained in:
committed by
Enrico Weigelt, metux IT consult
parent
e579e951e1
commit
34382aa4d1
@@ -832,8 +832,14 @@ set_font_authorizations(char **authorizations, int *authlen, void *client)
|
||||
|
||||
len = strlen(hnameptr) + 1;
|
||||
result = calloc(1, len + sizeof(AUTHORIZATION_NAME) + 4);
|
||||
if (!result)
|
||||
if (result == NULL) {
|
||||
#if defined(HAVE_GETADDRINFO)
|
||||
if (ai) {
|
||||
freeaddrinfo(ai);
|
||||
}
|
||||
#endif
|
||||
return 0;
|
||||
}
|
||||
|
||||
p = result;
|
||||
*p++ = sizeof(AUTHORIZATION_NAME) >> 8;
|
||||
|
||||
Reference in New Issue
Block a user