ICU-57132.0.1.tar.gz

[apple/icu.git] / icuSources / layout / ExtensionSubtables.cpp

diff --git a/icuSources/layout/ExtensionSubtables.cpp b/icuSources/layout/ExtensionSubtables.cpp
index 3453743e4052601340f870cf963d785b2140cca4..8ac03a2bb845f2ef1f6ec5ca8a8f3fe7edfa639d 100644 (file)
--- a/icuSources/layout/ExtensionSubtables.cpp
+++ b/icuSources/layout/ExtensionSubtables.cpp
@@ -1,7 +1,7 @@
 /*
  * %W% %E%
  *
- * (C) Copyright IBM Corp. 2002 - All Rights Reserved
+ * (C) Copyright IBM Corp. 2008-2013 - All Rights Reserved
  *
  */
 
@@ -15,18 +15,32 @@
 
 U_NAMESPACE_BEGIN
 
+// read a 32-bit value that might only be 16-bit-aligned in memory
+static inline le_uint32 READ_LONG(le_uint32 code) {
+    le_uint16* first = ((le_uint16*)&code);
+    le_uint16* second = (((le_uint16*)&code) + 1);
+    return (le_uint32)((SWAPW(*first) << 16) + SWAPW(*second));
+}
 
 // FIXME: should look at the format too... maybe have a sub-class for it?
 le_uint32 ExtensionSubtable::process(const LookupProcessor *lookupProcessor, le_uint16 lookupType,
-                                      GlyphIterator *glyphIterator, const LEFontInstance *fontInstance) const
+                                      GlyphIterator *glyphIterator, const LEFontInstance *fontInstance, LEErrorCode& success) const
 {
+    const LEReferenceTo<ExtensionSubtable> thisRef(lookupProcessor->getReference(), success); // create a reference to this
+
+    if (LE_FAILURE(success)) {
+        return 0;
+    }
+
     le_uint16 elt = SWAPW(extensionLookupType);
 
-    if (elt != lookupType) {
-        le_uint32 extOffset = SWAPL(extensionOffset);
-        LookupSubtable *subtable = (LookupSubtable *) ((char *) this + extOffset);
+    if (elt != lookupType) {      
+        le_uint32 extOffset = READ_LONG(extensionOffset);
+        LEReferenceTo<LookupSubtable> subtable(thisRef, success, extOffset);
 
-        return lookupProcessor->applySubtable(subtable, elt, glyphIterator, fontInstance);
+        if(LE_SUCCESS(success)) {
+          return lookupProcessor->applySubtable(subtable, elt, glyphIterator, fontInstance, success);
+        }
     }
 
     return 0;