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

xkb: replace xallocarray() by calloc() 

Only key difference that calloc(), in contrast to rellocarray(),
is zero-initializing. The overhead is hard to measure on today's
machines, and it's safer programming practise to always allocate
zero-initialized, so one can't forget to do it explicitly.

Cocci rule:

    @@
    expression COUNT;
    expression LEN;
    @@
    - xallocarray(COUNT,LEN)
    + calloc(COUNT,LEN)

Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
This commit is contained in:
Enrico Weigelt, metux IT consult 2025-02-24 12:15:01 +01:00 committed by Enrico Weigelt, metux IT consult .
parent 33c0ab942f
commit 39278054cb
3 changed files with 10 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
xkb/maprules.c
View File

@ -93,7 +93,7 @@ InputLineAddChar(InputLine * line, int ch)
{
if (line->num_line >= line->sz_line) {
if (line->line == line->buf) {
line->line = xallocarray(line->sz_line, 2);
line->line = calloc(line->sz_line, 2);
memcpy(line->line, line->buf, line->sz_line);
}
else {

6
xkb/xkb.c
View File

@ -2837,7 +2837,7 @@ XkbSendCompatMap(ClientPtr client,
int size;
if (rep->length > 0) {
data = xallocarray(rep->length, 4);
data = calloc(rep->length, 4);
if (data) {
register unsigned i, bit;
xkbModsWireDesc *grp;
@ -3224,7 +3224,7 @@ XkbSendIndicatorMap(ClientPtr client,
if (rep->length > 0) {
CARD8 *to;
to = map = xallocarray(rep->length, 4);
to = map = calloc(rep->length, 4);
if (map) {
xkbIndicatorMapWireDesc *wire = (xkbIndicatorMapWireDesc *) to;
@ -5036,7 +5036,7 @@ XkbSendGeometry(ClientPtr client,
int len;
if (geom != NULL) {
start = desc = xallocarray(rep->length, 4);
start = desc = calloc(rep->length, 4);
if (!start)
return BadAlloc;
len = rep->length * 4;

13
xkb/xkbUtils.c
View File

@ -1065,8 +1065,8 @@ _XkbCopyClientMap(XkbDescPtr src, XkbDescPtr dst)
}
else if (!dtype->map_count || !dtype->map ||
i >= dst->map->num_types) {
tmp = xallocarray(stype->map_count,
sizeof(XkbKTMapEntryRec));
tmp = calloc(stype->map_count,
sizeof(XkbKTMapEntryRec));
if (!tmp)
return FALSE;
dtype->map = tmp;
@ -1094,8 +1094,7 @@ _XkbCopyClientMap(XkbDescPtr src, XkbDescPtr dst)
}
else if (!dtype->preserve || !dtype->map_count ||
i >= dst->map->num_types) {
tmp = xallocarray(stype->map_count,
sizeof(XkbModsRec));
tmp = calloc(stype->map_count, sizeof(XkbModsRec));
if (!tmp)
return FALSE;
dtype->preserve = tmp;
@ -1605,8 +1604,8 @@ _XkbCopyGeom(XkbDescPtr src, XkbDescPtr dst)
j < sshape->num_outlines;
j++, soutline++, doutline++) {
if (soutline->num_points) {
tmp = xallocarray(soutline->num_points,
sizeof(XkbPointRec));
tmp = calloc(soutline->num_points,
sizeof(XkbPointRec));
if (!tmp)
return FALSE;
doutline->points = tmp;
@ -1733,7 +1732,7 @@ _XkbCopyGeom(XkbDescPtr src, XkbDescPtr dst)
for (j = 0, srow = ssection->rows, drow = dsection->rows;
j < ssection->num_rows; j++, srow++, drow++) {
if (srow->num_keys) {
tmp = xallocarray(srow->num_keys, sizeof(XkbKeyRec));
tmp = calloc(srow->num_keys, sizeof(XkbKeyRec));
if (!tmp)
return FALSE;
drow->keys = tmp;