If the the socket is connected and lpFrom is not NULL, Linux will set
msg_namelen to zero and ignore msg_name in recvfrom so don't try try
to update lpFrom.

This allows applications to wait for state transitions to be really
complete. Fixes some xvid crashes.

Prevents a null pointer dereference.

Should fix Valgrind warnings 'Conditional jump or move depends on
uninitialised value(s)' in inetcomm tests.

The conformance needs to be added on to the offset in the buffer so
set this before calling each the writer of each class of pointer
description in write_pointer_description.

Pass the passed in offsets to buffer and memory to
write_pointer_description_offsets in
write_varying_array_pointer_descriptions.

rpcrt4: Use abrtitrary pointer identifiers when marshalling pointers instead of using pointer values.

This ensures that no information about the server memory layout is
leaked to clients.

Free the memory associated with the field and value of the custom
header otherwise it will be leaked. Found by Valgrind.

This ensures that it's also done correctly when a read from the pipe
returns EOF.

The combobox to select the alignment in the paragraph format dialog
did not have enough height to display the dropdown list.  Also, code was
missing to set the alignment when OK button is pressed. This patch fixes
these issues.

The changes were previously being applied only when the combobox's edit
control lost focus notifying an end edit event.  In addition, the
selection of an item from the dropdown list should also cause the
changes to be applied. New functions were created to handle the
duplication in expected behaviour for these two events.

When run with the native richedit control the selection will be hidden
when focus is lost without the ES_NOHIDESEL style applied to the
richedit control.  This can be annoying in native windows when focus is
on a combobox in the toolbar, because it won't show what text will be
changed.  This patch adds the ES_NOHIDESEL style to be consistent with
native wordpad.