On 07/12/2011 09:01 AM, Masatake YAMATO wrote: > Very sorry for posting again and again. > fd shuold be closed even if read is failed. > > Signed-off-by: Masatake YAMATO <yamato@xxxxxxxxxx> > > diff --git a/support/nfs/exports.c b/support/nfs/exports.c > index c250383..c96500f 100644 > --- a/support/nfs/exports.c > +++ b/support/nfs/exports.c > @@ -784,8 +784,9 @@ struct export_features *get_export_features(void) > fd = open(path, O_RDONLY); > if (fd == -1) > goto good; > - fd = read(fd, buf, 50); > - if (fd == -1) > + c = read(fd, buf, 50); > + close(fd); > + if (c == -1) > goto err; > c = sscanf(buf, "%x %x", &ef.flags, &ef.secinfo_flags); > if (c != 2) > Committed... steved. -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html