[PATCH] nfs-server-generator: fix a potential memory leak issue

[PATCH] nfs-server-generator: fix a potential memory leak issue

[ChunYu Wang]

Signed-off-by: ChunYu Wang <chunwang@redhat.com>
---
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);
 	fprintf(f, "# Automatically generated by nfs-server-generator\n\n[Unit]\n");
 
-- 
2.7.4

Re: [PATCH] nfs-server-generator: fix a potential memory leak issue

[Steve Dickson]

On 04/09/2017 12:12 PM, ChunYu Wang wrote:
> Signed-off-by: ChunYu Wang <chunwang@redhat.com>
> ---
> 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");
>  
> 

Re: [PATCH] nfs-server-generator: fix a potential memory leak issue

[ChunYu Wang]

En, yes,

Thanks for reviewing and reminding, it is my fault [ : ( ].

Valgrind did not notice me about the syntax rationality, but it's my duty.
Get down to learn, and thanks a lot.

Regards,
ChunYu Wang

On Mon, Apr 10, 2017 at 2:33 AM, Steve Dickson <SteveD@redhat.com> wrote:
>
>
> On 04/09/2017 12:12 PM, ChunYu Wang wrote:
>> Signed-off-by: ChunYu Wang <chunwang@redhat.com>
>> ---
>> 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");
>>
>>