Hi Dan, On 07/18/2016 11:55 PM, Daniel Gryniewicz wrote: > Hi, Soumya > > Coverity caught an issue (150368) with glusterfs_setttr2() that looks > legitimate. The problem is that the error cases all set status, and > then goto out. However, the first thing done after the out label is to > overwrite status from retval. It looks like either everything should be > using status, or everything should be using retval. This may mask error > returns in this case, I believe. >
Right. Thanks for reporting. Have submitted the fix for review- https://review.gerrithub.io/#/c/284432/ Thanks, Soumya > Thanks, > Daniel ------------------------------------------------------------------------------ What NetFlow Analyzer can do for you? Monitors network bandwidth and traffic patterns at an interface-level. Reveals which users, apps, and protocols are consuming the most bandwidth. Provides multi-vendor support for NetFlow, J-Flow, sFlow and other flows. Make informed decisions using capacity planning reports.http://sdm.link/zohodev2dev _______________________________________________ Nfs-ganesha-devel mailing list Nfs-ganesha-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel
