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

(submit/cleanup-alloc-funcs) treewide: replace strdup() calls to Xstrdup() 

This has been nothing but an alias for two decades now (somewhere in R6.6),
so there doesn't seem to be any practical need for this indirection.

The macro still needs to remain, as long as (external) drivers still using it.

Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
This commit is contained in:
Enrico Weigelt, metux IT consult 2024-05-08 17:36:00 +02:00
parent 35a6920340
commit f550afcd58
7 changed files with 11 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
hw/xfree86/common/xf86Bus.c
View File

@ -272,7 +272,7 @@ StringToBusType(const char *busID, const char **retID)
return BUS_PCI;
}
s = xstrdup(busID);
s = Xstrdup(busID);
p = strtok(s, ":");
if (p == NULL || *p == 0) {
free(s);

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

@ -719,7 +719,7 @@ MergeInputClasses(const InputInfoPtr idev, const InputAttributes * attrs)
classopts = xf86optionListDup(cl->option_lst);
if (cl->driver) {
free((void *) idev->driver);
idev->driver = xstrdup(cl->driver);
idev->driver = Xstrdup(cl->driver);
if (!idev->driver) {
xf86Msg(X_ERROR, "Failed to allocate memory while merging "
"InputClass configuration");
@ -1046,7 +1046,7 @@ NewInputDeviceRequest(InputOption *options, InputAttributes * attrs,
rval = BadRequest;
goto unwind;
}
pInfo->driver = xstrdup(value);
pInfo->driver = Xstrdup(value);
if (!pInfo->driver) {
rval = BadAlloc;
goto unwind;
@ -1058,7 +1058,7 @@ NewInputDeviceRequest(InputOption *options, InputAttributes * attrs,
rval = BadRequest;
goto unwind;
}
pInfo->name = xstrdup(value);
pInfo->name = Xstrdup(value);
if (!pInfo->name) {
rval = BadAlloc;
goto unwind;

2
hw/xfree86/common/xf86pciBus.c
View File

@ -272,7 +272,7 @@ xf86ParsePciBusString(const char *busID, int *bus, int *device, int *func)
if (StringToBusType(busID, &id) != BUS_PCI)
return FALSE;
s = xstrdup(id);
s = Xstrdup(id);
p = strtok(s, ":");
if (p == NULL || *p == 0) {
free(s);

2
hw/xfree86/loader/loadmod.c
View File

@ -733,7 +733,7 @@ LoadModule(const char *module, void *options, const XF86ModReqInfo *modreq,
* check the elements in the path
*/
if (PathIsAbsolute(module))
found = xstrdup(module);
found = Xstrdup(module);
path_elem = pathlist;
while (!found && *path_elem != NULL) {
found = FindModule(m, *path_elem, patterns);

2
hw/xfree86/modes/xf86Modes.c
View File

@ -681,7 +681,7 @@ xf86GetConfigModes(XF86ConfModeLinePtr conf_mode)
mode = calloc(1, sizeof(DisplayModeRec));
if (!mode)
continue;
mode->name = xstrdup(conf_mode->ml_identifier);
mode->name = Xstrdup(conf_mode->ml_identifier);
if (!mode->name) {
free(mode);
continue;

2
hw/xwayland/xwayland-xtest.c
View File

@ -361,7 +361,7 @@ setup_ei(ClientPtr client)
}
xwl_ei_client = calloc(1, sizeof *xwl_ei_client);
xwl_ei_client->cmdline = xstrdup(cmdname);
xwl_ei_client->cmdline = Xstrdup(cmdname);
if (!xwl_ei_client) {
error_ei("OOM, cannot setup EI\n");
goto out;

6
xkb/xkbUtils.c
View File

@ -1466,8 +1466,8 @@ _XkbCopyGeom(XkbDescPtr src, XkbDescPtr dst)
strcpy(dprop->value, sprop->value);
}
else {
dprop->name = xstrdup(sprop->name);
dprop->value = xstrdup(sprop->value);
dprop->name = Xstrdup(sprop->name);
dprop->value = Xstrdup(sprop->value);
}
}
@ -1520,7 +1520,7 @@ _XkbCopyGeom(XkbDescPtr src, XkbDescPtr dst)
strcpy(dcolor->spec, scolor->spec);
}
else {
dcolor->spec = xstrdup(scolor->spec);
dcolor->spec = Xstrdup(scolor->spec);
}
dcolor->pixel = scolor->pixel;
}