2009/3/26 Kirby Zhou <[email protected]>: > My patch: > > ############## > --- glusterfs-2.0.0rc7.ori/xlators/storage/posix/src/posix.c 2009-03-26 > 18:50:11.000000000 +0800 > +++ glusterfs-2.0.0rc7/xlators/storage/posix/src/posix.c 2009-03-26 > 18:50:19.000000000 +0800 > @@ -866,11 +866,8 @@ > > fd = open (real_path, O_RDONLY); > if (fd == -1) { > - op_ret = -1; > - op_errno = errno; > gf_log (this->name, GF_LOG_WARNING, > "open of %s failed: %s", loc->path, strerror > (op_errno)); > - goto out; > } > > op_ret = unlink (real_path);
It'd be better to simply not do the open() on any file other than regular files. The fix is on its way to being committed. Thanks for spotting this, and sending the patch :) Vikas -- Engineer - Z Research http://gluster.com/ _______________________________________________ Gluster-users mailing list [email protected] http://zresearch.com/cgi-bin/mailman/listinfo/gluster-users
