Regression by commit 2d8847f4

Closes https://github.com/MusicPlayerDaemon/MPD/issues/1106

Closes https://github.com/MusicPlayerDaemon/MPD/issues/1285

Without clearing the "uri" field, the next Open() call attempts to
reuse the old InputStream, but it has already been closed, so Open()
always returns nullptr.

Closes https://github.com/MusicPlayerDaemon/MPD/issues/1300

Almost all callers have string literal, and the length is known at
compile time.

The inverted check was introduced by commit 46d00dd8, and commit
8ad17d25 added a check for the wrong variable.  D'oh!

Closes https://github.com/MusicPlayerDaemon/MPD/issues/1303

Closes https://github.com/MusicPlayerDaemon/MPD/issues/1297