<?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>197115</bug_id>
          
          <creation_ts>2019-04-19 14:02:31 -0700</creation_ts>
          <short_desc>Web Inspector: Timelines: CPU: ensure that tracking stops when disconnecting Web Inspector</short_desc>
          <delta_ts>2019-04-19 17:33:27 -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>Web Inspector</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</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="Devin Rousso">hi</reporter>
          <assigned_to name="Devin Rousso">hi</assigned_to>
          <cc>commit-queue</cc>
    
    <cc>ews-watchlist</cc>
    
    <cc>hi</cc>
    
    <cc>inspector-bugzilla-changes</cc>
    
    <cc>joepeck</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1528854</commentid>
    <comment_count>0</comment_count>
    <who name="Devin Rousso">hi</who>
    <bug_when>2019-04-19 14:02:31 -0700</bug_when>
    <thetext>It may be possible to UAF the `InspectorCPUProfilerAgent`, since it adds itself (via a C++ lambda) as an observer to another object.  If the inspector disconnects while actively recording, we currently don&apos;t remove that.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1528855</commentid>
    <comment_count>1</comment_count>
    <who name="Devin Rousso">hi</who>
    <bug_when>2019-04-19 14:02:42 -0700</bug_when>
    <thetext>&lt;rdar://problem/49877875&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1528856</commentid>
    <comment_count>2</comment_count>
      <attachid>367823</attachid>
    <who name="Devin Rousso">hi</who>
    <bug_when>2019-04-19 14:03:33 -0700</bug_when>
    <thetext>Created attachment 367823
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1528940</commentid>
    <comment_count>3</comment_count>
      <attachid>367823</attachid>
    <who name="Joseph Pecoraro">joepeck</who>
    <bug_when>2019-04-19 16:24:24 -0700</bug_when>
    <thetext>Comment on attachment 367823
Patch

r=me. Are there any other agents that could benefit from this?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1528943</commentid>
    <comment_count>4</comment_count>
    <who name="Devin Rousso">hi</who>
    <bug_when>2019-04-19 16:27:15 -0700</bug_when>
    <thetext>(In reply to Joseph Pecoraro from comment #3)
&gt; r=me. Are there any other agents that could benefit from this?
I checked the other timelines agents, and all of them either weren&apos;t susceptible  or already had a &quot;fix&quot; for this issue.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1528983</commentid>
    <comment_count>5</comment_count>
      <attachid>367823</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2019-04-19 17:33:26 -0700</bug_when>
    <thetext>Comment on attachment 367823
Patch

Clearing flags on attachment: 367823

Committed r244478: &lt;https://trac.webkit.org/changeset/244478&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1528984</commentid>
    <comment_count>6</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2019-04-19 17:33:27 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>367823</attachid>
            <date>2019-04-19 14:03:33 -0700</date>
            <delta_ts>2019-04-19 17:33:26 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-197115-20190419140333.patch</filename>
            <type>text/plain</type>
            <size>1514</size>
            <attacher name="Devin Rousso">hi</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0No
YW5nZUxvZwppbmRleCBkZjZmYzczOGUyMjY5M2E0ODczZDM3Yzg3MmE3MjQ5ZDE3MzA2MTIzLi4z
ZDhjZDlkY2IyN2QyZWNkZmExMWY4NmQzNzA4ZDgzZmM4NTFiOTE5IDEwMDY0NAotLS0gYS9Tb3Vy
Y2UvV2ViQ29yZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0x
LDMgKzEsMTQgQEAKKzIwMTktMDQtMTkgIERldmluIFJvdXNzbyAgPGRyb3Vzc29AYXBwbGUuY29t
PgorCisgICAgICAgIFdlYiBJbnNwZWN0b3I6IFRpbWVsaW5lczogQ1BVOiBlbnN1cmUgdGhhdCB0
cmFja2luZyBzdG9wcyB3aGVuIGRpc2Nvbm5lY3RpbmcgV2ViIEluc3BlY3RvcgorICAgICAgICBo
dHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTk3MTE1CisgICAgICAgIDxy
ZGFyOi8vcHJvYmxlbS80OTg3Nzg3NT4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9P
UFMhKS4KKworICAgICAgICAqIGluc3BlY3Rvci9hZ2VudHMvSW5zcGVjdG9yQ1BVUHJvZmlsZXJB
Z2VudC5jcHA6CisgICAgICAgIChXZWJDb3JlOjpJbnNwZWN0b3JDUFVQcm9maWxlckFnZW50Ojp3
aWxsRGVzdHJveUZyb250ZW5kQW5kQmFja2VuZCk6CisKIDIwMTktMDQtMTkgIFdlbnNvbiBIc2ll
aCAgPHdlbnNvbl9oc2llaEBhcHBsZS5jb20+CiAKICAgICAgICAgW2lPU10gQWRkIHF1aXJrcyB0
byBkaXNhYmxlIGF1dG9jb3JyZWN0aW9uIGFuZCBhdXRvY2FwaXRhbGl6YXRpb24gaW4gaGlkZGVu
IGVkaXRhYmxlIGFyZWFzIG9uIHNvbWUgd2Vic2l0ZXMKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJD
b3JlL2luc3BlY3Rvci9hZ2VudHMvSW5zcGVjdG9yQ1BVUHJvZmlsZXJBZ2VudC5jcHAgYi9Tb3Vy
Y2UvV2ViQ29yZS9pbnNwZWN0b3IvYWdlbnRzL0luc3BlY3RvckNQVVByb2ZpbGVyQWdlbnQuY3Bw
CmluZGV4IDZhMjk3MmQ2ODg1MTZkY2YyODYxYjNlMTVhNTQxNmFjMDIxZmI0ZjIuLmNmOTRjYTgw
OTZmZjU3MWQ3NGQ3NzNiMWYwMDI3NTA1NGMwY2UxZjkgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJD
b3JlL2luc3BlY3Rvci9hZ2VudHMvSW5zcGVjdG9yQ1BVUHJvZmlsZXJBZ2VudC5jcHAKKysrIGIv
U291cmNlL1dlYkNvcmUvaW5zcGVjdG9yL2FnZW50cy9JbnNwZWN0b3JDUFVQcm9maWxlckFnZW50
LmNwcApAQCAtNTEsNiArNTEsOSBAQCB2b2lkIEluc3BlY3RvckNQVVByb2ZpbGVyQWdlbnQ6OmRp
ZENyZWF0ZUZyb250ZW5kQW5kQmFja2VuZChGcm9udGVuZFJvdXRlciosIEJhYwogCiB2b2lkIElu
c3BlY3RvckNQVVByb2ZpbGVyQWdlbnQ6OndpbGxEZXN0cm95RnJvbnRlbmRBbmRCYWNrZW5kKERp
c2Nvbm5lY3RSZWFzb24pCiB7CisgICAgRXJyb3JTdHJpbmcgaWdub3JlZDsKKyAgICBzdG9wVHJh
Y2tpbmcoaWdub3JlZCk7CisKICAgICBtX2luc3RydW1lbnRpbmdBZ2VudHMuc2V0SW5zcGVjdG9y
Q1BVUHJvZmlsZXJBZ2VudChudWxscHRyKTsKIH0KIAo=
</data>

          </attachment>
      

    </bug>

</bugzilla>