I also refactored error handling. Should be cleaner now.

Nested conditionals are the devil. Printing should be consistent now.

Oops. This is the problem with overloading methods.

Badchars apply to all targets.

Win some, lose some. php -r spawns a new (obvious) command. :/

Check method and version detection also rewritten. :)

I really couldn't decide, especially once I got rid of CmdStager.

Also fully document the module options.

Oops.

Also spiff up comments.

This was supposed to imply that we couldn't configure the exploit for a
targetable version. Instead, it just read weirdly. I think it was
missing "to target" at the end. "Determine" is a much better word,
though, since we may be doing detection instead of mere configuration.

The block quote was ripped directly from the module description. It
isn't necessary in the dedicated documentation. Reads better now.

Reads a little better now.

Additionally drop a file in a writable directory in case CWD fails.

Makes sense to me. Even though it's technically the advisory.

Looks like 8.x has core/CHANGELOG.txt instead.

We're no longer matching multiple targets like /In-Memory/ or /Dropper/,
so it makes sense to match on a specific value now.

Old matching in this commit: 1900aa2.