wined3d: Always handle WINED3DSPR_INPUT registers as input registers in shader_get_registers_used().

The "attributes" vertexshader field is now derived from the input signature,
and only used to speed up matching D3D9 vertex declaration elements to shader
inputs. D3D8 and D3D10 both explicitly specify input registers.

The original implementation treated the string target in IHlink as a
separate entity from the moniker target.  In reality, the string target 
is just the moniker target's display name and setting one reference also
sets the other.

jscript: Make sure retv pointer is initialised in exec_source even if no value needs to be returned.

Spotted by Gregor Schneider.

ntdll: Fix typo in RTL_ReportRegistryValue which caused a counted string to be passed into QueryFunction which expects a nul-terminated string.

The documentation for libmpg123 clearly state that mpg123_getformat MUST
be called if an error MPG123_NEW_FORMAT is returned, for libmpg123 < 1.8.0.
On versions above that, the call is optional. The patch make sure to call
that function all the time (instead of just when trace is activated) so
that mp3 works in Morrowind and others.

Reported by David Quintana.

secur32: Don't allocate context handle in wrapper InitializeSecurityContextA/W if it is the same as the handle passed into the function.

secur32: Simplify memory management by not allocating memory for the CredHandle and CtxtHandle pointers.

Instead store the handles directly in the SspiData structure and pass
the addresses of these into SSPI functions.