[pkg-wine-party] [PATCH v2] sfnt2fnt: Fix broken .fon files on architectures that are big-endian or where char is unsigned.
Hilko Bengen
bengen at debian.org
Sat Jun 9 22:40:38 UTC 2012
---
tools/sfnt2fnt.c | 111 +++++++++++++++++++++++++++++++++++++++---------------
1 file changed, 80 insertions(+), 31 deletions(-)
diff --git a/tools/sfnt2fnt.c b/tools/sfnt2fnt.c
index df43ec0..31b9953 100644
--- a/tools/sfnt2fnt.c
+++ b/tools/sfnt2fnt.c
@@ -46,6 +46,8 @@
#include "pshpack1.h"
+#include <endian.h>
+
typedef struct
{
INT16 dfType;
@@ -378,7 +380,7 @@ static struct fontinfo *fill_fontinfo( const char *face_name, int ppem, int enc,
size_table = (bitmapSizeTable_t *)(eblc + 1);
for(i = 0; i < num_sizes; i++)
{
- if(size_table->hori.ascender - size_table->hori.descender == ppem)
+ if( (signed char)size_table->hori.ascender - (signed char)size_table->hori.descender == ppem)
{
ascent = size_table->hori.ascender;
break;
@@ -570,11 +572,50 @@ static struct fontinfo *fill_fontinfo( const char *face_name, int ppem, int enc,
return info;
}
+static void adjust_fontinfo( FONTINFO16 * fi )
+{
+ fi->dfType = htole16(fi->dfType);
+ fi->dfPoints = htole16(fi->dfPoints);
+ fi->dfVertRes = htole16(fi->dfVertRes);
+ fi->dfHorizRes = htole16(fi->dfHorizRes);
+ fi->dfAscent = htole16(fi->dfAscent);
+ fi->dfInternalLeading = htole16(fi->dfInternalLeading);
+ fi->dfExternalLeading = htole16(fi->dfExternalLeading);
+ fi->dfWeight = htole16(fi->dfWeight);
+ fi->dfPixWidth = htole16(fi->dfPixWidth);
+ fi->dfPixHeight = htole16(fi->dfPixHeight);
+ fi->dfAvgWidth = htole16(fi->dfAvgWidth);
+ fi->dfMaxWidth = htole16(fi->dfMaxWidth);
+ fi->dfWidthBytes = htole16(fi->dfWidthBytes);
+ fi->dfAspace = htole16(fi->dfAspace);
+ fi->dfBspace = htole16(fi->dfBspace);
+ fi->dfCspace = htole16(fi->dfCspace);
+ fi->dfDevice = htole32(fi->dfDevice);
+ fi->dfFace = htole32(fi->dfFace);
+ fi->dfBitsPointer = htole32(fi->dfBitsPointer);
+ fi->dfBitsOffset = htole32(fi->dfBitsOffset);
+ fi->dfFlags = htole32(fi->dfFlags);
+ fi->dfColorPointer = htole32(fi->dfColorPointer);
+}
+
static void write_fontinfo( const struct fontinfo *info, FILE *fp )
{
- fwrite( &info->hdr, sizeof(info->hdr), 1, fp );
- fwrite( info->dfCharTable + info->hdr.fi.dfFirstChar, sizeof(*info->dfCharTable),
- ((unsigned char)info->hdr.fi.dfLastChar - (unsigned char)info->hdr.fi.dfFirstChar) + 3, fp );
+ FNT_HEADER tmp_hdr;
+ memcpy(&tmp_hdr, &info->hdr, sizeof(info->hdr));
+ tmp_hdr.dfVersion = htole16(tmp_hdr.dfVersion);
+ tmp_hdr.dfSize = htole32(tmp_hdr.dfSize);
+ adjust_fontinfo(&(tmp_hdr.fi));
+ fwrite( &tmp_hdr, sizeof(info->hdr), 1, fp );
+ int num_chars = ((unsigned char)info->hdr.fi.dfLastChar - (unsigned char)info->hdr.fi.dfFirstChar) + 3;
+
+ CHAR_TABLE_ENTRY tmp_chartable[num_chars];
+ memcpy(&tmp_chartable, info->dfCharTable + info->hdr.fi.dfFirstChar, num_chars * sizeof(CHAR_TABLE_ENTRY));
+ int i;
+ for (i=0; i < num_chars; ++i) {
+ tmp_chartable[i].width = htole16(tmp_chartable[i].width);
+ tmp_chartable[i].offset = htole32(tmp_chartable[i].offset);
+ }
+ fwrite( tmp_chartable, sizeof(CHAR_TABLE_ENTRY), num_chars, fp );
fwrite( info->data, info->hdr.dfSize - info->hdr.fi.dfBitsOffset, 1, fp );
}
@@ -696,21 +737,21 @@ int main(int argc, char **argv)
non_resident_name_off = sizeof(MZ_hdr) + module_ref_off + sizeof(align);
memset(&NE_hdr, 0, sizeof(NE_hdr));
- NE_hdr.ne_magic = 0x454e;
+ NE_hdr.ne_magic = htole16(0x454e);
NE_hdr.ne_ver = 5;
NE_hdr.ne_rev = 1;
- NE_hdr.ne_flags = NE_FFLAGS_LIBMODULE | NE_FFLAGS_GUI;
- NE_hdr.ne_cbnrestab = non_resident_name_len;
- NE_hdr.ne_segtab = sizeof(NE_hdr);
- NE_hdr.ne_rsrctab = sizeof(NE_hdr);
- NE_hdr.ne_restab = resident_name_off;
- NE_hdr.ne_modtab = module_ref_off;
- NE_hdr.ne_imptab = module_ref_off;
+ NE_hdr.ne_flags = htole16(NE_FFLAGS_LIBMODULE | NE_FFLAGS_GUI);
+ NE_hdr.ne_cbnrestab = htole16(non_resident_name_len);
+ NE_hdr.ne_segtab = htole16(sizeof(NE_hdr));
+ NE_hdr.ne_rsrctab = htole16(sizeof(NE_hdr));
+ NE_hdr.ne_restab = htole16(resident_name_off);
+ NE_hdr.ne_modtab = htole16(module_ref_off);
+ NE_hdr.ne_imptab = htole16(module_ref_off);
NE_hdr.ne_enttab = NE_hdr.ne_modtab;
- NE_hdr.ne_nrestab = non_resident_name_off;
- NE_hdr.ne_align = 4;
+ NE_hdr.ne_nrestab = htole32(non_resident_name_off);
+ NE_hdr.ne_align = htole16(4);
NE_hdr.ne_exetyp = NE_OSFLAGS_WINDOWS;
- NE_hdr.ne_expver = 0x400;
+ NE_hdr.ne_expver = htole16(0x400);
fontdir_off = (non_resident_name_off + non_resident_name_len + 15) & ~0xf;
font_off = (fontdir_off + fontdir_len + 15) & ~0x0f;
@@ -749,34 +790,34 @@ int main(int argc, char **argv)
fwrite(MZ_hdr, sizeof(MZ_hdr), 1, ofp);
fwrite(&NE_hdr, sizeof(NE_hdr), 1, ofp);
- align = 4;
+ align = htole16(4);
fwrite(&align, sizeof(align), 1, ofp);
- rc_type.type_id = NE_RSCTYPE_FONTDIR;
- rc_type.count = 1;
+ rc_type.type_id = htole16(NE_RSCTYPE_FONTDIR);
+ rc_type.count = htole16(1);
rc_type.resloader = 0;
fwrite(&rc_type, sizeof(rc_type), 1, ofp);
- rc_name.offset = fontdir_off >> 4;
- rc_name.length = (fontdir_len + 15) >> 4;
- rc_name.flags = NE_SEGFLAGS_MOVEABLE | NE_SEGFLAGS_PRELOAD;
- rc_name.id = resident_name_off - sizeof("FONTDIR") - NE_hdr.ne_rsrctab;
+ rc_name.offset = htole16(fontdir_off >> 4);
+ rc_name.length = htole16((fontdir_len + 15) >> 4);
+ rc_name.flags = htole16(NE_SEGFLAGS_MOVEABLE | NE_SEGFLAGS_PRELOAD);
+ rc_name.id = htole16(resident_name_off - sizeof("FONTDIR") - sizeof(NE_hdr));
rc_name.handle = 0;
rc_name.usage = 0;
fwrite(&rc_name, sizeof(rc_name), 1, ofp);
- rc_type.type_id = NE_RSCTYPE_FONT;
- rc_type.count = num_files;
+ rc_type.type_id = htole16(NE_RSCTYPE_FONT);
+ rc_type.count = htole16(num_files);
rc_type.resloader = 0;
fwrite(&rc_type, sizeof(rc_type), 1, ofp);
for(res = first_res | 0x8000, i = 0; i < num_files; i++, res++) {
int len = (info[i]->hdr.dfSize + 15) & ~0xf;
- rc_name.offset = font_off >> 4;
- rc_name.length = len >> 4;
- rc_name.flags = NE_SEGFLAGS_MOVEABLE | NE_SEGFLAGS_SHAREABLE | NE_SEGFLAGS_DISCARDABLE;
- rc_name.id = res;
+ rc_name.offset = htole16(font_off >> 4);
+ rc_name.length = htole16(len >> 4);
+ rc_name.flags = htole16(NE_SEGFLAGS_MOVEABLE | NE_SEGFLAGS_SHAREABLE | NE_SEGFLAGS_DISCARDABLE);
+ rc_name.id = htole16(res);
rc_name.handle = 0;
rc_name.usage = 0;
fwrite(&rc_name, sizeof(rc_name), 1, ofp);
@@ -812,12 +853,20 @@ int main(int argc, char **argv)
fputc(0x00, ofp);
/* FONTDIR resource */
- fwrite(&num_files, sizeof(num_files), 1, ofp);
+ short tmp16 = htole16(num_files);
+ fwrite(&tmp16, sizeof(tmp16), 1, ofp);
for(res = first_res, i = 0; i < num_files; i++, res++) {
const char *name = get_face_name( info[i] );
- fwrite(&res, sizeof(res), 1, ofp);
- fwrite(&info[i]->hdr, FIELD_OFFSET(FNT_HEADER,fi.dfBitsOffset), 1, ofp);
+ tmp16 = htole16(res);
+ fwrite(&tmp16, sizeof(tmp16), 1, ofp);
+ int sz = FIELD_OFFSET(FNT_HEADER,fi.dfBitsOffset);
+ FNT_HEADER tmp_hdr;
+ memcpy(&tmp_hdr, &info[i]->hdr, sz);
+ tmp_hdr.dfVersion = htole16(tmp_hdr.dfVersion);
+ tmp_hdr.dfSize = htole32(tmp_hdr.dfSize);
+ adjust_fontinfo(&(tmp_hdr.fi));
+ fwrite(&tmp_hdr, FIELD_OFFSET(FNT_HEADER,fi.dfBitsOffset), 1, ofp);
fputc(0x00, ofp);
fwrite(name, strlen(name) + 1, 1, ofp);
}
--
1.7.10
More information about the pkg-wine-party
mailing list