[lvm-devel] [PATCH 19/29] Test for error status

Alasdair G Kergon agk at redhat.com
Mon Nov 29 21:02:14 UTC 2010


On Thu, Nov 25, 2010 at 11:55:23AM +0100, Zdenek Kabelac wrote:
> +		if (write(child_pipe[1], &status, sizeof(status)) < 0)
> +			log_sys_error("write", "child_pipe");

Still incomplete - add FIXME to use a proper wrapper around write().

> +		if (close(child_pipe[1]))
> +			log_sys_error("close", "child_pipe");

Ack.

Alasdair




More information about the lvm-devel mailing list