On 04/09/2017 12:12 PM, ChunYu Wang wrote: > Signed-off-by: ChunYu Wang <chunwang@xxxxxxxxxx> > --- > Fix a potential memory leak issue caused by unfree pathname 'path'. > > systemd/nfs-server-generator.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/systemd/nfs-server-generator.c b/systemd/nfs-server-generator.c > index 4aa6509..441cec5 100644 > --- a/systemd/nfs-server-generator.c > +++ b/systemd/nfs-server-generator.c > @@ -139,6 +139,7 @@ int main(int argc, char *argv[]) > strcat(path, filebase); > f = fopen(path, "w"); > if (!f) > + free(path); > exit(1); First of all you need some '{' '}' so the process will not always exit when f != NULL... Secondly dead process don't leak memory. :-) They don't leak anything... they are died! ;-) steved. > fprintf(f, "# Automatically generated by nfs-server-generator\n\n[Unit]\n"); > > -- 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