At Wed, 17 Apr 2013 11:05:26 +0800,
Liu Yuan wrote:
>
> On 04/16/2013 05:04 PM, Hitoshi Mitake wrote:
> > + ret = rename(tmp_config_path, config_path);
> > + if (ret < 0) {
> > + sd_eprintf("failed to rename, %m");
> > + ret = SD_RES_EIO;
> > + }
> > +
> > + unlink(tmp_config_path);
>
> rename already implies unlink() the old dentry. unlink(tmp_config_path)
> isn't necessary.
Oops, sorry for the silly bug. Thanks for your catching.
Hitoshi
--
sheepdog mailing list
[email protected]
http://lists.wpkg.org/mailman/listinfo/sheepdog
