On 10/23/2013 05:18 PM, NeilBrown wrote: > On Wed, 23 Oct 2013 12:36:23 -0500 Tony Asleson <tasleson@xxxxxxxxxx> wrote: >> For example, if the strdup call fails in function client_init, we fail >> the operation and return exit value of 0. Unlikely, but just the first >> example I stumbled across. > > I think it is a lot closer to "impossible" than just "unlikely". > malloc doesn't fail in this sort of context, the OOM killer kills something > off instead. > My personal preference is to replace all malloc/calloc/strdup calls with > the xmalloc, xstrdup etc calls in support/nfs/xcommon.c. > If you are worried about malloc failing, I'd much prefer to see a patch which > changes nfs-utils to use those uniformly. Sorry, my real point was that there are other ways for exportfs to fail to do the operation and fail to report it, thus the reason my patch was more invasive. I will try to find another more likely example that illustrates this. Regards, Tony -- 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
