[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [Libvir] [PATCH] Handle failed strdup and malloc.



Jim Meyering wrote:
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.

Rich.

--
Emerging Technologies, Red Hat - http://et.redhat.com/~rjones/
Registered Address: Red Hat UK Ltd, Amberley Place, 107-111 Peascod
Street, Windsor, Berkshire, SL4 1TE, United Kingdom.  Registered in
England and Wales under Company Registration No. 03798903

Attachment: smime.p7s
Description: S/MIME Cryptographic Signature


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]