[sheepdog] [PATCH 2/3] util:add a new API atomic_create_and_write()

Liu Yuan namei.unix at gmail.com
Wed Apr 17 07:55:09 CEST 2013


On 04/17/2013 01:45 PM, Hitoshi Mitake wrote:
> +	if (!access(tmp_path, F_OK))
> +		panic("temporal file: %s exists, this implies invalid usage of"
> +			" atomic_create_and_write() of sheepdog", tmp_path);

panic() is too stringent and not necessary. open(2) will return error if
it has any problem with the tmp_path. So I'd suggest remove this if clause.

Thanks,
Yuan



More information about the sheepdog mailing list