aboutsummaryrefslogtreecommitdiff
path: root/nx-X11
diff options
context:
space:
mode:
authorUlrich Sibiller <uli42@gmx.de>2020-04-09 18:57:10 +0200
committerMike Gabriel <mike.gabriel@das-netzwerkteam.de>2020-05-07 14:58:17 +0200
commit58b3c8cdd7e1d31147ce52c2cd28714ac6d0039c (patch)
tree18f6461f8eda53f8be0ef5b5583bb2fff5a16d5d /nx-X11
parentacd45283bc7c7405a73015e0579f85e7aeea2d71 (diff)
downloadnx-libs-58b3c8cdd7e1d31147ce52c2cd28714ac6d0039c.tar.gz
nx-libs-58b3c8cdd7e1d31147ce52c2cd28714ac6d0039c.tar.bz2
nx-libs-58b3c8cdd7e1d31147ce52c2cd28714ac6d0039c.zip
Font.c: fix realloc bugs
Diffstat (limited to 'nx-X11')
-rw-r--r--nx-X11/programs/Xserver/hw/nxagent/Font.c43
1 files changed, 23 insertions, 20 deletions
diff --git a/nx-X11/programs/Xserver/hw/nxagent/Font.c b/nx-X11/programs/Xserver/hw/nxagent/Font.c
index 9df7380d6..369ae3b2c 100644
--- a/nx-X11/programs/Xserver/hw/nxagent/Font.c
+++ b/nx-X11/programs/Xserver/hw/nxagent/Font.c
@@ -192,7 +192,6 @@ void nxagentFreeFontCache(void)
}
SAFE_free(CACHE_ENTRY_PTR);
- CACHE_ENTRY_PTR = NULL;
CACHE_INDEX = 0;
CACHE_SIZE = 0;
@@ -289,16 +288,16 @@ void nxagentListRemoteAddName(const char *name, int status)
if (nxagentRemoteFontList.length == nxagentRemoteFontList.listSize)
{
- /* FIXME: if realloc fails the pointer is lost! */
- nxagentRemoteFontList.list = realloc(nxagentRemoteFontList.list, sizeof(nxagentFontRecPtr)
- * (nxagentRemoteFontList.listSize + 1000));
+ int num = nxagentRemoteFontList.listSize + 1000;
+ nxagentFontRecPtr *tmp1 = realloc(nxagentRemoteFontList.list, sizeof(nxagentFontRecPtr) * num);
- if (nxagentRemoteFontList.list == NULL)
+ if (tmp1 == NULL)
{
FatalError("Font: remote list memory re-allocation failed!.\n");
}
- nxagentRemoteFontList.listSize += 1000;
+ nxagentRemoteFontList.list = tmp1;
+ nxagentRemoteFontList.listSize = num;
}
if (pos < nxagentRemoteFontList.length)
@@ -542,15 +541,18 @@ Bool nxagentRealizeFont(ScreenPtr pScreen, FontPtr pFont)
if (CACHE_INDEX == CACHE_SIZE)
{
- /* FIXME: if realloc fails the pointer is lost */
- CACHE_ENTRY_PTR = realloc(CACHE_ENTRY_PTR, sizeof(nxCacheFontEntryRecPtr) * (CACHE_SIZE + 100));
+ int num = CACHE_SIZE + 100;
- if (CACHE_ENTRY_PTR == NULL)
+ nxCacheFontEntryRecPtr *tmp1 = realloc(CACHE_ENTRY_PTR,
+ sizeof(nxCacheFontEntryRecPtr) * num);
+
+ if (tmp1 == NULL)
{
FatalError("Font: Cache list memory re-allocation failed.\n");
}
- CACHE_SIZE += 100;
+ CACHE_ENTRY_PTR = tmp1;
+ CACHE_SIZE = num;
}
CACHE_ENTRY(CACHE_INDEX) = malloc(sizeof(nxCacheFontEntryRec));
@@ -888,22 +890,23 @@ static void nxagentCollectFailedFont(FontPtr fpt, XID id)
}
else if (nxagentFailedToReconnectFonts.index == nxagentFailedToReconnectFonts.size - 1)
{
- nxagentFailedToReconnectFonts.size *= 2;
-
- /* FIXME: if realloc fails the pointer is lost */
- nxagentFailedToReconnectFonts.font = realloc(nxagentFailedToReconnectFonts.font,
- nxagentFailedToReconnectFonts.size *
- sizeof(FontPtr));
+ int num = 2 * nxagentFailedToReconnectFonts.size;
- nxagentFailedToReconnectFonts.id = realloc(nxagentFailedToReconnectFonts.id,
- nxagentFailedToReconnectFonts.size *
- sizeof(XID));
+ FontPtr *tmp1 = realloc(nxagentFailedToReconnectFonts.font, num * sizeof(FontPtr));
+ XID *tmp2 = realloc(nxagentFailedToReconnectFonts.id, num * sizeof(XID));
- if (nxagentFailedToReconnectFonts.font == NULL || nxagentFailedToReconnectFonts.id == NULL)
+ if (tmp1 == NULL || tmp2 == NULL)
{
+ SAFE_free(tmp1);
+ SAFE_free(tmp2);
+
FatalError("Font: font not reconnected memory re-allocation failed!.\n");
}
+ nxagentFailedToReconnectFonts.size = num;
+ nxagentFailedToReconnectFonts.font = tmp1;
+ nxagentFailedToReconnectFonts.id = tmp2;
+
#ifdef NXAGENT_RECONNECT_FONT_DEBUG
fprintf(stderr,"nxagentCollectFailedFont: reallocated memory.\n ");
#endif