RE: [PATCH v4 04/13] exfat: add directory operations

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> 
> …
> > +++ b/fs/exfat/dir.c
> …
> > +static int exfat_readdir(struct inode *inode, struct exfat_dir_entry
> *dir_entry)
> > +{
> …
> > +			if (!ep) {
> > +				ret = -EIO;
> > +				goto free_clu;
> > +			}
> 
> How do you think about to move a bit of common exception handling code
> (at similar places)?
Not sure it is good. Other review comments are okay. Will fix them on v5.
> 
> +			if (!ep)
> +				goto e_io;
> 
> 
> …
> > +free_clu:
> > +	kfree(clu);
> > +	return ret;
> 
> +
> +e_io:
> +	ret = -EIO;
> +	goto free_clu;
> 
> > +}






[Index of Archives]     [Kernel Development]     [Kernel Announce]     [Kernel Newbies]     [Linux Networking Development]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Device Mapper]

  Powered by Linux