<?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>273045</bug_id>
          
          <creation_ts>2024-04-21 09:51:09 -0700</creation_ts>
          <short_desc>[WK1] WebKit XML parsing can deny external entity loads from other in-process libxml2 clients</short_desc>
          <delta_ts>2024-05-01 09:08:32 -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>XML</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>
          <dependson>259235</dependson>
          <blocked>273553</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="David Kilzer (:ddkilzer)">ddkilzer</reporter>
          <assigned_to name="David Kilzer (:ddkilzer)">ddkilzer</assigned_to>
          <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>2030095</commentid>
    <comment_count>0</comment_count>
    <who name="David Kilzer (:ddkilzer)">ddkilzer</who>
    <bug_when>2024-04-21 09:51:09 -0700</bug_when>
    <thetext>WebKit XML parsing can deny external entity loads from other libxml2 clients.

Caused by:

    Check if external entity loads from libxslt are allowed before loading them
    &lt;https://bugs.webkit.org/show_bug.cgi?id=259235&gt;
    &lt;rdar://111457167&gt;
    &lt;https://commits.webkit.org/269108@main&gt;

&lt;rdar://126476952&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2030097</commentid>
    <comment_count>1</comment_count>
    <who name="David Kilzer (:ddkilzer)">ddkilzer</who>
    <bug_when>2024-04-21 09:56:58 -0700</bug_when>
    <thetext>The fix for Bug 259235 replaced the default libxml2 external entity loader function with one from WebKit that implements the same-origin policy, but that means that WebKit1 clients that use libxml2 for parsing independent of WebKit also start using this function, which can cause load failures depending on the libxml2 API used.

One example API method that&apos;s affected is xmlCtxtReadFile(), which calls xmlLoadExternalEntity() to load the file.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2030098</commentid>
    <comment_count>2</comment_count>
    <who name="David Kilzer (:ddkilzer)">ddkilzer</who>
    <bug_when>2024-04-21 10:08:53 -0700</bug_when>
    <thetext>Pull request: https://github.com/WebKit/WebKit/pull/27562</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2032221</commentid>
    <comment_count>3</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2024-04-30 08:20:32 -0700</bug_when>
    <thetext>Committed 278168@main (7b1fb05b974f): &lt;https://commits.webkit.org/278168@main&gt;

Reviewed commits have been landed. Closing PR #27562 and removing active labels.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2032493</commentid>
    <comment_count>4</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2024-05-01 09:04:09 -0700</bug_when>
    <thetext>Committed 272448.976@safari-7618-branch (27da22ef6db2): &lt;https://commits.webkit.org/272448.976@safari-7618-branch&gt;

Reviewed commits have been landed. Closing PR #1245 and removing active labels.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>