Global memcached groups names

[rebeccahum]

Bug Description

We should not allow use of global memcached cache group names, as they are being used on a global scale and things go wrong (such as user login) when they are overwritten: https://github.com/Automattic/wp-memcached

Minimal Code Snippet

wp_cache_set( 'blahblah', 'blah', 'users', 1 * MINUTE_IN_SECONDS );

Tested Against master branch?

• I have verified the issue still exists in the master branch of VIPCS.

[GaryJones]

global memcached cache group names

Have you got a list of those @rebeccahum?

Can you give more examples of what should be flagged, and some use cases that shouldn't?

[rebeccahum]

In the readme.txt, the list seems to be here:

{$taxonomy}_relationships
{$meta_type}_meta
{$taxonomy}_relationships
blog-details
blog-id-cache
blog-lookup
bookmark
calendar
category
comment
counts
general
global-posts
options
plugins
post_ancestors
post_meta
posts
rss
site-lookup
site-options
site-transient
terms
themes
timeinfo
transient
user_meta
useremail
userlogins
usermeta
users
userslugs
widget

[GaryJones]

So, new sniff, look for wp_cache_set function calls, check third arg is one of those from the list, and give an Error, severity 5.