[PATCH 0/1] MR9480: win32u: Fix broken otmfsType flag generation.

[Patrick Hibbs (＠hibbsncc1701)]

Freetype uses a function to get the licensing flags, FT_Get_FSType_Flags(), and it's non-trivial.
  (See also: https://freetype.org/freetype2/docs/reference/ft2-information_retrieval.html...)
  Use FF's function to get the proper flags, and then set otmfsType based on them.

  Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=58987

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/9480

[PATCH 1/1] win32u: Fix broken otmfsType flag generation.

[Patrick Hibbs]

From: Patrick Hibbs <hibbsncc1701(a)gmail.com>

  Freetype uses a function to get the licensing flags, FT_Get_FSType_Flags(), and it's non-trivial.
  (See also: https://freetype.org/freetype2/docs/reference/ft2-information_retrieval.html...)
  Use FF's function to get the proper flags, and then set otmfsType based on them.

  Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=58987
---
 dlls/win32u/freetype.c | 13 ++++++++++++-
 1 file changed, 12 insertions(+), 1 deletion(-)

diff --git a/dlls/win32u/freetype.c b/dlls/win32u/freetype.c
index aeae190bc46..03c6389216e 100644
--- a/dlls/win32u/freetype.c
+++ b/dlls/win32u/freetype.c
@@ -92,6 +92,7 @@ static void *ft_handle = NULL;
 MAKE_FUNCPTR(FT_Done_Face);
 MAKE_FUNCPTR(FT_Get_Char_Index);
 MAKE_FUNCPTR(FT_Get_First_Char);
+MAKE_FUNCPTR(FT_Get_FSType_Flags);
 MAKE_FUNCPTR(FT_Get_Next_Char);
 MAKE_FUNCPTR(FT_Get_Sfnt_Name);
 MAKE_FUNCPTR(FT_Get_Sfnt_Name_Count);
@@ -1470,6 +1471,7 @@ static BOOL init_freetype(void)
     LOAD_FUNCPTR(FT_Done_Face)
     LOAD_FUNCPTR(FT_Get_Char_Index)
     LOAD_FUNCPTR(FT_Get_First_Char)
+    LOAD_FUNCPTR(FT_Get_FSType_Flags)
     LOAD_FUNCPTR(FT_Get_Next_Char)
     LOAD_FUNCPTR(FT_Get_Sfnt_Name)
     LOAD_FUNCPTR(FT_Get_Sfnt_Name_Count)
@@ -3532,7 +3534,16 @@ static BOOL freetype_set_outline_text_metrics( struct gdi_font *font )
     if (font->fake_bold)
         font->otm.otmfsSelection |= 1 << 5;
     /* Only return valid bits that define embedding and subsetting restrictions */
-    font->otm.otmfsType = pOS2->fsType & 0x30e;
+    font->otm.otmfsType = 0; /* FT_FSTYPE_INSTALLABLE_EMBEDDING */
+    int ff_fsflags = pFT_Get_FSType_Flags(ft_face);
+    if (ff_fsflags && FT_FSTYPE_RESTRICTED_LICENSE_EMBEDDING ||
+        ff_fsflags && FT_FSTYPE_NO_SUBSETTING ||
+        ff_fsflags && FT_FSTYPE_BITMAP_EMBEDDING_ONLY)
+        font->otm.otmfsType |= 1 << 1; /* Bit 1: No-embedding flag. */
+    if (ff_fsflags && FT_FSTYPE_PREVIEW_AND_PRINT_EMBEDDING)
+        font->otm.otmfsType |= 1 << 2; /* Bit 2: Read-Only flag. */
+    if (ff_fsflags && FT_FSTYPE_EDITABLE_EMBEDDING)
+        font->otm.otmfsType |= 1 << 3; /* Bit 3: Editable flag? */
     font->otm.otmsCharSlopeRise = pHori->caret_Slope_Rise;
     font->otm.otmsCharSlopeRun = pHori->caret_Slope_Run;
     font->otm.otmItalicAngle = 0; /* POST table */
-- 
GitLab

https://gitlab.winehq.org/wine/wine/-/merge_requests/9480