<?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>83004</bug_id>
          
          <creation_ts>2012-04-02 23:44:45 -0700</creation_ts>
          <short_desc>[Plugins] delete ws_info regardless of window mode</short_desc>
          <delta_ts>2012-04-17 00:47:52 -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>Plug-ins</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>All</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>0</everconfirmed>
          <reporter name="Kwangmin Bang">justine.bang</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>hausmann</cc>
    
    <cc>jmalonzo</cc>
    
    <cc>vestbo</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>594234</commentid>
    <comment_count>0</comment_count>
    <who name="Kwangmin Bang">justine.bang</who>
    <bug_when>2012-04-02 23:44:45 -0700</bug_when>
    <thetext>In UNIX port(Gtk/QT/Brackberry), m_npWindow.ws_info = new NPSetWindowCallbackStruct; is called in PluginView::platformStart().
each platform does not delete this object(m_npWindow.ws_info), and it is done in PluginView:stop().

However,
If m_isWindowed is false (it means windowless mode), it&apos;s not deleted.

m_npWindow.ws_info should be deleted even if plugin run as windowless mode to remove memory leak.


Every plugin works well after this patch without any crash.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>594237</commentid>
    <comment_count>1</comment_count>
      <attachid>135281</attachid>
    <who name="Kwangmin Bang">justine.bang</who>
    <bug_when>2012-04-02 23:48:19 -0700</bug_when>
    <thetext>Created attachment 135281
[UNIX Plugins] delete ws_info, remove memory leak

patch.

it&apos;s simple. i just removed a condition in the IF statement.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>602861</commentid>
    <comment_count>2</comment_count>
      <attachid>137267</attachid>
    <who name="Kwangmin Bang">justine.bang</who>
    <bug_when>2012-04-15 20:03:36 -0700</bug_when>
    <thetext>Created attachment 137267
[UNIX Plugins] delete ws_info, remove memory leak

rebase
and change casting code to C++ style.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>602953</commentid>
    <comment_count>3</comment_count>
      <attachid>135281</attachid>
    <who name="Simon Hausmann">hausmann</who>
    <bug_when>2012-04-16 01:04:53 -0700</bug_when>
    <thetext>Comment on attachment 135281
[UNIX Plugins] delete ws_info, remove memory leak

Mark this as obsolete because there&apos;s a second patch for review.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>602956</commentid>
    <comment_count>4</comment_count>
      <attachid>137267</attachid>
    <who name="Simon Hausmann">hausmann</who>
    <bug_when>2012-04-16 01:07:38 -0700</bug_when>
    <thetext>Comment on attachment 137267
[UNIX Plugins] delete ws_info, remove memory leak

View in context: https://bugs.webkit.org/attachment.cgi?id=137267&amp;action=review

Looks good in general, but I think you can remove the if() altogether and there are a few language nitpicks :)

&gt; Source/WebCore/ChangeLog:3
&gt; +        [Plugins] delete ws_info regardless window mode.

&quot;regardless window mode&quot; -&gt; &quot;regardless of the window mode&quot;

&gt; Source/WebCore/ChangeLog:10
&gt; +        Additionally, change casting synta to C++ style.

synta -&gt; syntax

&gt; Source/WebCore/plugins/PluginView.cpp:370
&gt; +    if (m_npWindow.ws_info)
&gt; +           delete static_cast&lt;NPSetWindowCallbackStruct *&gt;(m_npWindow.ws_info);

I don&apos;t think the if() is needed at all. delete 0 is perfectly valid C++.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>602966</commentid>
    <comment_count>5</comment_count>
      <attachid>137298</attachid>
    <who name="Kwangmin Bang">justine.bang</who>
    <bug_when>2012-04-16 01:30:08 -0700</bug_when>
    <thetext>Created attachment 137298
[UNIX Plugins] delete ws_info, remove memory leak

change the title.
revise wrong word.
remove unneeded if().</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>602970</commentid>
    <comment_count>6</comment_count>
      <attachid>137298</attachid>
    <who name="Kwangmin Bang">justine.bang</who>
    <bug_when>2012-04-16 01:33:59 -0700</bug_when>
    <thetext>Comment on attachment 137298
[UNIX Plugins] delete ws_info, remove memory leak

mark this as obsolete</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>602977</commentid>
    <comment_count>7</comment_count>
      <attachid>137298</attachid>
    <who name="Kwangmin Bang">justine.bang</who>
    <bug_when>2012-04-16 01:38:57 -0700</bug_when>
    <thetext>Comment on attachment 137298
[UNIX Plugins] delete ws_info, remove memory leak

sorry for my mistake</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>602991</commentid>
    <comment_count>8</comment_count>
      <attachid>137267</attachid>
    <who name="Kwangmin Bang">justine.bang</who>
    <bug_when>2012-04-16 02:01:08 -0700</bug_when>
    <thetext>Comment on attachment 137267
[UNIX Plugins] delete ws_info, remove memory leak

&gt;diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
&gt;index 1863a58..2b59ef9 100644
&gt;--- a/Source/WebCore/ChangeLog
&gt;+++ b/Source/WebCore/ChangeLog
&gt;@@ -1,3 +1,17 @@
&gt;+2012-04-15  Bang Kwang min  &lt;justine.bang@samsung.com&gt;
&gt;+
&gt;+        [Plugins] delete ws_info regardless of window mode.
&gt;+        https://bugs.webkit.org/show_bug.cgi?id=83004
&gt;+
&gt;+        Reviewed by NOBODY (OOPS!).
&gt;+
&gt;+        m_npWindow.ws_info should be deleted even if plugin run as windowless mode to remove memory leak.
&gt;+        Every plugin works well after this patch without any crash.
&gt;+        Additionally, change casting syntax to C++ style.
&gt;+
&gt;+        * plugins/PluginView.cpp:
&gt;+        (WebCore::PluginView::stop):
&gt;+
&gt; 2012-04-15  James Robinson  &lt;jamesr@chromium.org&gt;
&gt; 
&gt;         [chromium] LayerRendererChromium shouldn&apos;t know anything about CCLayerImpl
&gt;diff --git a/Source/WebCore/plugins/PluginView.cpp b/Source/WebCore/plugins/PluginView.cpp
&gt;index 749f609..389a8fe 100644
&gt;--- a/Source/WebCore/plugins/PluginView.cpp
&gt;+++ b/Source/WebCore/plugins/PluginView.cpp
&gt;@@ -366,8 +366,8 @@ void PluginView::stop()
&gt;     }
&gt; 
&gt; #ifdef XP_UNIX
&gt;-    if (m_isWindowed &amp;&amp; m_npWindow.ws_info)
&gt;-           delete (NPSetWindowCallbackStruct *)m_npWindow.ws_info;
&gt;+    if (m_npWindow.ws_info)
&gt;+           delete static_cast&lt;NPSetWindowCallbackStruct *&gt;(m_npWindow.ws_info);
&gt;     m_npWindow.ws_info = 0;
&gt; #endif
&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>602992</commentid>
    <comment_count>9</comment_count>
      <attachid>137298</attachid>
    <who name="Kwangmin Bang">justine.bang</who>
    <bug_when>2012-04-16 02:02:14 -0700</bug_when>
    <thetext>Comment on attachment 137298
[UNIX Plugins] delete ws_info, remove memory leak

as i think, delete 0 is the problem
so, please review previous patch. i modified the title and comment.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>603001</commentid>
    <comment_count>10</comment_count>
      <attachid>137307</attachid>
    <who name="Kwangmin Bang">justine.bang</who>
    <bug_when>2012-04-16 02:12:41 -0700</bug_when>
    <thetext>Created attachment 137307
[UNIX Plugins] delete ws_info, remove memory leak

rebase.

I&apos;m sorry i&apos;m not accustomed to manage the attachment.


-
if remove if(), it does not pass qt port test.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>603003</commentid>
    <comment_count>11</comment_count>
    <who name="Simon Hausmann">hausmann</who>
    <bug_when>2012-04-16 02:30:05 -0700</bug_when>
    <thetext>(In reply to comment #10)
&gt; Created an attachment (id=137307) [details]
&gt; [UNIX Plugins] delete ws_info, remove memory leak
&gt; 
&gt; rebase.
&gt; 
&gt; I&apos;m sorry i&apos;m not accustomed to manage the attachment.

No worries, it seems you got the hang of it :)
 
&gt; -
&gt; if remove if(), it does not pass qt port test.

Can you elaborate a bit on that?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>603022</commentid>
    <comment_count>12</comment_count>
    <who name="Simon Hausmann">hausmann</who>
    <bug_when>2012-04-16 03:48:10 -0700</bug_when>
    <thetext>(In reply to comment #9)
&gt; (From update of attachment 137298 [details])
&gt; as i think, delete 0 is the problem

See also http://www.parashift.com/c++-faq-lite/freestore-mgmt.html#faq-16.8</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>603053</commentid>
    <comment_count>13</comment_count>
    <who name="Kwangmin Bang">justine.bang</who>
    <bug_when>2012-04-16 04:52:24 -0700</bug_when>
    <thetext>(In reply to comment #12)
&gt; (In reply to comment #9)
&gt; &gt; (From update of attachment 137298 [details] [details])
&gt; &gt; as i think, delete 0 is the problem
&gt; 
&gt; See also http://www.parashift.com/c++-faq-lite/freestore-mgmt.html#faq-16.8

I agree with you, and i has uploaded the patch, but it failed to pass test.
please see attachment id 137298. you can see qt, qt-wk2 and efl are pink color.

To know why it failed, i&apos;m trying to build QT.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>603583</commentid>
    <comment_count>14</comment_count>
      <attachid>137433</attachid>
    <who name="Kwangmin Bang">justine.bang</who>
    <bug_when>2012-04-16 17:04:14 -0700</bug_when>
    <thetext>Created attachment 137433
delete without null checking.

delete without null checking. you can say that.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>603585</commentid>
    <comment_count>15</comment_count>
    <who name="Kwangmin Bang">justine.bang</who>
    <bug_when>2012-04-16 17:06:16 -0700</bug_when>
    <thetext>*** Bug 84021 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>603849</commentid>
    <comment_count>16</comment_count>
      <attachid>137433</attachid>
    <who name="Simon Hausmann">hausmann</who>
    <bug_when>2012-04-17 00:46:06 -0700</bug_when>
    <thetext>Comment on attachment 137433
delete without null checking.

View in context: https://bugs.webkit.org/attachment.cgi?id=137433&amp;action=review

&gt; Source/WebCore/plugins/PluginView.cpp:369
&gt; +    delete static_cast&lt;NPSetWindowCallbackStruct *&gt;(m_npWindow.ws_info);

Small coding style glitch, space after *. I&apos;ll fix it when landing.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>603850</commentid>
    <comment_count>17</comment_count>
    <who name="Simon Hausmann">hausmann</who>
    <bug_when>2012-04-17 00:47:52 -0700</bug_when>
    <thetext>Committed r114358: &lt;http://trac.webkit.org/changeset/114358&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>135281</attachid>
            <date>2012-04-02 23:48:19 -0700</date>
            <delta_ts>2012-04-16 01:04:53 -0700</delta_ts>
            <desc>[UNIX Plugins] delete ws_info, remove memory leak</desc>
            <filename>patch-[20120403]-[justine.bang@samsung.com]-[delete-ws_info].patch</filename>
            <type>text/plain</type>
            <size>1198</size>
            <attacher name="Kwangmin Bang">justine.bang</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0No
YW5nZUxvZwppbmRleCBhMmIxNWVmLi4zYzc1NmZhIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29y
ZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTYg
QEAKKzIwMTItMDQtMDIgIEJhbmcgS3dhbmcgbWluICA8anVzdGluZS5iYW5nQHNhbXN1bmcuY29t
PgorCisgICAgICAgIFtQbHVnaW5zXSBkZWxldGUgd3NfaW5mbyByZWdhcmRsZXNzIHdpbmRvdyBt
b2RlLgorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9ODMw
MDQKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBtX25w
V2luZG93LndzX2luZm8gc2hvdWxkIGJlIGRlbGV0ZWQgZXZlbiBpZiBwbHVnaW4gcnVuIGFzIHdp
bmRvd2xlc3MgbW9kZSB0byByZW1vdmUgbWVtb3J5IGxlYWsuCisgICAgICAgIEV2ZXJ5IHBsdWdp
biB3b3JrcyB3ZWxsIGFmdGVyIHRoaXMgcGF0Y2ggd2l0aG91dCBhbnkgY3Jhc2guCisKKyAgICAg
ICAgKiBwbHVnaW5zL1BsdWdpblZpZXcuY3BwOgorICAgICAgICAoV2ViQ29yZTo6UGx1Z2luVmll
dzo6c3RvcCk6CisKIDIwMTItMDQtMDIgIEFiaGlzaGVrIEFyeWEgIDxpbmZlcm5vQGNocm9taXVt
Lm9yZz4KIAogICAgICAgICA8c2VsZWN0PiBzaG91bGRuJ3QgaW50cnVkZSBhcyBhIHJ1bi1pbi4K
ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL3BsdWdpbnMvUGx1Z2luVmlldy5jcHAgYi9Tb3Vy
Y2UvV2ViQ29yZS9wbHVnaW5zL1BsdWdpblZpZXcuY3BwCmluZGV4IDc0OWY2MDkuLjQzNzE4ZDAg
MTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJDb3JlL3BsdWdpbnMvUGx1Z2luVmlldy5jcHAKKysrIGIv
U291cmNlL1dlYkNvcmUvcGx1Z2lucy9QbHVnaW5WaWV3LmNwcApAQCAtMzY2LDcgKzM2Niw3IEBA
IHZvaWQgUGx1Z2luVmlldzo6c3RvcCgpCiAgICAgfQogCiAjaWZkZWYgWFBfVU5JWAotICAgIGlm
IChtX2lzV2luZG93ZWQgJiYgbV9ucFdpbmRvdy53c19pbmZvKQorICAgIGlmIChtX25wV2luZG93
LndzX2luZm8pCiAgICAgICAgICAgIGRlbGV0ZSAoTlBTZXRXaW5kb3dDYWxsYmFja1N0cnVjdCAq
KW1fbnBXaW5kb3cud3NfaW5mbzsKICAgICBtX25wV2luZG93LndzX2luZm8gPSAwOwogI2VuZGlm
Cg==
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>137267</attachid>
            <date>2012-04-15 20:03:36 -0700</date>
            <delta_ts>2012-04-16 02:12:41 -0700</delta_ts>
            <desc>[UNIX Plugins] delete ws_info, remove memory leak</desc>
            <filename>patch-[20120403]-[justine.bang@samsung.com]-[delete-ws_info].patch</filename>
            <type>text/plain</type>
            <size>1374</size>
            <attacher name="Kwangmin Bang">justine.bang</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0No
YW5nZUxvZwppbmRleCAxODYzYTU4Li4yYjU5ZWY5IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29y
ZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTcg
QEAKKzIwMTItMDQtMTUgIEJhbmcgS3dhbmcgbWluICA8anVzdGluZS5iYW5nQHNhbXN1bmcuY29t
PgorCisgICAgICAgIFtQbHVnaW5zXSBkZWxldGUgd3NfaW5mbyByZWdhcmRsZXNzIHdpbmRvdyBt
b2RlLgorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9ODMw
MDQKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBtX25w
V2luZG93LndzX2luZm8gc2hvdWxkIGJlIGRlbGV0ZWQgZXZlbiBpZiBwbHVnaW4gcnVuIGFzIHdp
bmRvd2xlc3MgbW9kZSB0byByZW1vdmUgbWVtb3J5IGxlYWsuCisgICAgICAgIEV2ZXJ5IHBsdWdp
biB3b3JrcyB3ZWxsIGFmdGVyIHRoaXMgcGF0Y2ggd2l0aG91dCBhbnkgY3Jhc2guCisgICAgICAg
IEFkZGl0aW9uYWxseSwgY2hhbmdlIGNhc3Rpbmcgc3ludGEgdG8gQysrIHN0eWxlLgorCisgICAg
ICAgICogcGx1Z2lucy9QbHVnaW5WaWV3LmNwcDoKKyAgICAgICAgKFdlYkNvcmU6OlBsdWdpblZp
ZXc6OnN0b3ApOgorCiAyMDEyLTA0LTE1ICBKYW1lcyBSb2JpbnNvbiAgPGphbWVzckBjaHJvbWl1
bS5vcmc+CiAKICAgICAgICAgW2Nocm9taXVtXSBMYXllclJlbmRlcmVyQ2hyb21pdW0gc2hvdWxk
bid0IGtub3cgYW55dGhpbmcgYWJvdXQgQ0NMYXllckltcGwKZGlmZiAtLWdpdCBhL1NvdXJjZS9X
ZWJDb3JlL3BsdWdpbnMvUGx1Z2luVmlldy5jcHAgYi9Tb3VyY2UvV2ViQ29yZS9wbHVnaW5zL1Bs
dWdpblZpZXcuY3BwCmluZGV4IDc0OWY2MDkuLjM4OWE4ZmUgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9X
ZWJDb3JlL3BsdWdpbnMvUGx1Z2luVmlldy5jcHAKKysrIGIvU291cmNlL1dlYkNvcmUvcGx1Z2lu
cy9QbHVnaW5WaWV3LmNwcApAQCAtMzY2LDggKzM2Niw4IEBAIHZvaWQgUGx1Z2luVmlldzo6c3Rv
cCgpCiAgICAgfQogCiAjaWZkZWYgWFBfVU5JWAotICAgIGlmIChtX2lzV2luZG93ZWQgJiYgbV9u
cFdpbmRvdy53c19pbmZvKQotICAgICAgICAgICBkZWxldGUgKE5QU2V0V2luZG93Q2FsbGJhY2tT
dHJ1Y3QgKiltX25wV2luZG93LndzX2luZm87CisgICAgaWYgKG1fbnBXaW5kb3cud3NfaW5mbykK
KyAgICAgICAgICAgZGVsZXRlIHN0YXRpY19jYXN0PE5QU2V0V2luZG93Q2FsbGJhY2tTdHJ1Y3Qg
Kj4obV9ucFdpbmRvdy53c19pbmZvKTsKICAgICBtX25wV2luZG93LndzX2luZm8gPSAwOwogI2Vu
ZGlmCiAK
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>137298</attachid>
            <date>2012-04-16 01:30:08 -0700</date>
            <delta_ts>2012-04-16 02:02:14 -0700</delta_ts>
            <desc>[UNIX Plugins] delete ws_info, remove memory leak</desc>
            <filename>patch-[20120403]-[justine.bang@samsung.com]-[delete-ws_info].patch</filename>
            <type>text/plain</type>
            <size>1346</size>
            <attacher name="Kwangmin Bang">justine.bang</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0No
YW5nZUxvZwppbmRleCAxODYzYTU4Li43NjJiMzgwIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29y
ZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTcg
QEAKKzIwMTItMDQtMTUgIEJhbmcgS3dhbmcgbWluICA8anVzdGluZS5iYW5nQHNhbXN1bmcuY29t
PgorCisgICAgICAgIFtQbHVnaW5zXSBkZWxldGUgd3NfaW5mbyByZWdhcmRsZXNzIG9mIHRoZSB3
aW5kb3cgbW9kZS4KKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dp
P2lkPTgzMDA0CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAg
ICAgbV9ucFdpbmRvdy53c19pbmZvIHNob3VsZCBiZSBkZWxldGVkIGV2ZW4gaWYgcGx1Z2luIHJ1
biBhcyB3aW5kb3dsZXNzIG1vZGUgdG8gcmVtb3ZlIG1lbW9yeSBsZWFrLgorICAgICAgICBFdmVy
eSBwbHVnaW4gd29ya3Mgd2VsbCBhZnRlciB0aGlzIHBhdGNoIHdpdGhvdXQgYW55IGNyYXNoLgor
ICAgICAgICBBZGRpdGlvbmFsbHksIGNoYW5nZSBjYXN0aW5nIHN5bnRheCB0byBDKysgc3R5bGUu
CisKKyAgICAgICAgKiBwbHVnaW5zL1BsdWdpblZpZXcuY3BwOgorICAgICAgICAoV2ViQ29yZTo6
UGx1Z2luVmlldzo6c3RvcCk6CisKIDIwMTItMDQtMTUgIEphbWVzIFJvYmluc29uICA8amFtZXNy
QGNocm9taXVtLm9yZz4KIAogICAgICAgICBbY2hyb21pdW1dIExheWVyUmVuZGVyZXJDaHJvbWl1
bSBzaG91bGRuJ3Qga25vdyBhbnl0aGluZyBhYm91dCBDQ0xheWVySW1wbApkaWZmIC0tZ2l0IGEv
U291cmNlL1dlYkNvcmUvcGx1Z2lucy9QbHVnaW5WaWV3LmNwcCBiL1NvdXJjZS9XZWJDb3JlL3Bs
dWdpbnMvUGx1Z2luVmlldy5jcHAKaW5kZXggNzQ5ZjYwOS4uMmM3NWM1YiAxMDA2NDQKLS0tIGEv
U291cmNlL1dlYkNvcmUvcGx1Z2lucy9QbHVnaW5WaWV3LmNwcAorKysgYi9Tb3VyY2UvV2ViQ29y
ZS9wbHVnaW5zL1BsdWdpblZpZXcuY3BwCkBAIC0zNjYsOCArMzY2LDcgQEAgdm9pZCBQbHVnaW5W
aWV3OjpzdG9wKCkKICAgICB9CiAKICNpZmRlZiBYUF9VTklYCi0gICAgaWYgKG1faXNXaW5kb3dl
ZCAmJiBtX25wV2luZG93LndzX2luZm8pCi0gICAgICAgICAgIGRlbGV0ZSAoTlBTZXRXaW5kb3dD
YWxsYmFja1N0cnVjdCAqKW1fbnBXaW5kb3cud3NfaW5mbzsKKyAgICBkZWxldGUgc3RhdGljX2Nh
c3Q8TlBTZXRXaW5kb3dDYWxsYmFja1N0cnVjdCAqPihtX25wV2luZG93LndzX2luZm8pOwogICAg
IG1fbnBXaW5kb3cud3NfaW5mbyA9IDA7CiAjZW5kaWYKIAo=
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>137307</attachid>
            <date>2012-04-16 02:12:41 -0700</date>
            <delta_ts>2012-04-16 17:04:14 -0700</delta_ts>
            <desc>[UNIX Plugins] delete ws_info, remove memory leak</desc>
            <filename>patch-[20120403]-[justine.bang@samsung.com]-[delete-ws_info].patch</filename>
            <type>text/plain</type>
            <size>1304</size>
            <attacher name="Kwangmin Bang">justine.bang</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0No
YW5nZUxvZwppbmRleCAyMDJiY2NlLi4wZjNmYzc3IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29y
ZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTcg
QEAKKzIwMTItMDQtMTYgIEJhbmcgS3dhbmcgbWluICA8anVzdGluZS5iYW5nQHNhbXN1bmcuY29t
PgorCisgICAgICAgIFtQbHVnaW5zXSBkZWxldGUgd3NfaW5mbyByZWdhcmRsZXNzIG9mIHdpbmRv
dyBtb2RlLgorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9
ODMwMDQKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBt
X25wV2luZG93LndzX2luZm8gc2hvdWxkIGJlIGRlbGV0ZWQgZXZlbiBpZiBwbHVnaW4gcnVuIGFz
IHdpbmRvd2xlc3MgbW9kZSB0byByZW1vdmUgbWVtb3J5IGxlYWsuCisgICAgICAgIEV2ZXJ5IHBs
dWdpbiB3b3JrcyB3ZWxsIGFmdGVyIHRoaXMgcGF0Y2ggd2l0aG91dCBhbnkgY3Jhc2guCisgICAg
ICAgIEFkZGl0aW9uYWxseSwgY2hhbmdlIGNhc3Rpbmcgc3ludGF4IHRvIEMrKyBzdHlsZS4KKwor
ICAgICAgICAqIHBsdWdpbnMvUGx1Z2luVmlldy5jcHA6CisgICAgICAgIChXZWJDb3JlOjpQbHVn
aW5WaWV3OjpzdG9wKToKKwogMjAxMi0wNC0xNiAgSm9uIExlZSAgPGpvbmxlZUBhcHBsZS5jb20+
CiAKICAgICAgICAgQnVpbGQgZml4LgpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYkNvcmUvcGx1Z2lu
cy9QbHVnaW5WaWV3LmNwcCBiL1NvdXJjZS9XZWJDb3JlL3BsdWdpbnMvUGx1Z2luVmlldy5jcHAK
aW5kZXggNzQ5ZjYwOS4uMzg5YThmZSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvcGx1Z2lu
cy9QbHVnaW5WaWV3LmNwcAorKysgYi9Tb3VyY2UvV2ViQ29yZS9wbHVnaW5zL1BsdWdpblZpZXcu
Y3BwCkBAIC0zNjYsOCArMzY2LDggQEAgdm9pZCBQbHVnaW5WaWV3OjpzdG9wKCkKICAgICB9CiAK
ICNpZmRlZiBYUF9VTklYCi0gICAgaWYgKG1faXNXaW5kb3dlZCAmJiBtX25wV2luZG93LndzX2lu
Zm8pCi0gICAgICAgICAgIGRlbGV0ZSAoTlBTZXRXaW5kb3dDYWxsYmFja1N0cnVjdCAqKW1fbnBX
aW5kb3cud3NfaW5mbzsKKyAgICBpZiAobV9ucFdpbmRvdy53c19pbmZvKQorICAgICAgICAgICBk
ZWxldGUgc3RhdGljX2Nhc3Q8TlBTZXRXaW5kb3dDYWxsYmFja1N0cnVjdCAqPihtX25wV2luZG93
LndzX2luZm8pOwogICAgIG1fbnBXaW5kb3cud3NfaW5mbyA9IDA7CiAjZW5kaWYKIAo=
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>137433</attachid>
            <date>2012-04-16 17:04:14 -0700</date>
            <delta_ts>2012-04-17 00:46:06 -0700</delta_ts>
            <desc>delete without null checking.</desc>
            <filename>patch-[20120403]-[justine.bang@samsung.com]-[delete-ws_info].patch</filename>
            <type>text/plain</type>
            <size>1268</size>
            <attacher name="Kwangmin Bang">justine.bang</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0No
YW5nZUxvZwppbmRleCAyMDJiY2NlLi4wZjNmYzc3IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29y
ZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTcg
QEAKKzIwMTItMDQtMTYgIEJhbmcgS3dhbmcgbWluICA8anVzdGluZS5iYW5nQHNhbXN1bmcuY29t
PgorCisgICAgICAgIFtQbHVnaW5zXSBkZWxldGUgd3NfaW5mbyByZWdhcmRsZXNzIG9mIHdpbmRv
dyBtb2RlLgorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9
ODMwMDQKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBt
X25wV2luZG93LndzX2luZm8gc2hvdWxkIGJlIGRlbGV0ZWQgZXZlbiBpZiBwbHVnaW4gcnVuIGFz
IHdpbmRvd2xlc3MgbW9kZSB0byByZW1vdmUgbWVtb3J5IGxlYWsuCisgICAgICAgIEV2ZXJ5IHBs
dWdpbiB3b3JrcyB3ZWxsIGFmdGVyIHRoaXMgcGF0Y2ggd2l0aG91dCBhbnkgY3Jhc2guCisgICAg
ICAgIEFkZGl0aW9uYWxseSwgY2hhbmdlIGNhc3Rpbmcgc3ludGF4IHRvIEMrKyBzdHlsZS4KKwor
ICAgICAgICAqIHBsdWdpbnMvUGx1Z2luVmlldy5jcHA6CisgICAgICAgIChXZWJDb3JlOjpQbHVn
aW5WaWV3OjpzdG9wKToKKwogMjAxMi0wNC0xNiAgSm9uIExlZSAgPGpvbmxlZUBhcHBsZS5jb20+
CiAKICAgICAgICAgQnVpbGQgZml4LgpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYkNvcmUvcGx1Z2lu
cy9QbHVnaW5WaWV3LmNwcCBiL1NvdXJjZS9XZWJDb3JlL3BsdWdpbnMvUGx1Z2luVmlldy5jcHAK
aW5kZXggNzQ5ZjYwOS4uMzg5YThmZSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvcGx1Z2lu
cy9QbHVnaW5WaWV3LmNwcAorKysgYi9Tb3VyY2UvV2ViQ29yZS9wbHVnaW5zL1BsdWdpblZpZXcu
Y3BwCkBAIC0zNjYsOCArMzY2LDcgQEAgdm9pZCBQbHVnaW5WaWV3OjpzdG9wKCkKICAgICB9CiAK
ICNpZmRlZiBYUF9VTklYCi0gICAgaWYgKG1faXNXaW5kb3dlZCAmJiBtX25wV2luZG93LndzX2lu
Zm8pCi0gICAgICAgICAgIGRlbGV0ZSAoTlBTZXRXaW5kb3dDYWxsYmFja1N0cnVjdCAqKW1fbnBX
aW5kb3cud3NfaW5mbzsKKyAgICBkZWxldGUgc3RhdGljX2Nhc3Q8TlBTZXRXaW5kb3dDYWxsYmFj
a1N0cnVjdCAqPihtX25wV2luZG93LndzX2luZm8pOwogICAgIG1fbnBXaW5kb3cud3NfaW5mbyA9
IDA7CiAjZW5kaWYKIAo=
</data>
<flag name="review"
          id="142501"
          type_id="1"
          status="+"
          setter="hausmann"
    />
    <flag name="commit-queue"
          id="142502"
          type_id="3"
          status="-"
          setter="hausmann"
    />
          </attachment>
      

    </bug>

</bugzilla>