Fix unconditional 'true' return from StartUPnP() if HAS_UPNP is true.
Fix possible memory leak in Settings.cpp
Always close test socket in CWebServer::Start()
Convert #defines to real integral constants in SmartPlaylist.cpp

The name "group" used in these macros gets confused with the name
of the parameter "group" in various functions in this file when the
macro is expanded (See translateGroup(), etc).

This causes the sizeof() calculation to use the size of a pointer,
rather than the intended size.
Fix incorrect iterator comparison in CAlbum::Save()
Remove NULL check that can never be true.

We've already validated that render is non-null on all possible
control paths at this point.
Fix incorrect virtual override in ProfilesManager
Fix validation checks in Deserialize()
Fix possible memory leaks in iso9660.cpp
Fix possible integer overflow in udf25.cpp
Avoid possible divide-by-0 in DirectXGraphics.cpp
Check return value of GetResumeBookMark()
Fix incorret iterator comparison in MusicInfoScanner.cpp
Prevent possible divide-by-0 in karaokevideobackground.cpp
Check return of musicUrl.FromString() in GUIWindowMusicBase.cpp
Avoid incorrect signed-unsigned conversion on error in GUIWindowLoginScreen.cpp
[AEChannelInfo] The condition should be a conjunction.

This additionally reverts commit add8fab77bd3a646569164a79d96f1ee2b57ac0f.
[CoreAudio/IOS]  Assignment of function parameter has no effect outside the function.
[Cores] Style: Reduce scope of variables.
[PulseAESound] Correct oder of initializer list.