path: root/xbmc-10.1-libpng-1.5.patch

fix building with newer libpng.  patch by Ian Stakenvicius.

https://bugs.gentoo.org/380127

--- a/xbmc/lib/cximage-6.0/CxImage/ximapng.h
+++ b/xbmc/lib/cximage-6.0/CxImage/ximapng.h
@@ -69,8 +69,13 @@
 
     static void PNGAPI user_error_fn(png_structp png_ptr,png_const_charp error_msg)
 	{
+#if PNG_LIBPNG_VER > 10399
+		strncpy((char*)png_get_error_ptr(png_ptr),error_msg,255);
+		longjmp(png_jmpbuf(png_ptr), 1);
+#else
 		strncpy((char*)png_ptr->error_ptr,error_msg,255);
 		longjmp(png_ptr->jmpbuf, 1);
+#endif
 	}
 };
 
--- a/xbmc/lib/cximage-6.0/CxImage/ximapng.cpp
+++ b/xbmc/lib/cximage-6.0/CxImage/ximapng.cpp
@@ -15,7 +15,11 @@
 void CxImagePNG::ima_png_error(png_struct *png_ptr, char *message)
 {
 	strcpy(info.szLastError,message);
+#if PNG_LIBPNG_VER > 10399
+	longjmp(png_jmpbuf(png_ptr), 1);
+#else
 	longjmp(png_ptr->jmpbuf, 1);
+#endif
 }
 ////////////////////////////////////////////////////////////////////////////////
 #if CXIMAGE_SUPPORT_DECODE
@@ -62,7 +66,11 @@
     /* Set error handling if you are using the setjmp/longjmp method (this is
     * the normal method of doing things with libpng).  REQUIRED unless you
     * set up your own error handlers in the png_create_read_struct() earlier. */
+#if PNG_LIBPNG_VER > 10399
+	if (setjmp(png_jmpbuf(png_ptr))) {
+#else
 	if (setjmp(png_ptr->jmpbuf)) {
+#endif
 		/* Free all of the memory associated with the png_ptr and info_ptr */
 		delete [] row_pointers;
 		png_destroy_read_struct(&png_ptr, &info_ptr, (png_infopp)NULL);
@@ -70,16 +78,35 @@
 	/* read the file information */
 	png_read_info(png_ptr, info_ptr);
 
+	png_uint_32 _width,_height;
+	int _bit_depth,_color_type,_interlace_type,_compression_type,_filter_type;
+#if PNG_LIBPNG_VER > 10399
+	png_get_IHDR(png_ptr,info_ptr,&_width,&_height,&_bit_depth,&_color_type,
+		&_interlace_type,&_compression_type,&_filter_type);
+#else
+	_width=info_ptr->width;
+	_height=info_ptr->height;
+	_bit_depth=info_ptr->bit_depth;
+	_color_type=info_ptr->color_type;
+	_interlace_type=info_ptr->interlace_type;
+	_compression_type=info_ptr->compression_type;
+	_filter_type=info_ptr->filter_type;
+#endif
+
 	if (info.nEscape == -1){
-		head.biWidth = info_ptr->width;
-		head.biHeight= info_ptr->height;
+		head.biWidth = _width;
+		head.biHeight= _height;
 		info.dwType = CXIMAGE_FORMAT_PNG;
+#if PNG_LIBPNG_VER > 10399
+		longjmp(png_jmpbuf(png_ptr), 1);
+#else
 		longjmp(png_ptr->jmpbuf, 1);
+#endif
 	}
 
 	/* calculate new number of channels */
 	int channels=0;
-	switch(info_ptr->color_type){
+	switch(_color_type){
 	case PNG_COLOR_TYPE_GRAY:
 	case PNG_COLOR_TYPE_PALETTE:
 		channels = 1;
@@ -101,71 +128,108 @@
 		break;
 	default:
 		strcpy(info.szLastError,"unknown PNG color type");
+#if PNG_LIBPNG_VER > 10399
+		longjmp(png_jmpbuf(png_ptr), 1);
+#else
 		longjmp(png_ptr->jmpbuf, 1);
+#endif
 	}
 
 	//find the right pixel depth used for cximage
+#if PNG_LIBPNG_VER > 10399
+	int pixel_depth = _bit_depth * png_get_channels(png_ptr,info_ptr);
+#else
 	int pixel_depth = info_ptr->pixel_depth;
+#endif
 	if (channels == 1 && pixel_depth>8) pixel_depth=8;
 	if (channels == 2) pixel_depth=8;
 	if (channels >= 3) pixel_depth=24;
 
-	if (!Create(info_ptr->width, info_ptr->height, pixel_depth, CXIMAGE_FORMAT_PNG)){
+	if (!Create(_width, _height, pixel_depth, CXIMAGE_FORMAT_PNG)){
+#if PNG_LIBPNG_VER > 10399
+		longjmp(png_jmpbuf(png_ptr), 1);
+#else
 		longjmp(png_ptr->jmpbuf, 1);
+#endif
 	}
 
 	/* get metrics */
-	switch (info_ptr->phys_unit_type)
+	png_uint_32 _x_pixels_per_unit,_y_pixels_per_unit;
+	int _phys_unit_type;
+#if PNG_LIBPNG_VER > 10399
+	png_get_pHYs(png_ptr,info_ptr,&_x_pixels_per_unit,&_y_pixels_per_unit,&_phys_unit_type);
+#else
+	_x_pixels_per_unit=info_ptr->x_pixels_per_unit;
+	_y_pixels_per_unit=info_ptr->y_pixels_per_unit;
+	_phys_unit_type=info_ptr->phys_unit_type;
+#endif
+	switch (_phys_unit_type)
 	{
 	case PNG_RESOLUTION_UNKNOWN:
-		SetXDPI(info_ptr->x_pixels_per_unit);
-		SetYDPI(info_ptr->y_pixels_per_unit);
+		SetXDPI(_x_pixels_per_unit);
+		SetYDPI(_y_pixels_per_unit);
 		break;
 	case PNG_RESOLUTION_METER:
-		SetXDPI((long)floor(info_ptr->x_pixels_per_unit * 254.0 / 10000.0 + 0.5));
-		SetYDPI((long)floor(info_ptr->y_pixels_per_unit * 254.0 / 10000.0 + 0.5));
+		SetXDPI((long)floor(_x_pixels_per_unit * 254.0 / 10000.0 + 0.5));
+		SetYDPI((long)floor(_y_pixels_per_unit * 254.0 / 10000.0 + 0.5));
 		break;
 	}
 
-	if (info_ptr->num_palette>0){
-		SetPalette((rgb_color*)info_ptr->palette,info_ptr->num_palette);
-		SetClrImportant(info_ptr->num_palette);
-	} else if (info_ptr->bit_depth ==2) { //<DP> needed for 2 bpp grayscale PNGs
+	int _num_palette;
+	png_colorp _palette;
+#if PNG_LIBPNG_VER > 10399
+	png_get_PLTE(png_ptr,info_ptr,&_palette,&_num_palette);
+#else
+	_num_palette=info_ptr->num_palette;
+	_palette=info_ptr->palette;
+#endif
+	if (_num_palette>0){
+		SetPalette((rgb_color*)_palette,_num_palette);
+		SetClrImportant(_num_palette);
+	} else if (_bit_depth ==2) { //<DP> needed for 2 bpp grayscale PNGs
 		SetPaletteColor(0,0,0,0);
 		SetPaletteColor(1,85,85,85);
 		SetPaletteColor(2,170,170,170);
 		SetPaletteColor(3,255,255,255);
 	} else SetGrayPalette(); //<DP> needed for grayscale PNGs
 	
-	int nshift = max(0,(info_ptr->bit_depth>>3)-1)<<3;
+	int nshift = max(0,(_bit_depth>>3)-1)<<3;
 
-	if (info_ptr->num_trans!=0){ //palette transparency
-		if (info_ptr->num_trans==1){
-			if (info_ptr->color_type == PNG_COLOR_TYPE_PALETTE){
+	png_bytep _trans_alpha;
+	int _num_trans;
+	png_color_16p _trans_color;
+#if PNG_LIBPNG_VER > 10399
+	png_get_tRNS(png_ptr,info_ptr,&_trans_alpha,&_num_trans,&_trans_color);
+#else
+	_num_trans=info_ptr->num_trans;
+#endif
+	if (_num_trans!=0){ //palette transparency
+		if (_num_trans==1){
+			if (_color_type == PNG_COLOR_TYPE_PALETTE){
 #if PNG_LIBPNG_VER > 10399
-				info.nBkgndIndex = info_ptr->trans_color.index;
+				info.nBkgndIndex = _trans_color->index;
 #else
 				info.nBkgndIndex = info_ptr->trans_values.index;
 #endif
 			} else{
 #if PNG_LIBPNG_VER > 10399
-				info.nBkgndIndex = info_ptr->trans_color.gray>>nshift;
+				info.nBkgndIndex = _trans_color->gray>>nshift;
 #else
 				info.nBkgndIndex = info_ptr->trans_values.gray>>nshift;
 #endif
 			}
 		}
-		if (info_ptr->num_trans>1){
+		if (_num_trans>1){
 			RGBQUAD* pal=GetPalette();
 			if (pal){
 				DWORD ip;
-				for (ip=0;ip<min(head.biClrUsed,(unsigned long)info_ptr->num_trans);ip++)
+				for (ip=0;ip<min(head.biClrUsed,(unsigned long)_num_trans);ip++)
 #if PNG_LIBPNG_VER > 10399
-					pal[ip].rgbReserved=info_ptr->trans_alpha[ip];
+					pal[ip].rgbReserved=_trans_alpha[ip];
 #else
 					pal[ip].rgbReserved=info_ptr->trans[ip];
 #endif
-				for (ip=info_ptr->num_trans;ip<head.biClrUsed;ip++){
+				for (ip=_num_trans;ip<head.biClrUsed;ip++){
 					pal[ip].rgbReserved=255;
 				}
 				info.bAlphaPaletteEnabled=true;
@@ -174,14 +238,12 @@
 	}
 
 	if (channels == 3){ //check RGB binary transparency
-		png_bytep trans;
-		int num_trans;
-		png_color_16 *image_background;
-		if (png_get_tRNS(png_ptr, info_ptr, &trans, &num_trans, &image_background)){
-#if PNG_LIBPNG_VER > 10399
-			info.nBkgndColor.rgbRed   = (BYTE)(info_ptr->trans_color.red>>nshift);
-			info.nBkgndColor.rgbGreen = (BYTE)(info_ptr->trans_color.green>>nshift);
-			info.nBkgndColor.rgbBlue  = (BYTE)(info_ptr->trans_color.blue>>nshift);
+		/* seems unnecessary to call again, but the conditional must be important so... */
+		if (png_get_tRNS(png_ptr,info_ptr,&_trans_alpha,&_num_trans,&_trans_color)){
+#if PNG_LIBPNG_VER > 10399
+			info.nBkgndColor.rgbRed   = (BYTE)(_trans_color->red>>nshift);
+			info.nBkgndColor.rgbGreen = (BYTE)(_trans_color->green>>nshift);
+			info.nBkgndColor.rgbBlue  = (BYTE)(_trans_color->blue>>nshift);
 #else
 			info.nBkgndColor.rgbRed   = (BYTE)(info_ptr->trans_values.red>>nshift);
 			info.nBkgndColor.rgbGreen = (BYTE)(info_ptr->trans_values.green>>nshift);
@@ -202,15 +264,24 @@
 	}
 
 	// <vho> - flip the RGB pixels to BGR (or RGBA to BGRA)
-	if (info_ptr->color_type & PNG_COLOR_MASK_COLOR){
+	if (_color_type & PNG_COLOR_MASK_COLOR){
 		png_set_bgr(png_ptr);
 	}
 
 	// <vho> - handle cancel
-	if (info.nEscape) longjmp(png_ptr->jmpbuf, 1);
+	if (info.nEscape)
+#if PNG_LIBPNG_VER > 10399
+		longjmp(png_jmpbuf(png_ptr), 1);
+#else
+		longjmp(png_ptr->jmpbuf, 1);
+#endif
 
 	// row_bytes is the width x number of channels x (bit-depth / 8)
+#if PNG_LIBPNG_VER > 10399
+	row_pointers = new BYTE[png_get_rowbytes(png_ptr,info_ptr) + 8];
+#else
 	row_pointers = new BYTE[info_ptr->rowbytes + 8];
+#endif
 
 	// turn on interlace handling
 	int number_passes = png_set_interlace_handling(png_ptr);
@@ -221,8 +292,12 @@
 		SetCodecOption(0);
 	}
 
-	int chan_offset = info_ptr->bit_depth >> 3;
+	int chan_offset = _bit_depth >> 3;
+#if PNG_LIBPNG_VER > 10399
+	int pixel_offset = (_bit_depth * png_get_channels(png_ptr,info_ptr)) >> 3;
+#else
 	int pixel_offset = info_ptr->pixel_depth >> 3;
+#endif
 
 	for (int pass=0; pass < number_passes; pass++) {
 		iter.Upset();
@@ -230,7 +305,12 @@
 		do	{
 
 			// <vho> - handle cancel
-			if (info.nEscape) longjmp(png_ptr->jmpbuf, 1);
+			if (info.nEscape)
+#if PNG_LIBPNG_VER > 10399
+				longjmp(png_jmpbuf(png_ptr), 1);
+#else
+				longjmp(png_ptr->jmpbuf, 1);
+#endif
 
 #if CXIMAGE_SUPPORT_ALPHA	// <vho>
 			if (AlphaIsValid()) {
@@ -241,7 +321,7 @@
 				BYTE* prow= iter.GetRow(ay);
 
 				//recover data from previous scan
-				if (info_ptr->interlace_type && pass>0 && pass!=7){
+				if (_interlace_type && pass>0 && pass!=7){
 					for(ax=0;ax<head.biWidth;ax++){
 						long px = ax * pixel_offset;
 						if (channels == 2){
@@ -278,10 +358,14 @@
 #endif // CXIMAGE_SUPPORT_ALPHA		// vho
 			{
 				//recover data from previous scan
-				if (info_ptr->interlace_type && pass>0){
+				if (_interlace_type && pass>0){
+#if PNG_LIBPNG_VER > 10399
+					iter.GetRow(row_pointers, png_get_rowbytes(png_ptr,info_ptr));
+#else
 					iter.GetRow(row_pointers, info_ptr->rowbytes);
+#endif
 					//re-expand buffer for images with bit depth > 8
-					if (info_ptr->bit_depth > 8){
+					if (_bit_depth > 8){
 						for(long ax=(head.biWidth*channels-1);ax>=0;ax--)
 							row_pointers[ax*chan_offset] = row_pointers[ax];
 					}
@@ -291,15 +375,19 @@
 				png_read_row(png_ptr, row_pointers, NULL);
 
 				//shrink 16 bit depth images down to 8 bits
-				if (info_ptr->bit_depth > 8){
+				if (_bit_depth > 8){
 					for(long ax=0;ax<(head.biWidth*channels);ax++)
 						row_pointers[ax] = row_pointers[ax*chan_offset];
 				}
 
 				//copy the pixels
+#if PNG_LIBPNG_VER > 10399
+				iter.SetRow(row_pointers, png_get_rowbytes(png_ptr,info_ptr));
+#else
 				iter.SetRow(row_pointers, info_ptr->rowbytes);
+#endif
 				//<DP> expand 2 bpp images only in the last pass
-				if (info_ptr->bit_depth==2 && pass==(number_passes-1))
+				if (_bit_depth==2 && pass==(number_passes-1))
 					expand2to4bpp(iter.GetRow());
 
 				//go on
@@ -361,9 +449,13 @@
    /* Set error handling.  REQUIRED if you aren't supplying your own
     * error hadnling functions in the png_create_write_struct() call.
     */
+#if PNG_LIBPNG_VER > 10399
+	if (setjmp(png_jmpbuf(png_ptr))){
+#else
 	if (setjmp(png_ptr->jmpbuf)){
 		/* If we get here, we had a problem reading the file */
 		if (info_ptr->palette) free(info_ptr->palette);
+#endif
 		png_destroy_write_struct(&png_ptr,  (png_infopp)&info_ptr);
 		cx_throw("Error saving PNG file");
 	}
@@ -372,9 +464,23 @@
 	//png_init_io(png_ptr, hFile);
 
 	// use custom I/O functions
-    png_set_write_fn(png_ptr,hFile,/*(png_rw_ptr)*/user_write_data,/*(png_flush_ptr)*/user_flush_data);
+	png_set_write_fn(png_ptr,hFile,/*(png_rw_ptr)*/user_write_data,/*(png_flush_ptr)*/user_flush_data);
 
 	/* set the file information here */
+#if PNG_LIBPNG_VER > 10399
+	/* use variables to hold the values so it isnt necessary to png_get them later */
+	png_uint_32 _width,_height;
+	int _bit_depth,_color_type,_interlace_type,_compression_type,_filter_type;
+	png_byte _channels,_pixel_depth;
+
+	_width = GetWidth();
+	_height = GetHeight();
+	_pixel_depth = (BYTE)GetBpp();
+	_channels = (GetBpp()>8) ? (BYTE)3: (BYTE)1;
+	_bit_depth = (BYTE)(GetBpp()/_channels);
+	_compression_type = PNG_COMPRESSION_TYPE_DEFAULT;
+	_filter_type = PNG_FILTER_TYPE_DEFAULT;
+#else
 	info_ptr->width = GetWidth();
 	info_ptr->height = GetHeight();
 	info_ptr->pixel_depth = (BYTE)GetBpp();
@@ -382,13 +488,22 @@
 	info_ptr->bit_depth = (BYTE)(GetBpp()/info_ptr->channels);
 	info_ptr->compression_type = info_ptr->filter_type = 0;
 	info_ptr->valid = 0;
+#endif
 
 	switch(GetCodecOption(CXIMAGE_FORMAT_PNG)){
 	case 1:
+#if PNG_LIBPNG_VER > 10399
+		_interlace_type = PNG_INTERLACE_ADAM7;
+#else
 		info_ptr->interlace_type = PNG_INTERLACE_ADAM7;
+#endif
 		break;
 	default:
+#if PNG_LIBPNG_VER > 10399
+		_interlace_type = PNG_INTERLACE_NONE;
+#else
 		info_ptr->interlace_type = PNG_INTERLACE_NONE;
+#endif
 	}
 
 	/* set compression level */
@@ -398,22 +513,47 @@
 
 	if (GetNumColors()){
 		if (bGrayScale){
+#if PNG_LIBPNG_VER > 10399
+			_color_type = PNG_COLOR_TYPE_GRAY;
+#else
 			info_ptr->color_type = PNG_COLOR_TYPE_GRAY;
+#endif
 		} else {
+#if PNG_LIBPNG_VER > 10399
+			_color_type = PNG_COLOR_TYPE_PALETTE;
+#else
 			info_ptr->color_type = PNG_COLOR_TYPE_PALETTE;
+#endif
 		}
 	} else {
+#if PNG_LIBPNG_VER > 10399
+		_color_type = PNG_COLOR_TYPE_RGB;
+#else
 		info_ptr->color_type = PNG_COLOR_TYPE_RGB;
+#endif
 	}
 #if CXIMAGE_SUPPORT_ALPHA
 	if (AlphaIsValid()){
+#if PNG_LIBPNG_VER > 10399
+		_color_type |= PNG_COLOR_MASK_ALPHA;
+		_channels++;
+		_bit_depth = 8;
+		_pixel_depth += 8;
+#else
 		info_ptr->color_type |= PNG_COLOR_MASK_ALPHA;
 		info_ptr->channels++;
 		info_ptr->bit_depth = 8;
 		info_ptr->pixel_depth += 8;
+#endif
 	}
 #endif
 
+#if PNG_LIBPNG_VER > 10399
+	/* set the header here, since we're done modifying these values */
+	png_set_IHDR(png_ptr,info_ptr,_width,_height,_bit_depth,_color_type,_interlace_type,
+		_compression_type,_filter_type);
+#endif
+
 	/* set background */
 	png_color_16 image_background={ 0, 255, 255, 255, 0 };
 	RGBQUAD tc = GetTransColor();
@@ -427,22 +567,24 @@
 	/* set metrics */
 	png_set_pHYs(png_ptr, info_ptr, head.biXPelsPerMeter, head.biYPelsPerMeter, PNG_RESOLUTION_METER);
 
+#if PNG_LIBPNG_VER <= 10399
 	png_set_IHDR(png_ptr, info_ptr, info_ptr->width, info_ptr->height, info_ptr->bit_depth,
 				info_ptr->color_type, info_ptr->interlace_type,
 				PNG_COMPRESSION_TYPE_BASE, PNG_FILTER_TYPE_BASE);
+#endif
 
 	//<DP> simple transparency
 	if (info.nBkgndIndex >= 0){
-		info_ptr->num_trans = 1;
-		info_ptr->valid |= PNG_INFO_tRNS;
 #if PNG_LIBPNG_VER > 10399
-		info_ptr->trans_alpha = trans;
-		info_ptr->trans_color.index = (BYTE)info.nBkgndIndex;
-		info_ptr->trans_color.red   = tc.rgbRed;
-		info_ptr->trans_color.green = tc.rgbGreen;
-		info_ptr->trans_color.blue  = tc.rgbBlue;
-		info_ptr->trans_color.gray  = info_ptr->trans_color.index;
+		png_color_16 _trans_color;
+		_trans_color.index = (BYTE)info.nBkgndIndex;
+		_trans_color.red   = tc.rgbRed;
+		_trans_color.green = tc.rgbGreen;
+		_trans_color.blue  = tc.rgbBlue;
+		_trans_color.gray  = _trans_color.index;
 #else
+		info_ptr->num_trans = 1;
+		info_ptr->valid |= PNG_INFO_tRNS;
 		info_ptr->trans = trans;
 		info_ptr->trans_values.index = (BYTE)info.nBkgndIndex;
 		info_ptr->trans_values.red   = tc.rgbRed;
@@ -454,34 +596,53 @@
 		// the transparency indexes start from 0 for non grayscale palette
 		if (!bGrayScale && head.biClrUsed && info.nBkgndIndex)
 			SwapIndex(0,(BYTE)info.nBkgndIndex);
+
+#if PNG_LIBPNG_VER > 10399
+		png_set_tRNS(png_ptr,info_ptr,(png_bytep)trans,1,&_trans_color);
+#endif
 	}
 
 	/* set the palette if there is one */
+#if PNG_LIBPNG_VER > 10399
+	png_colorp _palette;
+#endif
 	if (GetPalette()){
+#if PNG_LIBPNG_VER <= 10399
 		if (!bGrayScale){
 			info_ptr->valid |= PNG_INFO_PLTE;
 		}
+#endif
 
 		int nc = GetClrImportant();
 		if (nc==0) nc = GetNumColors();
 
+		// copy the palette colors
+#if PNG_LIBPNG_VER > 10399
+		_palette = new png_color[nc];
+#else
+		info_ptr->palette = new png_color[nc];
+		info_ptr->num_palette = (png_uint_16) nc;
+#endif
+		for (int i=0; i<nc; i++)
+#if PNG_LIBPNG_VER > 10399
+			GetPaletteColor(i, &_palette[i].red, &_palette[i].green, &_palette[i].blue);
+
+		png_set_PLTE(png_ptr,info_ptr,_palette,nc);
+#else
+			GetPaletteColor(i, &info_ptr->palette[i].red, &info_ptr->palette[i].green, &info_ptr->palette[i].blue);
+#endif
+
 		if (info.bAlphaPaletteEnabled){
 			for(WORD ip=0; ip<nc;ip++)
 				trans[ip]=GetPaletteColor((BYTE)ip).rgbReserved;
-			info_ptr->num_trans = (WORD)nc;
-			info_ptr->valid |= PNG_INFO_tRNS;
 #if PNG_LIBPNG_VER > 10399
-			info_ptr->trans_alpha = trans;
+			png_set_tRNS(png_ptr,info_ptr,(png_bytep)trans,nc,NULL);
 #else
+			info_ptr->num_trans = (WORD)nc;
+			info_ptr->valid |= PNG_INFO_tRNS;
 			info_ptr->trans = trans;
 #endif
 		}
-
-		// copy the palette colors
-		info_ptr->palette = new png_color[nc];
-		info_ptr->num_palette = (png_uint_16) nc;
-		for (int i=0; i<nc; i++)
-			GetPaletteColor(i, &info_ptr->palette[i].red, &info_ptr->palette[i].green, &info_ptr->palette[i].blue);
 	}  
 
 #if CXIMAGE_SUPPORT_ALPHA	// <vho>
@@ -495,8 +656,12 @@
 	}	}	}
 #endif // CXIMAGE_SUPPORT_ALPHA	// <vho>
 
+#if PNG_LIBPNG_VER > 10399
+	int row_size = max(info.dwEffWidth, (_width * _channels * _bit_depth / 8));
+#else
 	int row_size = max(info.dwEffWidth, info_ptr->width*info_ptr->channels*(info_ptr->bit_depth/8));
 	info_ptr->rowbytes = row_size;
+#endif
 	BYTE *row_pointers = new BYTE[row_size];
 
 	/* write the file information */
@@ -514,7 +679,11 @@
 			if (AlphaIsValid()){
 				for (long ax=head.biWidth-1; ax>=0;ax--){
 					c = BlindGetPixelColor(ax,ay);
+#if PNG_LIBPNG_VER > 10399
+					int px = ax * _channels;
+#else
 					int px = ax * info_ptr->channels;
+#endif
 					if (!bGrayScale){
 						row_pointers[px++]=c.rgbRed;
 						row_pointers[px++]=c.rgbGreen;
@@ -529,7 +698,11 @@
 #endif //CXIMAGE_SUPPORT_ALPHA	// <vho>
 			{
 				iter.GetRow(row_pointers, row_size);
+#if PNG_LIBPNG_VER > 10399
+				if (_color_type == PNG_COLOR_TYPE_RGB) //HACK BY OP
+#else
 				if (info_ptr->color_type == PNG_COLOR_TYPE_RGB) //HACK BY OP
+#endif
 					RGBtoBGR(row_pointers, row_size);
 				png_write_row(png_ptr, row_pointers);
 			}
@@ -546,9 +719,14 @@
 	png_write_end(png_ptr, info_ptr);
 
 	/* if you malloced the palette, free it here */
+#if PNG_LIBPNG_VER > 10399
+	if (_palette){
+		delete [] (_palette);
+#else
 	if (info_ptr->palette){
 		delete [] (info_ptr->palette);
 		info_ptr->palette = NULL;
+#endif
 	}
 
 	/* clean up after the write, and free any memory allocated */