frederik
/
xserver
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

xfree86: common: gracefully handle allocation failure 

Better try to handle memory allocation gracefully than just hard
crashing by segfault.

Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
This commit is contained in:
Enrico Weigelt, metux IT consult 2025-05-06 17:43:49 +02:00
parent 111ca8af21
commit d3c4e100ed
2 changed files with 13 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
hw/xfree86/common/xf86Configure.c
View File

@ -181,7 +181,9 @@ configureInputSection(void)
}
}
mouse = calloc(1, sizeof(XF86ConfInputRec));
if (!(mouse = calloc(1, sizeof(XF86ConfInputRec))))
return NULL;
mouse->inp_identifier = XNFstrdup("Mouse0");
mouse->inp_driver = XNFstrdup("mouse");
mouse->inp_option_lst =
@ -213,9 +215,9 @@ configureScreenSection(int screennum)
ptr->scrn_device_str = tmp;
for (i = 0; i < ARRAY_SIZE(depths); i++) {
XF86ConfDisplayPtr conf_display;
conf_display = calloc(1, sizeof(XF86ConfDisplayRec));
XF86ConfDisplayPtr conf_display = calloc(1, sizeof(XF86ConfDisplayRec));
if (!conf_display)
continue;
conf_display->disp_depth = depths[i];
conf_display->disp_black.red = conf_display->disp_white.red = -1;
conf_display->disp_black.green = conf_display->disp_white.green = -1;
@ -412,9 +414,9 @@ configureModuleSection(void)
elist = LoaderListDir("extensions", NULL);
if (elist) {
for (el = elist; *el; el++) {
XF86LoadPtr module;
module = calloc(1, sizeof(XF86LoadRec));
XF86LoadPtr module = calloc(1, sizeof(XF86LoadRec));
if (!module)
return ptr;
module->load_name = *el;
ptr->mod_load_lst = (XF86LoadPtr) xf86addListItem((glp) ptr->
mod_load_lst,

6
hw/xfree86/common/xf86Option.c
View File

@ -844,13 +844,15 @@ xf86NameCmp(const char *s1, const char *s2)
char *
xf86NormalizeName(const char *s)
{
char *ret, *q;
char *q;
const char *p;
if (s == NULL)
return NULL;
ret = malloc(strlen(s) + 1);
char *ret = calloc(1, strlen(s) + 1);
if (!ret)
return NULL;
for (p = s, q = ret; *p != 0; p++) {
switch (*p) {
case '_':