Synchronised line endinge with release branch

1 files changed, 403 insertions, 403 deletions

diff --git a/libXpm/src/CrDatFrI.c b/libXpm/src/CrDatFrI.c
index 04bb9a046..0dacf5127 100644
--- a/libXpm/src/CrDatFrI.c
+++ b/libXpm/src/CrDatFrI.c
@@ -1,403 +1,403 @@
-/*

- * Copyright (C) 1989-95 GROUPE BULL

- *

- * Permission is hereby granted, free of charge, to any person obtaining a copy

- * of this software and associated documentation files (the "Software"), to

- * deal in the Software without restriction, including without limitation the

- * rights to use, copy, modify, merge, publish, distribute, sublicense, and/or

- * sell copies of the Software, and to permit persons to whom the Software is

- * furnished to do so, subject to the following conditions:

- *

- * The above copyright notice and this permission notice shall be included in

- * all copies or substantial portions of the Software.

- *

- * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR

- * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,

- * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL

- * GROUPE BULL BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN

- * AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN

- * CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

- *

- * Except as contained in this notice, the name of GROUPE BULL shall not be

- * used in advertising or otherwise to promote the sale, use or other dealings

- * in this Software without prior written authorization from GROUPE BULL.

- */

-

-/*****************************************************************************\

-*  CrDataFI.c:                                                                *

-*                                                                             *

-*  XPM library                                                                *

-*  Scan an image and possibly its mask and create an XPM array                *

-*                                                                             *

-*  Developed by Arnaud Le Hors                                                *

-\*****************************************************************************/

-

-/* October 2004, source code review by Thomas Biege <thomas@suse.de> */

-

-#ifdef HAVE_CONFIG_H

-#include <config.h>

-#endif

-#include "XpmI.h"

-

-LFUNC(CreateColors, int, (char **dataptr, unsigned int *data_size,

-			  XpmColor *colors, unsigned int ncolors,

-			  unsigned int cpp));

-

-LFUNC(CreatePixels, void, (char **dataptr, unsigned int data_size,

-			   unsigned int width,

-			   unsigned int height, unsigned int cpp,

-			   unsigned int *pixels, XpmColor *colors));

-

-LFUNC(CountExtensions, void, (XpmExtension *ext, unsigned int num,

-			      unsigned int *ext_size,

-			      unsigned int *ext_nlines));

-

-LFUNC(CreateExtensions, void, (char **dataptr, unsigned int data_size,

-			       unsigned int offset,

-			       XpmExtension *ext, unsigned int num,

-			       unsigned int ext_nlines));

-

-int

-XpmCreateDataFromImage(

-    Display		  *display,

-    char		***data_return,

-    XImage		  *image,

-    XImage		  *shapeimage,

-    XpmAttributes	  *attributes)

-{

-    XpmImage xpmimage;

-    XpmInfo info;

-    int ErrorStatus;

-

-    /* initialize return value */

-    if (data_return)

-	*data_return = NULL;

-

-    /* create an XpmImage from the image */

-    ErrorStatus = XpmCreateXpmImageFromImage(display, image, shapeimage,

-					     &xpmimage, attributes);

-    if (ErrorStatus != XpmSuccess)

-	return (ErrorStatus);

-

-    /* create the data from the XpmImage */

-    if (attributes) {

-	xpmSetInfo(&info, attributes);

-	ErrorStatus = XpmCreateDataFromXpmImage(data_return, &xpmimage, &info);

-    } else

-	ErrorStatus = XpmCreateDataFromXpmImage(data_return, &xpmimage, NULL);

-

-    /* free the XpmImage */

-    XpmFreeXpmImage(&xpmimage);

-

-    return (ErrorStatus);

-}

-

-#undef RETURN

-#define RETURN(status) \

-do \

-{ \

-      ErrorStatus = status; \

-      goto exit; \

-} while(0)

-

-int

-XpmCreateDataFromXpmImage(

-    char	***data_return,

-    XpmImage	  *image,

-    XpmInfo	  *info)

-{

-    /* calculation variables */

-    int ErrorStatus;

-    char buf[BUFSIZ];

-    char **header = NULL, **data, **sptr, **sptr2, *s;

-    unsigned int header_size, header_nlines;

-    unsigned int data_size, data_nlines;

-    unsigned int extensions = 0, ext_size = 0, ext_nlines = 0;

-    unsigned int offset, l, n;

-

-    *data_return = NULL;

-

-    extensions = info && (info->valuemask & XpmExtensions)

-	&& info->nextensions;

-

-    /* compute the number of extensions lines and size */

-    if (extensions)

-	CountExtensions(info->extensions, info->nextensions,

-			&ext_size, &ext_nlines);

-

-    /*

-     * alloc a temporary array of char pointer for the header section which

-     * is the hints line + the color table lines

-     */

-    header_nlines = 1 + image->ncolors; /* this may wrap and/or become 0 */

-

-    /* 2nd check superfluous if we do not need header_nlines any further */

-    if(header_nlines <= image->ncolors ||

-       header_nlines >= UINT_MAX / sizeof(char *))

-    	return(XpmNoMemory);

-

-    header_size = sizeof(char *) * header_nlines;

-    if (header_size >= UINT_MAX / sizeof(char *))

-	return (XpmNoMemory);

-    header = (char **) XpmCalloc(header_size, sizeof(char *)); /* can we trust image->ncolors */

-    if (!header)

-	return (XpmNoMemory);

-

-    /* print the hints line */

-    s = buf;

-#ifndef VOID_SPRINTF

-    s +=

-#endif

-    sprintf(s, "%d %d %d %d", image->width, image->height,

-	    image->ncolors, image->cpp);

-#ifdef VOID_SPRINTF

-    s += strlen(s);

-#endif

-

-    if (info && (info->valuemask & XpmHotspot)) {

-#ifndef VOID_SPRINTF

-	s +=

-#endif

-	sprintf(s, " %d %d", info->x_hotspot, info->y_hotspot);

-#ifdef VOID_SPRINTF

-	s += strlen(s);

-#endif

-    }

-    if (extensions) {

-	strcpy(s, " XPMEXT");

-	s += 7;

-    }

-    l = s - buf + 1;

-    *header = (char *) XpmMalloc(l);

-    if (!*header)

-	RETURN(XpmNoMemory);

-    header_size += l;

-    strcpy(*header, buf);

-

-    /* print colors */

-    ErrorStatus = CreateColors(header + 1, &header_size,

-			       image->colorTable, image->ncolors, image->cpp);

-

-    if (ErrorStatus != XpmSuccess)

-	RETURN(ErrorStatus);

-

-    /* now we know the size needed, alloc the data and copy the header lines */

-    offset = image->width * image->cpp + 1;

-

-    if(offset <= image->width || offset <= image->cpp)

-	RETURN(XpmNoMemory);

-

-    if( (image->height + ext_nlines) >= UINT_MAX / sizeof(char *))

-	RETURN(XpmNoMemory);

-    data_size = (image->height + ext_nlines) * sizeof(char *);

-

-    if (image->height > UINT_MAX / offset ||

-        image->height * offset > UINT_MAX - data_size)

-	RETURN(XpmNoMemory);

-    data_size += image->height * offset;

-

-    if( (header_size + ext_size) >= (UINT_MAX - data_size) )

-	RETURN(XpmNoMemory);

-    data_size += header_size + ext_size;

-

-    data = (char **) XpmMalloc(data_size);

-    if (!data)

-	RETURN(XpmNoMemory);

-

-    data_nlines = header_nlines + image->height + ext_nlines;

-    *data = (char *) (data + data_nlines);

-

-    /* can header have less elements then n suggests? */

-    n = image->ncolors;

-    for (l = 0, sptr = data, sptr2 = header; l <= n && sptr && sptr2; l++, sptr++, sptr2++) {

-	strcpy(*sptr, *sptr2);

-	*(sptr + 1) = *sptr + strlen(*sptr2) + 1;

-    }

-

-    /* print pixels */

-    data[header_nlines] = (char *) data + header_size

-	+ (image->height + ext_nlines) * sizeof(char *);

-

-    CreatePixels(data + header_nlines, data_size-header_nlines, image->width, image->height,

-		 image->cpp, image->data, image->colorTable);

-

-    /* print extensions */

-    if (extensions)

-	CreateExtensions(data + header_nlines + image->height - 1,

-			 data_size - header_nlines - image->height + 1, offset,

-			 info->extensions, info->nextensions,

-			 ext_nlines);

-

-    *data_return = data;

-    ErrorStatus = XpmSuccess;

-

-/* exit point, free only locally allocated variables */

-exit:

-    if (header) {

-	for (l = 0; l < header_nlines; l++)

-	    if (header[l])

-		XpmFree(header[l]);

-		XpmFree(header);

-    }

-    return(ErrorStatus);

-}

-

-static int

-CreateColors(

-    char		**dataptr,

-    unsigned int	 *data_size,

-    XpmColor		 *colors,

-    unsigned int	  ncolors,

-    unsigned int	  cpp)

-{

-    char buf[BUFSIZ];

-    unsigned int a, key, l;

-    char *s, *s2;

-    char **defaults;

-

-    /* can ncolors be trusted here? */

-    for (a = 0; a < ncolors; a++, colors++, dataptr++) {

-

-	defaults = (char **) colors;

-	if(sizeof(buf) <= cpp)

-	    return(XpmNoMemory);

-	strncpy(buf, *defaults++, cpp);

-	s = buf + cpp;

-

-	if(sizeof(buf) <= (s-buf))

-		return XpmNoMemory;

-

-	for (key = 1; key <= NKEYS; key++, defaults++) {

-	    if ((s2 = *defaults)) {

-#ifndef VOID_SPRINTF

-		s +=

-#endif

-		/* assume C99 compliance */

-			snprintf(s, sizeof(buf)-(s-buf), "\t%s %s", xpmColorKeys[key - 1], s2);

-#ifdef VOID_SPRINTF

-		s += strlen(s);

-#endif

-		/* does s point out-of-bounds? */

-		if(sizeof(buf) < (s-buf))

-			return XpmNoMemory;

-	    }

-	}

-	/* what about using strdup()? */

-	l = s - buf + 1;

-	s = (char *) XpmMalloc(l);

-	if (!s)

-	    return (XpmNoMemory);

-	*data_size += l;

-	*dataptr = strcpy(s, buf);

-    }

-    return (XpmSuccess);

-}

-

-static void

-CreatePixels(

-    char		**dataptr,

-    unsigned int	  data_size,

-    unsigned int	  width,

-    unsigned int	  height,

-    unsigned int	  cpp,

-    unsigned int	 *pixels,

-    XpmColor		 *colors)

-{

-    char *s;

-    unsigned int x, y, h, offset;

-

-    if(height <= 1)

-    	return;

-

-    h = height - 1;

-

-    offset = width * cpp + 1;

-

-    if(offset <= width || offset <= cpp)

-    	return;

-

-    /* why trust h? */

-    for (y = 0; y < h; y++, dataptr++) {

-	s = *dataptr;

-	/* why trust width? */

-	for (x = 0; x < width; x++, pixels++) {

-	    if(cpp > (data_size - (s - *dataptr)))

-	    	return;

-	    strncpy(s, colors[*pixels].string, cpp); /* why trust pixel? */

-	    s += cpp;

-	}

-	*s = '\0';

-	if(offset > data_size)

-		return;

-	*(dataptr + 1) = *dataptr + offset;

-    }

-    /* duplicate some code to avoid a test in the loop */

-    s = *dataptr;

-    /* why trust width? */

-    for (x = 0; x < width; x++, pixels++) {

-	if(cpp > data_size - (s - *dataptr))

-	    	return;

-	strncpy(s, colors[*pixels].string, cpp); /* why should we trust *pixel? */

-	s += cpp;

-    }

-    *s = '\0';

-}

-

-static void

-CountExtensions(

-    XpmExtension	*ext,

-    unsigned int	 num,

-    unsigned int	*ext_size,

-    unsigned int	*ext_nlines)

-{

-    unsigned int x, y, a, size, nlines;

-    char **line;

-

-    size = 0;

-    nlines = 0;

-    for (x = 0; x < num; x++, ext++) {

-	/* 1 for the name */

-	nlines += ext->nlines + 1;

-	/* 8 = 7 (for "XPMEXT ") + 1 (for 0) */

-	size += strlen(ext->name) + 8;

-	a = ext->nlines;

-	for (y = 0, line = ext->lines; y < a; y++, line++)

-	    size += strlen(*line) + 1;

-    }

-    /* 10 and 1 are for the ending "XPMENDEXT" */

-    *ext_size = size + 10;

-    *ext_nlines = nlines + 1;

-}

-

-static void

-CreateExtensions(

-    char		**dataptr,

-    unsigned int	  data_size,

-    unsigned int	  offset,

-    XpmExtension	 *ext,

-    unsigned int	  num,

-    unsigned int	  ext_nlines)

-{

-    unsigned int x, y, a, b;

-    char **line;

-

-    *(dataptr + 1) = *dataptr + offset;

-    dataptr++;

-    a = 0;

-    for (x = 0; x < num; x++, ext++) {

-	snprintf(*dataptr, data_size, "XPMEXT %s", ext->name);

-	a++;

-	if (a < ext_nlines)

-	    *(dataptr + 1) = *dataptr + strlen(ext->name) + 8;

-	dataptr++;

-	b = ext->nlines; /* can we trust these values? */

-	for (y = 0, line = ext->lines; y < b; y++, line++) {

-	    strcpy(*dataptr, *line);

-	    a++;

-	    if (a < ext_nlines)

-		*(dataptr + 1) = *dataptr + strlen(*line) + 1;

-	    dataptr++;

-	}

-    }

-    strcpy(*dataptr, "XPMENDEXT");

-}

+/*
+ * Copyright (C) 1989-95 GROUPE BULL
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a copy
+ * of this software and associated documentation files (the "Software"), to
+ * deal in the Software without restriction, including without limitation the
+ * rights to use, copy, modify, merge, publish, distribute, sublicense, and/or
+ * sell copies of the Software, and to permit persons to whom the Software is
+ * furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice shall be included in
+ * all copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
+ * GROUPE BULL BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN
+ * AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
+ * CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
+ *
+ * Except as contained in this notice, the name of GROUPE BULL shall not be
+ * used in advertising or otherwise to promote the sale, use or other dealings
+ * in this Software without prior written authorization from GROUPE BULL.
+ */
+
+/*****************************************************************************\
+*  CrDataFI.c:                                                                *
+*                                                                             *
+*  XPM library                                                                *
+*  Scan an image and possibly its mask and create an XPM array                *
+*                                                                             *
+*  Developed by Arnaud Le Hors                                                *
+\*****************************************************************************/
+
+/* October 2004, source code review by Thomas Biege <thomas@suse.de> */
+
+#ifdef HAVE_CONFIG_H
+#include <config.h>
+#endif
+#include "XpmI.h"
+
+LFUNC(CreateColors, int, (char **dataptr, unsigned int *data_size,
+			  XpmColor *colors, unsigned int ncolors,
+			  unsigned int cpp));
+
+LFUNC(CreatePixels, void, (char **dataptr, unsigned int data_size,
+			   unsigned int width,
+			   unsigned int height, unsigned int cpp,
+			   unsigned int *pixels, XpmColor *colors));
+
+LFUNC(CountExtensions, void, (XpmExtension *ext, unsigned int num,
+			      unsigned int *ext_size,
+			      unsigned int *ext_nlines));
+
+LFUNC(CreateExtensions, void, (char **dataptr, unsigned int data_size,
+			       unsigned int offset,
+			       XpmExtension *ext, unsigned int num,
+			       unsigned int ext_nlines));
+
+int
+XpmCreateDataFromImage(
+    Display		  *display,
+    char		***data_return,
+    XImage		  *image,
+    XImage		  *shapeimage,
+    XpmAttributes	  *attributes)
+{
+    XpmImage xpmimage;
+    XpmInfo info;
+    int ErrorStatus;
+
+    /* initialize return value */
+    if (data_return)
+	*data_return = NULL;
+
+    /* create an XpmImage from the image */
+    ErrorStatus = XpmCreateXpmImageFromImage(display, image, shapeimage,
+					     &xpmimage, attributes);
+    if (ErrorStatus != XpmSuccess)
+	return (ErrorStatus);
+
+    /* create the data from the XpmImage */
+    if (attributes) {
+	xpmSetInfo(&info, attributes);
+	ErrorStatus = XpmCreateDataFromXpmImage(data_return, &xpmimage, &info);
+    } else
+	ErrorStatus = XpmCreateDataFromXpmImage(data_return, &xpmimage, NULL);
+
+    /* free the XpmImage */
+    XpmFreeXpmImage(&xpmimage);
+
+    return (ErrorStatus);
+}
+
+#undef RETURN
+#define RETURN(status) \
+do \
+{ \
+      ErrorStatus = status; \
+      goto exit; \
+} while(0)
+
+int
+XpmCreateDataFromXpmImage(
+    char	***data_return,
+    XpmImage	  *image,
+    XpmInfo	  *info)
+{
+    /* calculation variables */
+    int ErrorStatus;
+    char buf[BUFSIZ];
+    char **header = NULL, **data, **sptr, **sptr2, *s;
+    unsigned int header_size, header_nlines;
+    unsigned int data_size, data_nlines;
+    unsigned int extensions = 0, ext_size = 0, ext_nlines = 0;
+    unsigned int offset, l, n;
+
+    *data_return = NULL;
+
+    extensions = info && (info->valuemask & XpmExtensions)
+	&& info->nextensions;
+
+    /* compute the number of extensions lines and size */
+    if (extensions)
+	CountExtensions(info->extensions, info->nextensions,
+			&ext_size, &ext_nlines);
+
+    /*
+     * alloc a temporary array of char pointer for the header section which
+     * is the hints line + the color table lines
+     */
+    header_nlines = 1 + image->ncolors; /* this may wrap and/or become 0 */
+
+    /* 2nd check superfluous if we do not need header_nlines any further */
+    if(header_nlines <= image->ncolors ||
+       header_nlines >= UINT_MAX / sizeof(char *))
+    	return(XpmNoMemory);
+
+    header_size = sizeof(char *) * header_nlines;
+    if (header_size >= UINT_MAX / sizeof(char *))
+	return (XpmNoMemory);
+    header = (char **) XpmCalloc(header_size, sizeof(char *)); /* can we trust image->ncolors */
+    if (!header)
+	return (XpmNoMemory);
+
+    /* print the hints line */
+    s = buf;
+#ifndef VOID_SPRINTF
+    s +=
+#endif
+    sprintf(s, "%d %d %d %d", image->width, image->height,
+	    image->ncolors, image->cpp);
+#ifdef VOID_SPRINTF
+    s += strlen(s);
+#endif
+
+    if (info && (info->valuemask & XpmHotspot)) {
+#ifndef VOID_SPRINTF
+	s +=
+#endif
+	sprintf(s, " %d %d", info->x_hotspot, info->y_hotspot);
+#ifdef VOID_SPRINTF
+	s += strlen(s);
+#endif
+    }
+    if (extensions) {
+	strcpy(s, " XPMEXT");
+	s += 7;
+    }
+    l = s - buf + 1;
+    *header = (char *) XpmMalloc(l);
+    if (!*header)
+	RETURN(XpmNoMemory);
+    header_size += l;
+    strcpy(*header, buf);
+
+    /* print colors */
+    ErrorStatus = CreateColors(header + 1, &header_size,
+			       image->colorTable, image->ncolors, image->cpp);
+
+    if (ErrorStatus != XpmSuccess)
+	RETURN(ErrorStatus);
+
+    /* now we know the size needed, alloc the data and copy the header lines */
+    offset = image->width * image->cpp + 1;
+
+    if(offset <= image->width || offset <= image->cpp)
+	RETURN(XpmNoMemory);
+
+    if( (image->height + ext_nlines) >= UINT_MAX / sizeof(char *))
+	RETURN(XpmNoMemory);
+    data_size = (image->height + ext_nlines) * sizeof(char *);
+
+    if (image->height > UINT_MAX / offset ||
+        image->height * offset > UINT_MAX - data_size)
+	RETURN(XpmNoMemory);
+    data_size += image->height * offset;
+
+    if( (header_size + ext_size) >= (UINT_MAX - data_size) )
+	RETURN(XpmNoMemory);
+    data_size += header_size + ext_size;
+
+    data = (char **) XpmMalloc(data_size);
+    if (!data)
+	RETURN(XpmNoMemory);
+
+    data_nlines = header_nlines + image->height + ext_nlines;
+    *data = (char *) (data + data_nlines);
+
+    /* can header have less elements then n suggests? */
+    n = image->ncolors;
+    for (l = 0, sptr = data, sptr2 = header; l <= n && sptr && sptr2; l++, sptr++, sptr2++) {
+	strcpy(*sptr, *sptr2);
+	*(sptr + 1) = *sptr + strlen(*sptr2) + 1;
+    }
+
+    /* print pixels */
+    data[header_nlines] = (char *) data + header_size
+	+ (image->height + ext_nlines) * sizeof(char *);
+
+    CreatePixels(data + header_nlines, data_size-header_nlines, image->width, image->height,
+		 image->cpp, image->data, image->colorTable);
+
+    /* print extensions */
+    if (extensions)
+	CreateExtensions(data + header_nlines + image->height - 1,
+			 data_size - header_nlines - image->height + 1, offset,
+			 info->extensions, info->nextensions,
+			 ext_nlines);
+
+    *data_return = data;
+    ErrorStatus = XpmSuccess;
+
+/* exit point, free only locally allocated variables */
+exit:
+    if (header) {
+	for (l = 0; l < header_nlines; l++)
+	    if (header[l])
+		XpmFree(header[l]);
+		XpmFree(header);
+    }
+    return(ErrorStatus);
+}
+
+static int
+CreateColors(
+    char		**dataptr,
+    unsigned int	 *data_size,
+    XpmColor		 *colors,
+    unsigned int	  ncolors,
+    unsigned int	  cpp)
+{
+    char buf[BUFSIZ];
+    unsigned int a, key, l;
+    char *s, *s2;
+    char **defaults;
+
+    /* can ncolors be trusted here? */
+    for (a = 0; a < ncolors; a++, colors++, dataptr++) {
+
+	defaults = (char **) colors;
+	if(sizeof(buf) <= cpp)
+	    return(XpmNoMemory);
+	strncpy(buf, *defaults++, cpp);
+	s = buf + cpp;
+
+	if(sizeof(buf) <= (s-buf))
+		return XpmNoMemory;
+
+	for (key = 1; key <= NKEYS; key++, defaults++) {
+	    if ((s2 = *defaults)) {
+#ifndef VOID_SPRINTF
+		s +=
+#endif
+		/* assume C99 compliance */
+			snprintf(s, sizeof(buf)-(s-buf), "\t%s %s", xpmColorKeys[key - 1], s2);
+#ifdef VOID_SPRINTF
+		s += strlen(s);
+#endif
+		/* does s point out-of-bounds? */
+		if(sizeof(buf) < (s-buf))
+			return XpmNoMemory;
+	    }
+	}
+	/* what about using strdup()? */
+	l = s - buf + 1;
+	s = (char *) XpmMalloc(l);
+	if (!s)
+	    return (XpmNoMemory);
+	*data_size += l;
+	*dataptr = strcpy(s, buf);
+    }
+    return (XpmSuccess);
+}
+
+static void
+CreatePixels(
+    char		**dataptr,
+    unsigned int	  data_size,
+    unsigned int	  width,
+    unsigned int	  height,
+    unsigned int	  cpp,
+    unsigned int	 *pixels,
+    XpmColor		 *colors)
+{
+    char *s;
+    unsigned int x, y, h, offset;
+
+    if(height <= 1)
+    	return;
+
+    h = height - 1;
+
+    offset = width * cpp + 1;
+
+    if(offset <= width || offset <= cpp)
+    	return;
+
+    /* why trust h? */
+    for (y = 0; y < h; y++, dataptr++) {
+	s = *dataptr;
+	/* why trust width? */
+	for (x = 0; x < width; x++, pixels++) {
+	    if(cpp > (data_size - (s - *dataptr)))
+	    	return;
+	    strncpy(s, colors[*pixels].string, cpp); /* why trust pixel? */
+	    s += cpp;
+	}
+	*s = '\0';
+	if(offset > data_size)
+		return;
+	*(dataptr + 1) = *dataptr + offset;
+    }
+    /* duplicate some code to avoid a test in the loop */
+    s = *dataptr;
+    /* why trust width? */
+    for (x = 0; x < width; x++, pixels++) {
+	if(cpp > data_size - (s - *dataptr))
+	    	return;
+	strncpy(s, colors[*pixels].string, cpp); /* why should we trust *pixel? */
+	s += cpp;
+    }
+    *s = '\0';
+}
+
+static void
+CountExtensions(
+    XpmExtension	*ext,
+    unsigned int	 num,
+    unsigned int	*ext_size,
+    unsigned int	*ext_nlines)
+{
+    unsigned int x, y, a, size, nlines;
+    char **line;
+
+    size = 0;
+    nlines = 0;
+    for (x = 0; x < num; x++, ext++) {
+	/* 1 for the name */
+	nlines += ext->nlines + 1;
+	/* 8 = 7 (for "XPMEXT ") + 1 (for 0) */
+	size += strlen(ext->name) + 8;
+	a = ext->nlines;
+	for (y = 0, line = ext->lines; y < a; y++, line++)
+	    size += strlen(*line) + 1;
+    }
+    /* 10 and 1 are for the ending "XPMENDEXT" */
+    *ext_size = size + 10;
+    *ext_nlines = nlines + 1;
+}
+
+static void
+CreateExtensions(
+    char		**dataptr,
+    unsigned int	  data_size,
+    unsigned int	  offset,
+    XpmExtension	 *ext,
+    unsigned int	  num,
+    unsigned int	  ext_nlines)
+{
+    unsigned int x, y, a, b;
+    char **line;
+
+    *(dataptr + 1) = *dataptr + offset;
+    dataptr++;
+    a = 0;
+    for (x = 0; x < num; x++, ext++) {
+	snprintf(*dataptr, data_size, "XPMEXT %s", ext->name);
+	a++;
+	if (a < ext_nlines)
+	    *(dataptr + 1) = *dataptr + strlen(ext->name) + 8;
+	dataptr++;
+	b = ext->nlines; /* can we trust these values? */
+	for (y = 0, line = ext->lines; y < b; y++, line++) {
+	    strcpy(*dataptr, *line);
+	    a++;
+	    if (a < ext_nlines)
+		*(dataptr + 1) = *dataptr + strlen(*line) + 1;
+	    dataptr++;
+	}
+    }
+    strcpy(*dataptr, "XPMENDEXT");
+}