<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>213719</bug_id>
          
          <creation_ts>2020-06-29 02:54:35 -0700</creation_ts>
          <short_desc>checked overflow  in WebCore::findClosestFont</short_desc>
          <delta_ts>2020-06-29 04:31:15 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>Text</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Antti Koivisto">koivisto</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>ddkilzer</cc>
    
    <cc>mmaxfield</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1667034</commentid>
    <comment_count>0</comment_count>
    <who name="Antti Koivisto">koivisto</who>
    <bug_when>2020-06-29 02:54:35 -0700</bug_when>
    <thetext>6 WebCore: WebCore::findClosestFont(WebCore::FontDatabase::InstalledFontFamily const&amp;, WebCore::FontSelectionRequest) &lt;==
        6 WebCore: WebCore::findClosestFont(WebCore::FontDatabase::InstalledFontFamily const&amp;, WebCore::FontSelectionRequest)
          6 WebCore: WebCore::FontCache::createFontPlatformData(WebCore::FontDescription const&amp;, WTF::AtomicString const&amp;, WebCore::FontTaggedSettings&lt;int&gt; const*, WebCore::FontVariantSettings const*, WebCore::FontSelectionSpecifiedCapabilities)
            6 WebCore: WebCore::FontCache::getCachedFontPlatformData(WebCore::FontDescription const&amp;, WTF::AtomicString const&amp;, WebCore::FontTaggedSettings&lt;int&gt; const*, WebCore::FontVariantSettings const*, WebCore::FontSelectionSpecifiedCapabilities, bool)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1667035</commentid>
    <comment_count>1</comment_count>
    <who name="Antti Koivisto">koivisto</who>
    <bug_when>2020-06-29 02:55:11 -0700</bug_when>
    <thetext>&lt;rdar://47765225&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1667039</commentid>
    <comment_count>2</comment_count>
      <attachid>403044</attachid>
    <who name="Antti Koivisto">koivisto</who>
    <bug_when>2020-06-29 03:04:02 -0700</bug_when>
    <thetext>Created attachment 403044
patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1667041</commentid>
    <comment_count>3</comment_count>
      <attachid>403045</attachid>
    <who name="Antti Koivisto">koivisto</who>
    <bug_when>2020-06-29 03:05:23 -0700</bug_when>
    <thetext>Created attachment 403045
patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1667054</commentid>
    <comment_count>4</comment_count>
      <attachid>403045</attachid>
    <who name="David Kilzer (:ddkilzer)">ddkilzer</who>
    <bug_when>2020-06-29 04:16:45 -0700</bug_when>
    <thetext>Comment on attachment 403045
patch

r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1667056</commentid>
    <comment_count>5</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2020-06-29 04:31:14 -0700</bug_when>
    <thetext>Committed r263654: &lt;https://trac.webkit.org/changeset/263654&gt;

All reviewed patches have been landed. Closing bug and clearing flags on attachment 403045.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>403044</attachid>
            <date>2020-06-29 03:04:02 -0700</date>
            <delta_ts>2020-06-29 03:05:23 -0700</delta_ts>
            <desc>patch</desc>
            <filename>font-filter-overflow.patch</filename>
            <type>text/plain</type>
            <size>1822</size>
            <attacher name="Antti Koivisto">koivisto</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0No
YW5nZUxvZwppbmRleCBjMjkzZjFjYmQzOS4uYjZmOGFmMTVjMmYgMTAwNjQ0Ci0tLSBhL1NvdXJj
ZS9XZWJDb3JlL0NoYW5nZUxvZworKysgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKQEAgLTEs
MyArMSwxNiBAQAorMjAyMC0wNi0yOSAgQW50dGkgS29pdmlzdG8gIDxhbnR0aUBhcHBsZS5jb20+
CisKKyAgICAgICAgY2hlY2tlZCBvdmVyZmxvdyBpbiBXZWJDb3JlOjpmaW5kQ2xvc2VzdEZvbnQK
KyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTIxMzcxOQor
ICAgICAgICA8cmRhcjovLzQ3NzY1MjI1PgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAo
T09QUyEpLgorCisgICAgICAgICogcGxhdGZvcm0vZ3JhcGhpY3MvY29jb2EvRm9udENhY2hlQ29y
ZVRleHQuY3BwOgorICAgICAgICAoV2ViQ29yZTo6ZmluZENsb3Nlc3RGb250KToKKworICAgICAg
ICBJZiBmaW5kQ2xvc2VzdEZvbnQgZG9lc24ndCBmaW5kIGFueXRoaW5nIGl0IHJldHVybnMgbm90
Rm91bmQgYW5kIGluZGV4aW5nIHRvIHRoZSB2ZWN0b3Igb3ZlcmZsb3dzLgorCiAyMDIwLTA2LTIz
ICBTZXJnaW8gVmlsbGFyIFNlbmluICA8c3ZpbGxhckBpZ2FsaWEuY29tPgogCiAgICAgICAgIFtX
ZWJYUl0gQ2hlY2sgZGV2aWNlIG9yaWVudGF0aW9uIHN1cHBvcnQgd2hlbiByZXF1ZXN0aW5nIGEg
cmVmZXJlbmNlIHNwYWNlCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9ncmFw
aGljcy9jb2NvYS9Gb250Q2FjaGVDb3JlVGV4dC5jcHAgYi9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9y
bS9ncmFwaGljcy9jb2NvYS9Gb250Q2FjaGVDb3JlVGV4dC5jcHAKaW5kZXggZGRmM2NjYzE5ZjIu
LjRhMjZlMDdjYWQ5IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGlj
cy9jb2NvYS9Gb250Q2FjaGVDb3JlVGV4dC5jcHAKKysrIGIvU291cmNlL1dlYkNvcmUvcGxhdGZv
cm0vZ3JhcGhpY3MvY29jb2EvRm9udENhY2hlQ29yZVRleHQuY3BwCkBAIC0xMTU2LDcgKzExNTYs
MTEgQEAgc3RhdGljIGNvbnN0IEZvbnREYXRhYmFzZTo6SW5zdGFsbGVkRm9udCogZmluZENsb3Nl
c3RGb250KGNvbnN0IEZvbnREYXRhYmFzZTo6SW4KICAgICBmb3IgKGF1dG8mIGZvbnQgOiBmYW1p
bHlGb250cy5pbnN0YWxsZWRGb250cykKICAgICAgICAgY2FwYWJpbGl0aWVzLnVuY2hlY2tlZEFw
cGVuZChmb250LmNhcGFiaWxpdGllcyk7CiAgICAgRm9udFNlbGVjdGlvbkFsZ29yaXRobSBmb250
U2VsZWN0aW9uQWxnb3JpdGhtKGZvbnRTZWxlY3Rpb25SZXF1ZXN0LCBjYXBhYmlsaXRpZXMsIGZh
bWlseUZvbnRzLmNhcGFiaWxpdGllcyk7Ci0gICAgcmV0dXJuICZmYW1pbHlGb250cy5pbnN0YWxs
ZWRGb250c1tmb250U2VsZWN0aW9uQWxnb3JpdGhtLmluZGV4T2ZCZXN0Q2FwYWJpbGl0aWVzKCld
OworICAgIGF1dG8gaW5kZXggPSBmb250U2VsZWN0aW9uQWxnb3JpdGhtLmluZGV4T2ZCZXN0Q2Fw
YWJpbGl0aWVzKCk7CisgICAgaWYgKGluZGV4ID09IG5vdEZvdW5kKQorICAgICAgICByZXR1cm4g
bnVsbHB0cjsKKworICAgIHJldHVybiAmZmFtaWx5Rm9udHMuaW5zdGFsbGVkRm9udHNbaW5kZXhd
OwogfQogCiBWZWN0b3I8Rm9udFNlbGVjdGlvbkNhcGFiaWxpdGllcz4gRm9udENhY2hlOjpnZXRG
b250U2VsZWN0aW9uQ2FwYWJpbGl0aWVzSW5GYW1pbHkoY29uc3QgQXRvbVN0cmluZyYgZmFtaWx5
TmFtZSwgQWxsb3dVc2VySW5zdGFsbGVkRm9udHMgYWxsb3dVc2VySW5zdGFsbGVkRm9udHMpCg==
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>403045</attachid>
            <date>2020-06-29 03:05:23 -0700</date>
            <delta_ts>2020-06-29 04:31:15 -0700</delta_ts>
            <desc>patch</desc>
            <filename>font-filter-overflow-2.patch</filename>
            <type>text/plain</type>
            <size>1830</size>
            <attacher name="Antti Koivisto">koivisto</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0No
YW5nZUxvZwppbmRleCBjMjkzZjFjYmQzOS4uMWIwZDgwNjk2ZjAgMTAwNjQ0Ci0tLSBhL1NvdXJj
ZS9XZWJDb3JlL0NoYW5nZUxvZworKysgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKQEAgLTEs
MyArMSwxNiBAQAorMjAyMC0wNi0yOSAgQW50dGkgS29pdmlzdG8gIDxhbnR0aUBhcHBsZS5jb20+
CisKKyAgICAgICAgY2hlY2tlZCBvdmVyZmxvdyBpbiBXZWJDb3JlOjpmaW5kQ2xvc2VzdEZvbnQK
KyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTIxMzcxOQor
ICAgICAgICA8cmRhcjovLzQ3NzY1MjI1PgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAo
T09QUyEpLgorCisgICAgICAgICogcGxhdGZvcm0vZ3JhcGhpY3MvY29jb2EvRm9udENhY2hlQ29y
ZVRleHQuY3BwOgorICAgICAgICAoV2ViQ29yZTo6ZmluZENsb3Nlc3RGb250KToKKworICAgICAg
ICBJZiBpbmRleE9mQmVzdENhcGFiaWxpdGllcyBkb2Vzbid0IGZpbmQgYW55dGhpbmcgaXQgcmV0
dXJucyBub3RGb3VuZCBhbmQgaW5kZXhpbmcgdG8gdGhlIHZlY3RvciBvdmVyZmxvd3MuCisKIDIw
MjAtMDYtMjMgIFNlcmdpbyBWaWxsYXIgU2VuaW4gIDxzdmlsbGFyQGlnYWxpYS5jb20+CiAKICAg
ICAgICAgW1dlYlhSXSBDaGVjayBkZXZpY2Ugb3JpZW50YXRpb24gc3VwcG9ydCB3aGVuIHJlcXVl
c3RpbmcgYSByZWZlcmVuY2Ugc3BhY2UKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL3BsYXRm
b3JtL2dyYXBoaWNzL2NvY29hL0ZvbnRDYWNoZUNvcmVUZXh0LmNwcCBiL1NvdXJjZS9XZWJDb3Jl
L3BsYXRmb3JtL2dyYXBoaWNzL2NvY29hL0ZvbnRDYWNoZUNvcmVUZXh0LmNwcAppbmRleCBkZGYz
Y2NjMTlmMi4uNGEyNmUwN2NhZDkgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJDb3JlL3BsYXRmb3Jt
L2dyYXBoaWNzL2NvY29hL0ZvbnRDYWNoZUNvcmVUZXh0LmNwcAorKysgYi9Tb3VyY2UvV2ViQ29y
ZS9wbGF0Zm9ybS9ncmFwaGljcy9jb2NvYS9Gb250Q2FjaGVDb3JlVGV4dC5jcHAKQEAgLTExNTYs
NyArMTE1NiwxMSBAQCBzdGF0aWMgY29uc3QgRm9udERhdGFiYXNlOjpJbnN0YWxsZWRGb250KiBm
aW5kQ2xvc2VzdEZvbnQoY29uc3QgRm9udERhdGFiYXNlOjpJbgogICAgIGZvciAoYXV0byYgZm9u
dCA6IGZhbWlseUZvbnRzLmluc3RhbGxlZEZvbnRzKQogICAgICAgICBjYXBhYmlsaXRpZXMudW5j
aGVja2VkQXBwZW5kKGZvbnQuY2FwYWJpbGl0aWVzKTsKICAgICBGb250U2VsZWN0aW9uQWxnb3Jp
dGhtIGZvbnRTZWxlY3Rpb25BbGdvcml0aG0oZm9udFNlbGVjdGlvblJlcXVlc3QsIGNhcGFiaWxp
dGllcywgZmFtaWx5Rm9udHMuY2FwYWJpbGl0aWVzKTsKLSAgICByZXR1cm4gJmZhbWlseUZvbnRz
Lmluc3RhbGxlZEZvbnRzW2ZvbnRTZWxlY3Rpb25BbGdvcml0aG0uaW5kZXhPZkJlc3RDYXBhYmls
aXRpZXMoKV07CisgICAgYXV0byBpbmRleCA9IGZvbnRTZWxlY3Rpb25BbGdvcml0aG0uaW5kZXhP
ZkJlc3RDYXBhYmlsaXRpZXMoKTsKKyAgICBpZiAoaW5kZXggPT0gbm90Rm91bmQpCisgICAgICAg
IHJldHVybiBudWxscHRyOworCisgICAgcmV0dXJuICZmYW1pbHlGb250cy5pbnN0YWxsZWRGb250
c1tpbmRleF07CiB9CiAKIFZlY3RvcjxGb250U2VsZWN0aW9uQ2FwYWJpbGl0aWVzPiBGb250Q2Fj
aGU6OmdldEZvbnRTZWxlY3Rpb25DYXBhYmlsaXRpZXNJbkZhbWlseShjb25zdCBBdG9tU3RyaW5n
JiBmYW1pbHlOYW1lLCBBbGxvd1VzZXJJbnN0YWxsZWRGb250cyBhbGxvd1VzZXJJbnN0YWxsZWRG
b250cykK
</data>

          </attachment>
      

    </bug>

</bugzilla>