Page 1 of 4 123 ... LastLast
Results 1 to 10 of 32

Thread: vSlider plugin injecting hidden code into page footers

  1. #1
    Ryan's Avatar
    Ryan is offline WordPress Legend
    Join Date
    Jan 2009
    Location
    New Zealand
    Posts
    2,801

    Default vSlider plugin injecting hidden code into page footers

    A client of mine asked me to help them install the following plugin:
    http://wordpress.org/extend/plugins/vslider/

    I quickly discovered this plugin contains a few intentional nasties and should not be used:
    PHP Code:
    <?php

    function vSlider_link() { ?>
    <noscript><a href="http://www.vibethemes.com/" target="_blank" title="wordpress themes">Vibe Themes</a></noscript>
    <?php }

    // other code here

    add_action('wp_footer''vSlider_link');

    ?>
    That code injects a link into your footer. However it only shows up when you have javascript turned off, hence no one has probably noticed yet.

    Just posting this here in case any of you Taverner's happen to use the plugin.

  2. #2
    williamsba's Avatar
    williamsba is offline Here For The Peanuts
    Join Date
    Jan 2009
    Location
    New Jersey
    Posts
    181

    Default

    Wow, that's a really shady technique. I'll share with @mark_r, and I'm sure he'll remove the plugin from the WP.org directory because of this.

  3. #3
    Kim's Avatar
    Kim
    Kim is offline Hello World
    Join Date
    Jan 2009
    Location
    Ohio
    Posts
    61

    Default

    This was reported on the wordpress.org forum 5 months ago, and the plugin's still in the repo? http://wordpress.org/support/topic/347001

    And that exact issue is listed in their FAQ on wordpress.org. How did this one get accepted into the repo in the first place?

  4. #4
    conorp's Avatar
    conorp is offline Kegger
    Join Date
    Jan 2009
    Location
    Australia
    Posts
    506

    Default

    Read the Faqs
    http://wordpress.org/extend/plugins/vslider/faq/
    This plugin injects a hidden link into the footer. It's unusable until it's fixed.

    It is a link to the author site wrapped in noscript tag like most traffic tracking services does. If it was a theme you would have no problem with a footer link but if it�s a plugin why is a different story?
    The lord of every land, rising for them,
    The Aton of the day, great of majesty.

    Great Hymn of the Aton

  5. #5
    williamsba's Avatar
    williamsba is offline Here For The Peanuts
    Join Date
    Jan 2009
    Location
    New Jersey
    Posts
    181

    Default

    The requirements for plugin inclusion state any links have to be opt-in, not opt-out which is the case here. http://wordpress.org/extend/plugins/about/

  6. #6
    Ryan's Avatar
    Ryan is offline WordPress Legend
    Join Date
    Jan 2009
    Location
    New Zealand
    Posts
    2,801

    Default

    I've emailed him via plugins@wordpress dot org already, but if you have a faster way of tracking him down that would be great :)

  7. #7
    Ipstenu's Avatar
    Ipstenu is offline Big Tipper
    Join Date
    Feb 2010
    Posts
    368

    Default

    To the Read the FAQ comment, you should too because of this:
    http://wordpress.org/extend/plugins/about/

    There are only a few restrictions
    [...]
    4. The plugin must not embed external links on the public site (like a "powered by" link) without explicitly asking the user's permission.
    [...]
    Last edited by Ipstenu; 06-11-2010 at 09:46 AM. Reason: Because showing WHO you reply to matters!

  8. #8
    Ryan's Avatar
    Ryan is offline WordPress Legend
    Join Date
    Jan 2009
    Location
    New Zealand
    Posts
    2,801

    Default

    Quote Originally Posted by Ipstenu View Post
    Perhaps what?

    Not sure what you are referring to there as the link you posted states that you aren't allowed to embed external links without asking permission.

  9. #9
    Ipstenu's Avatar
    Ipstenu is offline Big Tipper
    Join Date
    Feb 2010
    Posts
    368

    Default

    I was replying to this post and my PC hung up a moment so... that came out weird... *sigh*

    Point I was trying to make is this: You cannot tell someone 'read the faq, that's why and we're covered.' if you, in return, aren't reading and complying to the OVERALL faq.

  10. #10
    conorp's Avatar
    conorp is offline Kegger
    Join Date
    Jan 2009
    Location
    Australia
    Posts
    506

    Default

    Quote Originally Posted by Ipstenu View Post
    I was replying to this post and my PC hung up a moment so... that came out weird... *sigh*

    Point I was trying to make is this: You cannot tell someone 'read the faq, that's why and we're covered.' if you, in return, aren't reading and complying to the OVERALL faq.
    I'm actually against this plugin btw.

    I was just showing that they actually admit to it, compared to many who just chuck them in the footer.

    Still should be pulled from the repo though.
    The lord of every land, rising for them,
    The Aton of the day, great of majesty.

    Great Hymn of the Aton

Page 1 of 4 123 ... LastLast

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •