I noticed a bunch of unchecked strdup's in a row,
and audited the rest of the file:
Handle failed strdup and malloc.
* src/remote_internal.c: Don't dereference NULL after
failed strdup or malloc in doRemoteOpen.

This is all good stuff, except that the calls to error () should take
the conn (virConnectPtr) as first argument if conn is available, which
it is here. So +1 if that change is made.