<?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>221046</bug_id>
          
          <creation_ts>2021-01-27 09:12:37 -0800</creation_ts>
          <short_desc>[iOS] Fix sandbox violation file-read-data /dev/dtracehelper in the Networking process</short_desc>
          <delta_ts>2021-01-27 10:07:43 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>WebKit Misc.</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="Per Arne Vollan">pvollan</reporter>
          <assigned_to name="Per Arne Vollan">pvollan</assigned_to>
          <cc>bfulgham</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1723007</commentid>
    <comment_count>0</comment_count>
    <who name="Per Arne Vollan">pvollan</who>
    <bug_when>2021-01-27 09:12:37 -0800</bug_when>
    <thetext>Reading this file should be allowed in the Networking process on internal installs on iOS.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1723009</commentid>
    <comment_count>1</comment_count>
    <who name="Per Arne Vollan">pvollan</who>
    <bug_when>2021-01-27 09:13:02 -0800</bug_when>
    <thetext>&lt;rdar://problem/73625718&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1723011</commentid>
    <comment_count>2</comment_count>
      <attachid>418552</attachid>
    <who name="Per Arne Vollan">pvollan</who>
    <bug_when>2021-01-27 09:14:33 -0800</bug_when>
    <thetext>Created attachment 418552
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1723029</commentid>
    <comment_count>3</comment_count>
      <attachid>418552</attachid>
    <who name="Brent Fulgham">bfulgham</who>
    <bug_when>2021-01-27 09:50:02 -0800</bug_when>
    <thetext>Comment on attachment 418552
Patch

I think this change looks good, but I think you should take this further and (1) add the apple-internal check in the other places it is used, and (2) add the log silencing check everywhere. You can do that as a follow-up patch is you prefer.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1723042</commentid>
    <comment_count>4</comment_count>
    <who name="Per Arne Vollan">pvollan</who>
    <bug_when>2021-01-27 10:04:48 -0800</bug_when>
    <thetext>(In reply to Brent Fulgham from comment #3)
&gt; Comment on attachment 418552 [details]
&gt; Patch
&gt; 
&gt; I think this change looks good, but I think you should take this further and
&gt; (1) add the apple-internal check in the other places it is used, and (2) add
&gt; the log silencing check everywhere. You can do that as a follow-up patch is
&gt; you prefer.

I will post a follow up patch.

Thanks for reviewing!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1723048</commentid>
    <comment_count>5</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2021-01-27 10:07:42 -0800</bug_when>
    <thetext>Committed r271964: &lt;https://trac.webkit.org/changeset/271964&gt;

All reviewed patches have been landed. Closing bug and clearing flags on attachment 418552.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>418552</attachid>
            <date>2021-01-27 09:14:33 -0800</date>
            <delta_ts>2021-01-27 10:07:42 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-221046-20210127091432.patch</filename>
            <type>text/plain</type>
            <size>1517</size>
            <attacher name="Per Arne Vollan">pvollan</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9XZWJL
aXQvQ2hhbmdlTG9nCShyZXZpc2lvbiAyNzE5NTQpCisrKyBTb3VyY2UvV2ViS2l0L0NoYW5nZUxv
Zwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE1IEBACisyMDIxLTAxLTI3ICBQZXIgQXJuZSAg
PHB2b2xsYW5AYXBwbGUuY29tPgorCisgICAgICAgIFtpT1NdIEZpeCBzYW5kYm94IHZpb2xhdGlv
biBmaWxlLXJlYWQtZGF0YSAvZGV2L2R0cmFjZWhlbHBlciBpbiB0aGUgTmV0d29ya2luZyBwcm9j
ZXNzCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0yMjEw
NDYKKyAgICAgICAgPHJkYXI6Ly9wcm9ibGVtLzczNjI1NzE4PgorCisgICAgICAgIFJldmlld2Vk
IGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIFJlYWRpbmcgdGhpcyBmaWxlIHNob3VsZCBi
ZSBhbGxvd2VkIGluIHRoZSBOZXR3b3JraW5nIHByb2Nlc3Mgb24gaW50ZXJuYWwgaW5zdGFsbHMg
b24gaU9TLgorCisgICAgICAgICogUmVzb3VyY2VzL1NhbmRib3hQcm9maWxlcy9pb3MvY29tLmFw
cGxlLldlYktpdC5OZXR3b3JraW5nLnNiOgorCiAyMDIxLTAxLTI3ICBDb21taXQgUXVldWUgIDxj
b21taXQtcXVldWVAd2Via2l0Lm9yZz4KIAogICAgICAgICBVbnJldmlld2VkLCByZXZlcnRpbmcg
cjI3MTk0MC4KSW5kZXg6IFNvdXJjZS9XZWJLaXQvUmVzb3VyY2VzL1NhbmRib3hQcm9maWxlcy9p
b3MvY29tLmFwcGxlLldlYktpdC5OZXR3b3JraW5nLnNiCj09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9X
ZWJLaXQvUmVzb3VyY2VzL1NhbmRib3hQcm9maWxlcy9pb3MvY29tLmFwcGxlLldlYktpdC5OZXR3
b3JraW5nLnNiCShyZXZpc2lvbiAyNzE5NTApCisrKyBTb3VyY2UvV2ViS2l0L1Jlc291cmNlcy9T
YW5kYm94UHJvZmlsZXMvaW9zL2NvbS5hcHBsZS5XZWJLaXQuTmV0d29ya2luZy5zYgkod29ya2lu
ZyBjb3B5KQpAQCAtNjU3LDYgKzY1NywxNCBAQAogICAgIChsaXRlcmFsICIvZGV2L3JhbmRvbSIp
CiAgICAgKGxpdGVyYWwgIi9kZXYvdXJhbmRvbSIpKQogCisoaWYgKHN5c3RlbS1hdHRyaWJ1dGUg
YXBwbGUtaW50ZXJuYWwpCisgICAgKGFsbG93IGZpbGUtcmVhZCogZmlsZS13cml0ZS1kYXRhIGZp
bGUtaW9jdGwKKyAgICAgICAgKGxpdGVyYWwgIi9kZXYvZHRyYWNlaGVscGVyIikpCis7IGVsc2UK
KyAgICAoZGVueSAod2l0aCBuby1sb2cpIGZpbGUtcmVhZCogZmlsZS13cml0ZS1kYXRhIGZpbGUt
aW9jdGwKKyAgICAgICAgKGxpdGVyYWwgIi9kZXYvZHRyYWNlaGVscGVyIikpCispCisKIDs7IEFj
Y2VzcyB0byBNb2JpbGVHZXN0YWx0CiAoYWxsb3cgbWFjaC1sb29rdXAKICAgICAoZ2xvYmFsLW5h
bWUgImNvbS5hcHBsZS5tb2JpbGVnZXN0YWx0LnhwYyIpKQo=
</data>

          </attachment>
      

    </bug>

</bugzilla>