<?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>164103</bug_id>
          
          <creation_ts>2016-10-27 19:08:31 -0700</creation_ts>
          <short_desc>PluginInfoStore::loadPluginsIfNecessary can still load plugins multiple times</short_desc>
          <delta_ts>2016-11-02 02:52:02 -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>Other</version>
          <rep_platform>PC</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          <see_also>https://bugzilla.gnome.org/show_bug.cgi?id=756334</see_also>
    
    <see_also>https://bugs.webkit.org/show_bug.cgi?id=47677</see_also>
          <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>1</everconfirmed>
          <reporter name="Michael Catanzaro">mcatanzaro</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>bugs-noreply</cc>
    
    <cc>cgarcia</cc>
    
    <cc>commit-queue</cc>
    
    <cc>mcatanzaro</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1245397</commentid>
    <comment_count>0</comment_count>
    <who name="Michael Catanzaro">mcatanzaro</who>
    <bug_when>2016-10-27 19:08:31 -0700</bug_when>
    <thetext>PluginInfoStore::loadPluginsIfNecessary is trying hard to make sure plugin paths are unique, but it&apos;s possible for two unique paths to refer to the same directory. For instance, on Arch, /usr/lib64 is a symlink to /usr/lib. So all plugins that we find when scanning /usr/lib64/mozilla/plugins/ get found again when scanning /usr/lib/mozilla/plugins/ because it&apos;s the same directory.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1246414</commentid>
    <comment_count>1</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2016-10-31 07:20:44 -0700</bug_when>
    <thetext>I guess we could follow symlinks when adding paths to uniquePluginPaths</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1246415</commentid>
    <comment_count>2</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2016-10-31 07:33:24 -0700</bug_when>
    <thetext>Looking at about:plugins in firefox, it seems they follow symlinks, because the plugin path they show is always the symlink destination.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1246459</commentid>
    <comment_count>3</comment_count>
      <attachid>293429</attachid>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2016-10-31 09:39:03 -0700</bug_when>
    <thetext>Created attachment 293429
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1246463</commentid>
    <comment_count>4</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2016-10-31 09:41:13 -0700</bug_when>
    <thetext>This patch modifies one of the wasm.json files. Please ensure that any changes in one have been mirrored to the other. You can find the wasm.json files at &quot;Source/JavaScriptCore/wasm/wasm.json&quot; and &quot;JSTests/wasm/wasm.json&quot;.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1246481</commentid>
    <comment_count>5</comment_count>
      <attachid>293429</attachid>
    <who name="Michael Catanzaro">mcatanzaro</who>
    <bug_when>2016-10-31 10:42:28 -0700</bug_when>
    <thetext>Comment on attachment 293429
Patch

Wow, the readlink API is terrible.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1246482</commentid>
    <comment_count>6</comment_count>
    <who name="Michael Catanzaro">mcatanzaro</who>
    <bug_when>2016-10-31 10:44:22 -0700</bug_when>
    <thetext>Don&apos;t break EFL, looks like a missing #include:

../../Source/WebKit2/UIProcess/Plugins/unix/PluginInfoStoreUnix.cpp: In function &apos;WTF::String WebKit::followSymlink(const WTF::String&amp;)&apos;:
../../Source/WebKit2/UIProcess/Plugins/unix/PluginInfoStoreUnix.cpp:69:50: error: &apos;filenameToString&apos; was not declared in this scope
             return filenameToString(buffer.data());
                                                  ^

Also a question: this works not only when the last component of the pathname is not a symlink, but also when an intermediate component is a symlink, right? Since that&apos;s the case we&apos;re trying to fix.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1246875</commentid>
    <comment_count>7</comment_count>
      <attachid>293429</attachid>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2016-11-01 04:34:15 -0700</bug_when>
    <thetext>Comment on attachment 293429
Patch

Good point, we readlink only works if the given path is a link, we need realpath instead.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1246876</commentid>
    <comment_count>8</comment_count>
      <attachid>293548</attachid>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2016-11-01 04:47:08 -0700</bug_when>
    <thetext>Created attachment 293548
Updated patch using realpath

The patch is indeed simpler now :-)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1246877</commentid>
    <comment_count>9</comment_count>
      <attachid>293549</attachid>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2016-11-01 04:55:07 -0700</bug_when>
    <thetext>Created attachment 293549
Fix EFL build

filenameToString is GTK+ specific.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1246880</commentid>
    <comment_count>10</comment_count>
      <attachid>293549</attachid>
    <who name="Michael Catanzaro">mcatanzaro</who>
    <bug_when>2016-11-01 05:17:54 -0700</bug_when>
    <thetext>Comment on attachment 293549
Fix EFL build

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

Bonus points if you tested this manually.

&gt; Source/WebKit2/UIProcess/Plugins/unix/PluginInfoStoreUnix.cpp:64
&gt; +            String pluginPath = String::fromUTF8(normalizedPath);

Looking at the comment above the definition of filenameToString, I think it&apos;d be better to copy that function into FileSystemEfl and use it here unconditionally.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1247266</commentid>
    <comment_count>11</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2016-11-02 02:47:41 -0700</bug_when>
    <thetext>(In reply to comment #10)
&gt; Comment on attachment 293549 [details]
&gt; Fix EFL build
&gt; 
&gt; View in context:
&gt; https://bugs.webkit.org/attachment.cgi?id=293549&amp;action=review
&gt; 
&gt; Bonus points if you tested this manually.
&gt; 
&gt; &gt; Source/WebKit2/UIProcess/Plugins/unix/PluginInfoStoreUnix.cpp:64
&gt; &gt; +            String pluginPath = String::fromUTF8(normalizedPath);
&gt; 
&gt; Looking at the comment above the definition of filenameToString, I think
&gt; it&apos;d be better to copy that function into FileSystemEfl and use it here
&gt; unconditionally.

We already have that :-P stringFromFileSystemRepresentation()</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1247267</commentid>
    <comment_count>12</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2016-11-02 02:52:02 -0700</bug_when>
    <thetext>Committed r208273: &lt;http://trac.webkit.org/changeset/208273&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>293429</attachid>
            <date>2016-10-31 09:39:03 -0700</date>
            <delta_ts>2016-11-01 04:47:08 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>wk2-plugins-duplicated.diff</filename>
            <type>text/plain</type>
            <size>3363</size>
            <attacher name="Carlos Garcia Campos">cgarcia</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQyL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQyL0No
YW5nZUxvZwppbmRleCA0YTY4OTEwLi4wZWY1OTM0IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViS2l0
Mi9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYktpdDIvQ2hhbmdlTG9nCkBAIC0xLDUgKzEsMjAg
QEAKIDIwMTYtMTAtMzEgIENhcmxvcyBHYXJjaWEgQ2FtcG9zICA8Y2dhcmNpYUBpZ2FsaWEuY29t
PgogCisgICAgICAgIFBsdWdpbkluZm9TdG9yZTo6bG9hZFBsdWdpbnNJZk5lY2Vzc2FyeSBjYW4g
c3RpbGwgbG9hZCBwbHVnaW5zIG11bHRpcGxlIHRpbWVzCisgICAgICAgIGh0dHBzOi8vYnVncy53
ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xNjQxMDMKKworICAgICAgICBSZXZpZXdlZCBieSBO
T0JPRFkgKE9PUFMhKS4KKworICAgICAgICBGb2xsb3cgc3ltbGlua3Mgd2hlbiBzY2FubmluZyBw
bHVnaW5zIHRvIGF2b2lkIGR1cGxpY2F0ZXMuCisKKyAgICAgICAgKiBVSVByb2Nlc3MvUGx1Z2lu
cy9ndGsvUGx1Z2luSW5mb0NhY2hlLmNwcDogQnVtcCB0aGUgY2FjaGUgdmVyc2lvbiB0byBlbnN1
cmUgZHVwbGljYXRlZCBwbHVnaW5zIGFyZSByZW1vdmVkCisgICAgICAgIGZyb20gdGhlIGNhY2hl
LgorICAgICAgICAqIFVJUHJvY2Vzcy9QbHVnaW5zL3VuaXgvUGx1Z2luSW5mb1N0b3JlVW5peC5j
cHA6CisgICAgICAgIChXZWJLaXQ6OmZvbGxvd1N5bWxpbmspOiBIZWxwZXIgZnVuY3Rpb24gdG8g
cmVhZCB0aGUgbGluay4KKyAgICAgICAgKFdlYktpdDo6UGx1Z2luSW5mb1N0b3JlOjpwbHVnaW5Q
YXRoc0luRGlyZWN0b3J5KTogQ2FsbCBmb2xsb3dTeW1saW5rIHRoYXQgYWx3YXlzIHJldHVybiB0
aGUgYWN0dWFsIGZpbGUgcGF0aC4KKworMjAxNi0xMC0zMSAgQ2FybG9zIEdhcmNpYSBDYW1wb3Mg
IDxjZ2FyY2lhQGlnYWxpYS5jb20+CisKICAgICAgICAgTmV0d29ya1Nlc3Npb246IE5ldHdvcmsg
cHJvY2VzcyBjcmFzaCB3aGVuIGNvbnZlcnRpbmcgbWFpbiByZXNvdXJjZSB0byBkb3dubG9hZAog
ICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTY0MjIwCiAK
ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy9QbHVnaW5zL2d0ay9QbHVnaW5J
bmZvQ2FjaGUuY3BwIGIvU291cmNlL1dlYktpdDIvVUlQcm9jZXNzL1BsdWdpbnMvZ3RrL1BsdWdp
bkluZm9DYWNoZS5jcHAKaW5kZXggNjRiNGNhYy4uMjI3MGNiYiAxMDA2NDQKLS0tIGEvU291cmNl
L1dlYktpdDIvVUlQcm9jZXNzL1BsdWdpbnMvZ3RrL1BsdWdpbkluZm9DYWNoZS5jcHAKKysrIGIv
U291cmNlL1dlYktpdDIvVUlQcm9jZXNzL1BsdWdpbnMvZ3RrL1BsdWdpbkluZm9DYWNoZS5jcHAK
QEAgLTM1LDcgKzM1LDcgQEAKIAogbmFtZXNwYWNlIFdlYktpdCB7CiAKLXN0YXRpYyBjb25zdCB1
bnNpZ25lZCBnU2NoZW1hVmVyc2lvbiA9IDI7CitzdGF0aWMgY29uc3QgdW5zaWduZWQgZ1NjaGVt
YVZlcnNpb24gPSAzOwogCiBQbHVnaW5JbmZvQ2FjaGUmIFBsdWdpbkluZm9DYWNoZTo6c2luZ2xl
dG9uKCkKIHsKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy9QbHVnaW5zL3Vu
aXgvUGx1Z2luSW5mb1N0b3JlVW5peC5jcHAgYi9Tb3VyY2UvV2ViS2l0Mi9VSVByb2Nlc3MvUGx1
Z2lucy91bml4L1BsdWdpbkluZm9TdG9yZVVuaXguY3BwCmluZGV4IDM1YWZlOWYuLjk3ODA5NTcg
MTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy9QbHVnaW5zL3VuaXgvUGx1Z2lu
SW5mb1N0b3JlVW5peC5jcHAKKysrIGIvU291cmNlL1dlYktpdDIvVUlQcm9jZXNzL1BsdWdpbnMv
dW5peC9QbHVnaW5JbmZvU3RvcmVVbml4LmNwcApAQCAtNDksMTQgKzQ5LDQwIEBAIFZlY3RvcjxT
dHJpbmc+IFBsdWdpbkluZm9TdG9yZTo6cGx1Z2luc0RpcmVjdG9yaWVzKCkKICAgICByZXR1cm4g
V2ViS2l0OjpwbHVnaW5zRGlyZWN0b3JpZXMoKTsKIH0KIAorc3RhdGljIFN0cmluZyBmb2xsb3dT
eW1saW5rKGNvbnN0IFN0cmluZyYgcGF0aCkKK3sKKyAgICBDU3RyaW5nIGZpbGVuYW1lID0gZmls
ZVN5c3RlbVJlcHJlc2VudGF0aW9uKHBhdGgpOworICAgIFZlY3RvcjxjaGFyPiBidWZmZXIoMjU2
KTsKKyAgICB3aGlsZSAodHJ1ZSkgeworICAgICAgICBzc2l6ZV90IHJlc3VsdCA9IHJlYWRsaW5r
KGZpbGVuYW1lLmRhdGEoKSwgYnVmZmVyLmRhdGEoKSwgYnVmZmVyLnNpemUoKSk7CisgICAgICAg
IGlmIChyZXN1bHQgPT0gLTEpIHsKKyAgICAgICAgICAgIGlmIChlcnJubyA9PSBFSU5WQUwpIHsK
KyAgICAgICAgICAgICAgICAvLyBOb3QgYSBzeW1saW5rLCBqdXN0IHJldHVybiB0aGUgcGF0aC4K
KyAgICAgICAgICAgICAgICByZXR1cm4gcGF0aDsKKyAgICAgICAgICAgIH0KKworICAgICAgICAg
ICAgcmV0dXJuIFN0cmluZygpOworICAgICAgICB9CisKKyAgICAgICAgaWYgKHN0YXRpY19jYXN0
PHNpemVfdD4ocmVzdWx0KSA8IGJ1ZmZlci5zaXplKCkpIHsKKyAgICAgICAgICAgIGJ1ZmZlclty
ZXN1bHRdID0gJ1wwJzsKKyAgICAgICAgICAgIHJldHVybiBmaWxlbmFtZVRvU3RyaW5nKGJ1ZmZl
ci5kYXRhKCkpOworICAgICAgICB9CisKKyAgICAgICAgLy8gQnVmZmVyIHdhcyB0cnVuY2F0ZWQs
IHRyeSBhZ2FpbiB3aXRoIGEgbGFyZ2VyIG9uZS4KKyAgICAgICAgYnVmZmVyLmdyb3coYnVmZmVy
LnNpemUoKSAqIDIpOworICAgIH0KKworICAgIHJldHVybiBTdHJpbmcoKTsKK30KKwogVmVjdG9y
PFN0cmluZz4gUGx1Z2luSW5mb1N0b3JlOjpwbHVnaW5QYXRoc0luRGlyZWN0b3J5KGNvbnN0IFN0
cmluZyYgZGlyZWN0b3J5KQogewogICAgIFZlY3RvcjxTdHJpbmc+IHJlc3VsdDsKLSAgICBWZWN0
b3I8U3RyaW5nPiBwbHVnaW5QYXRocyA9IGxpc3REaXJlY3RvcnkoZGlyZWN0b3J5LCBTdHJpbmco
Iiouc28iKSk7Ci0gICAgVmVjdG9yPFN0cmluZz46OmNvbnN0X2l0ZXJhdG9yIGVuZCA9IHBsdWdp
blBhdGhzLmVuZCgpOwotICAgIGZvciAoVmVjdG9yPFN0cmluZz46OmNvbnN0X2l0ZXJhdG9yIGl0
ID0gcGx1Z2luUGF0aHMuYmVnaW4oKTsgaXQgIT0gZW5kOyArK2l0KSB7Ci0gICAgICAgIGlmIChm
aWxlRXhpc3RzKCppdCkpCi0gICAgICAgICAgICByZXN1bHQuYXBwZW5kKCppdCk7CisgICAgZm9y
IChjb25zdCBhdXRvJiBwYXRoIDogbGlzdERpcmVjdG9yeShkaXJlY3RvcnksIFN0cmluZygiKi5z
byIpKSkgeworICAgICAgICBTdHJpbmcgcGx1Z2luUGF0aCA9IGZvbGxvd1N5bWxpbmsocGF0aCk7
CisgICAgICAgIGlmICghcGx1Z2luUGF0aC5pc051bGwoKSkKKyAgICAgICAgICAgIHJlc3VsdC5h
cHBlbmQoV1RGTW92ZShwbHVnaW5QYXRoKSk7CiAgICAgfQogCiAgICAgcmV0dXJuIHJlc3VsdDsK
</data>
<flag name="review"
          id="316306"
          type_id="1"
          status="+"
          setter="mcatanzaro"
    />
    <flag name="commit-queue"
          id="316409"
          type_id="3"
          status="-"
          setter="cgarcia"
    />
          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>293548</attachid>
            <date>2016-11-01 04:47:08 -0700</date>
            <delta_ts>2016-11-01 04:55:07 -0700</delta_ts>
            <desc>Updated patch using realpath</desc>
            <filename>wk2-duplicated-plugins.diff</filename>
            <type>text/plain</type>
            <size>2719</size>
            <attacher name="Carlos Garcia Campos">cgarcia</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQyL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQyL0No
YW5nZUxvZwppbmRleCAzNDgzNjU5Li4xNTMzNDFhIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViS2l0
Mi9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYktpdDIvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTcg
QEAKKzIwMTYtMTAtMzEgIENhcmxvcyBHYXJjaWEgQ2FtcG9zICA8Y2dhcmNpYUBpZ2FsaWEuY29t
PgorCisgICAgICAgIFBsdWdpbkluZm9TdG9yZTo6bG9hZFBsdWdpbnNJZk5lY2Vzc2FyeSBjYW4g
c3RpbGwgbG9hZCBwbHVnaW5zIG11bHRpcGxlIHRpbWVzCisgICAgICAgIGh0dHBzOi8vYnVncy53
ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xNjQxMDMKKworICAgICAgICBSZXZpZXdlZCBieSBO
T0JPRFkgKE9PUFMhKS4KKworICAgICAgICBGb2xsb3cgc3ltbGlua3Mgd2hlbiBzY2FubmluZyBw
bHVnaW5zIHRvIGF2b2lkIGR1cGxpY2F0ZXMuCisKKyAgICAgICAgKiBVSVByb2Nlc3MvUGx1Z2lu
cy9ndGsvUGx1Z2luSW5mb0NhY2hlLmNwcDogQnVtcCB0aGUgY2FjaGUgdmVyc2lvbiB0byBlbnN1
cmUgZHVwbGljYXRlZCBwbHVnaW5zIGFyZSByZW1vdmVkCisgICAgICAgIGZyb20gdGhlIGNhY2hl
LgorICAgICAgICAqIFVJUHJvY2Vzcy9QbHVnaW5zL3VuaXgvUGx1Z2luSW5mb1N0b3JlVW5peC5j
cHA6CisgICAgICAgIChXZWJLaXQ6OlBsdWdpbkluZm9TdG9yZTo6cGx1Z2luUGF0aHNJbkRpcmVj
dG9yeSk6IFVzZSByZWFscGF0aCgpIHRvIGFsd2F5cyByZXR1cm4gdGhlIGFjdHVhbCBmaWxlIHBh
dGguCisKIDIwMTYtMTEtMDEgIENhcmxvcyBHYXJjaWEgQ2FtcG9zICA8Y2dhcmNpYUBpZ2FsaWEu
Y29tPgogCiAgICAgICAgIE5ldHdvcmtTZXNzaW9uOiBOZXR3b3JrIHByb2Nlc3MgY3Jhc2ggd2hl
biBjb252ZXJ0aW5nIG1haW4gcmVzb3VyY2UgdG8gZG93bmxvYWQKZGlmZiAtLWdpdCBhL1NvdXJj
ZS9XZWJLaXQyL1VJUHJvY2Vzcy9QbHVnaW5zL2d0ay9QbHVnaW5JbmZvQ2FjaGUuY3BwIGIvU291
cmNlL1dlYktpdDIvVUlQcm9jZXNzL1BsdWdpbnMvZ3RrL1BsdWdpbkluZm9DYWNoZS5jcHAKaW5k
ZXggNjRiNGNhYy4uMjI3MGNiYiAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdDIvVUlQcm9jZXNz
L1BsdWdpbnMvZ3RrL1BsdWdpbkluZm9DYWNoZS5jcHAKKysrIGIvU291cmNlL1dlYktpdDIvVUlQ
cm9jZXNzL1BsdWdpbnMvZ3RrL1BsdWdpbkluZm9DYWNoZS5jcHAKQEAgLTM1LDcgKzM1LDcgQEAK
IAogbmFtZXNwYWNlIFdlYktpdCB7CiAKLXN0YXRpYyBjb25zdCB1bnNpZ25lZCBnU2NoZW1hVmVy
c2lvbiA9IDI7CitzdGF0aWMgY29uc3QgdW5zaWduZWQgZ1NjaGVtYVZlcnNpb24gPSAzOwogCiBQ
bHVnaW5JbmZvQ2FjaGUmIFBsdWdpbkluZm9DYWNoZTo6c2luZ2xldG9uKCkKIHsKZGlmZiAtLWdp
dCBhL1NvdXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy9QbHVnaW5zL3VuaXgvUGx1Z2luSW5mb1N0b3Jl
VW5peC5jcHAgYi9Tb3VyY2UvV2ViS2l0Mi9VSVByb2Nlc3MvUGx1Z2lucy91bml4L1BsdWdpbklu
Zm9TdG9yZVVuaXguY3BwCmluZGV4IDM1YWZlOWYuLjBhOTVjZGIgMTAwNjQ0Ci0tLSBhL1NvdXJj
ZS9XZWJLaXQyL1VJUHJvY2Vzcy9QbHVnaW5zL3VuaXgvUGx1Z2luSW5mb1N0b3JlVW5peC5jcHAK
KysrIGIvU291cmNlL1dlYktpdDIvVUlQcm9jZXNzL1BsdWdpbnMvdW5peC9QbHVnaW5JbmZvU3Rv
cmVVbml4LmNwcApAQCAtMzUsNiArMzUsOCBAQAogI2luY2x1ZGUgIlBsdWdpblNlYXJjaFBhdGgu
aCIKICNpbmNsdWRlICJQcm9jZXNzRXhlY3V0YWJsZVBhdGguaCIKICNpbmNsdWRlIDxXZWJDb3Jl
L0ZpbGVTeXN0ZW0uaD4KKyNpbmNsdWRlIDxsaW1pdHMuaD4KKyNpbmNsdWRlIDxzdGRsaWIuaD4K
IAogI2lmIFBMQVRGT1JNKEdUSykKICNpbmNsdWRlICJQbHVnaW5JbmZvQ2FjaGUuaCIKQEAgLTUy
LDExICs1NCwxMSBAQCBWZWN0b3I8U3RyaW5nPiBQbHVnaW5JbmZvU3RvcmU6OnBsdWdpbnNEaXJl
Y3RvcmllcygpCiBWZWN0b3I8U3RyaW5nPiBQbHVnaW5JbmZvU3RvcmU6OnBsdWdpblBhdGhzSW5E
aXJlY3RvcnkoY29uc3QgU3RyaW5nJiBkaXJlY3RvcnkpCiB7CiAgICAgVmVjdG9yPFN0cmluZz4g
cmVzdWx0OwotICAgIFZlY3RvcjxTdHJpbmc+IHBsdWdpblBhdGhzID0gbGlzdERpcmVjdG9yeShk
aXJlY3RvcnksIFN0cmluZygiKi5zbyIpKTsKLSAgICBWZWN0b3I8U3RyaW5nPjo6Y29uc3RfaXRl
cmF0b3IgZW5kID0gcGx1Z2luUGF0aHMuZW5kKCk7Ci0gICAgZm9yIChWZWN0b3I8U3RyaW5nPjo6
Y29uc3RfaXRlcmF0b3IgaXQgPSBwbHVnaW5QYXRocy5iZWdpbigpOyBpdCAhPSBlbmQ7ICsraXQp
IHsKLSAgICAgICAgaWYgKGZpbGVFeGlzdHMoKml0KSkKLSAgICAgICAgICAgIHJlc3VsdC5hcHBl
bmQoKml0KTsKKyAgICBjaGFyIG5vcm1hbGl6ZWRQYXRoW1BBVEhfTUFYXTsKKyAgICBmb3IgKGNv
bnN0IGF1dG8mIHBhdGggOiBsaXN0RGlyZWN0b3J5KGRpcmVjdG9yeSwgU3RyaW5nKCIqLnNvIikp
KSB7CisgICAgICAgIENTdHJpbmcgZmlsZW5hbWUgPSBmaWxlU3lzdGVtUmVwcmVzZW50YXRpb24o
cGF0aCk7CisgICAgICAgIGlmIChyZWFscGF0aChmaWxlbmFtZS5kYXRhKCksIG5vcm1hbGl6ZWRQ
YXRoKSkKKyAgICAgICAgICAgIHJlc3VsdC5hcHBlbmQoZmlsZW5hbWVUb1N0cmluZyhub3JtYWxp
emVkUGF0aCkpOwogICAgIH0KIAogICAgIHJldHVybiByZXN1bHQ7Cg==
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>293549</attachid>
            <date>2016-11-01 04:55:07 -0700</date>
            <delta_ts>2016-11-01 05:17:54 -0700</delta_ts>
            <desc>Fix EFL build</desc>
            <filename>wk2-duplicated-plugins.diff</filename>
            <type>text/plain</type>
            <size>2887</size>
            <attacher name="Carlos Garcia Campos">cgarcia</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQyL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQyL0No
YW5nZUxvZwppbmRleCAzNDgzNjU5Li4xNTMzNDFhIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViS2l0
Mi9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYktpdDIvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTcg
QEAKKzIwMTYtMTAtMzEgIENhcmxvcyBHYXJjaWEgQ2FtcG9zICA8Y2dhcmNpYUBpZ2FsaWEuY29t
PgorCisgICAgICAgIFBsdWdpbkluZm9TdG9yZTo6bG9hZFBsdWdpbnNJZk5lY2Vzc2FyeSBjYW4g
c3RpbGwgbG9hZCBwbHVnaW5zIG11bHRpcGxlIHRpbWVzCisgICAgICAgIGh0dHBzOi8vYnVncy53
ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xNjQxMDMKKworICAgICAgICBSZXZpZXdlZCBieSBO
T0JPRFkgKE9PUFMhKS4KKworICAgICAgICBGb2xsb3cgc3ltbGlua3Mgd2hlbiBzY2FubmluZyBw
bHVnaW5zIHRvIGF2b2lkIGR1cGxpY2F0ZXMuCisKKyAgICAgICAgKiBVSVByb2Nlc3MvUGx1Z2lu
cy9ndGsvUGx1Z2luSW5mb0NhY2hlLmNwcDogQnVtcCB0aGUgY2FjaGUgdmVyc2lvbiB0byBlbnN1
cmUgZHVwbGljYXRlZCBwbHVnaW5zIGFyZSByZW1vdmVkCisgICAgICAgIGZyb20gdGhlIGNhY2hl
LgorICAgICAgICAqIFVJUHJvY2Vzcy9QbHVnaW5zL3VuaXgvUGx1Z2luSW5mb1N0b3JlVW5peC5j
cHA6CisgICAgICAgIChXZWJLaXQ6OlBsdWdpbkluZm9TdG9yZTo6cGx1Z2luUGF0aHNJbkRpcmVj
dG9yeSk6IFVzZSByZWFscGF0aCgpIHRvIGFsd2F5cyByZXR1cm4gdGhlIGFjdHVhbCBmaWxlIHBh
dGguCisKIDIwMTYtMTEtMDEgIENhcmxvcyBHYXJjaWEgQ2FtcG9zICA8Y2dhcmNpYUBpZ2FsaWEu
Y29tPgogCiAgICAgICAgIE5ldHdvcmtTZXNzaW9uOiBOZXR3b3JrIHByb2Nlc3MgY3Jhc2ggd2hl
biBjb252ZXJ0aW5nIG1haW4gcmVzb3VyY2UgdG8gZG93bmxvYWQKZGlmZiAtLWdpdCBhL1NvdXJj
ZS9XZWJLaXQyL1VJUHJvY2Vzcy9QbHVnaW5zL2d0ay9QbHVnaW5JbmZvQ2FjaGUuY3BwIGIvU291
cmNlL1dlYktpdDIvVUlQcm9jZXNzL1BsdWdpbnMvZ3RrL1BsdWdpbkluZm9DYWNoZS5jcHAKaW5k
ZXggNjRiNGNhYy4uMjI3MGNiYiAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdDIvVUlQcm9jZXNz
L1BsdWdpbnMvZ3RrL1BsdWdpbkluZm9DYWNoZS5jcHAKKysrIGIvU291cmNlL1dlYktpdDIvVUlQ
cm9jZXNzL1BsdWdpbnMvZ3RrL1BsdWdpbkluZm9DYWNoZS5jcHAKQEAgLTM1LDcgKzM1LDcgQEAK
IAogbmFtZXNwYWNlIFdlYktpdCB7CiAKLXN0YXRpYyBjb25zdCB1bnNpZ25lZCBnU2NoZW1hVmVy
c2lvbiA9IDI7CitzdGF0aWMgY29uc3QgdW5zaWduZWQgZ1NjaGVtYVZlcnNpb24gPSAzOwogCiBQ
bHVnaW5JbmZvQ2FjaGUmIFBsdWdpbkluZm9DYWNoZTo6c2luZ2xldG9uKCkKIHsKZGlmZiAtLWdp
dCBhL1NvdXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy9QbHVnaW5zL3VuaXgvUGx1Z2luSW5mb1N0b3Jl
VW5peC5jcHAgYi9Tb3VyY2UvV2ViS2l0Mi9VSVByb2Nlc3MvUGx1Z2lucy91bml4L1BsdWdpbklu
Zm9TdG9yZVVuaXguY3BwCmluZGV4IDM1YWZlOWYuLjI1NWY3ZWIgMTAwNjQ0Ci0tLSBhL1NvdXJj
ZS9XZWJLaXQyL1VJUHJvY2Vzcy9QbHVnaW5zL3VuaXgvUGx1Z2luSW5mb1N0b3JlVW5peC5jcHAK
KysrIGIvU291cmNlL1dlYktpdDIvVUlQcm9jZXNzL1BsdWdpbnMvdW5peC9QbHVnaW5JbmZvU3Rv
cmVVbml4LmNwcApAQCAtMzUsNiArMzUsOCBAQAogI2luY2x1ZGUgIlBsdWdpblNlYXJjaFBhdGgu
aCIKICNpbmNsdWRlICJQcm9jZXNzRXhlY3V0YWJsZVBhdGguaCIKICNpbmNsdWRlIDxXZWJDb3Jl
L0ZpbGVTeXN0ZW0uaD4KKyNpbmNsdWRlIDxsaW1pdHMuaD4KKyNpbmNsdWRlIDxzdGRsaWIuaD4K
IAogI2lmIFBMQVRGT1JNKEdUSykKICNpbmNsdWRlICJQbHVnaW5JbmZvQ2FjaGUuaCIKQEAgLTUy
LDExICs1NCwxNyBAQCBWZWN0b3I8U3RyaW5nPiBQbHVnaW5JbmZvU3RvcmU6OnBsdWdpbnNEaXJl
Y3RvcmllcygpCiBWZWN0b3I8U3RyaW5nPiBQbHVnaW5JbmZvU3RvcmU6OnBsdWdpblBhdGhzSW5E
aXJlY3RvcnkoY29uc3QgU3RyaW5nJiBkaXJlY3RvcnkpCiB7CiAgICAgVmVjdG9yPFN0cmluZz4g
cmVzdWx0OwotICAgIFZlY3RvcjxTdHJpbmc+IHBsdWdpblBhdGhzID0gbGlzdERpcmVjdG9yeShk
aXJlY3RvcnksIFN0cmluZygiKi5zbyIpKTsKLSAgICBWZWN0b3I8U3RyaW5nPjo6Y29uc3RfaXRl
cmF0b3IgZW5kID0gcGx1Z2luUGF0aHMuZW5kKCk7Ci0gICAgZm9yIChWZWN0b3I8U3RyaW5nPjo6
Y29uc3RfaXRlcmF0b3IgaXQgPSBwbHVnaW5QYXRocy5iZWdpbigpOyBpdCAhPSBlbmQ7ICsraXQp
IHsKLSAgICAgICAgaWYgKGZpbGVFeGlzdHMoKml0KSkKLSAgICAgICAgICAgIHJlc3VsdC5hcHBl
bmQoKml0KTsKKyAgICBjaGFyIG5vcm1hbGl6ZWRQYXRoW1BBVEhfTUFYXTsKKyAgICBmb3IgKGNv
bnN0IGF1dG8mIHBhdGggOiBsaXN0RGlyZWN0b3J5KGRpcmVjdG9yeSwgU3RyaW5nKCIqLnNvIikp
KSB7CisgICAgICAgIENTdHJpbmcgZmlsZW5hbWUgPSBmaWxlU3lzdGVtUmVwcmVzZW50YXRpb24o
cGF0aCk7CisgICAgICAgIGlmIChyZWFscGF0aChmaWxlbmFtZS5kYXRhKCksIG5vcm1hbGl6ZWRQ
YXRoKSkgeworI2lmIFBMQVRGT1JNKEdUSykKKyAgICAgICAgICAgIFN0cmluZyBwbHVnaW5QYXRo
ID0gZmlsZW5hbWVUb1N0cmluZyhub3JtYWxpemVkUGF0aCk7CisjZWxzZQorICAgICAgICAgICAg
U3RyaW5nIHBsdWdpblBhdGggPSBTdHJpbmc6OmZyb21VVEY4KG5vcm1hbGl6ZWRQYXRoKTsKKyNl
bmRpZgorICAgICAgICAgICAgcmVzdWx0LmFwcGVuZChXVEZNb3ZlKHBsdWdpblBhdGgpKTsKKyAg
ICAgICAgfQogICAgIH0KIAogICAgIHJldHVybiByZXN1bHQ7Cg==
</data>
<flag name="review"
          id="316411"
          type_id="1"
          status="+"
          setter="mcatanzaro"
    />
          </attachment>
      

    </bug>

</bugzilla>