tag 18145 notabug
close 18145
stop
On 07/29/2014 08:02 AM, srinivas devaki wrote:
> i think there is a memory leak in pwd.c
> I’m extremely sorry if this is wrong. I’m sending the git diff.
>
> [eightnoteight@mr coreutils]$ git diff
> diff --git a/src/pwd.c b/src/pwd.c
> index 4993dfb..13d84cd 100644
> --- a/src/pwd.c
> +++ b/src/pwd.c
> @@ -367,6 +367,7 @@ main (int argc, char **argv)
> if (wd)
> {
> puts (wd);
> + free (wd);
> exit (EXIT_SUCCESS);
> }
> }
>
> and how can i contribute to gnu coreutils. how can i understand the code of
> utils. which libraries should i get used to, before contributing to the gnu
> coreutils.
valgrind doesn't concur:
$ valgrind src/pwd -L
==30954== All heap blocks were freed -- no leaks are possible
You can see that logical_getcwd() doesn't allocate,
so no free is necessary.
On the general point of coreutils dev there is:
http://git.sv.gnu.org/gitweb/?p=coreutils.git;a=blob;f=README-hacking;hb=HEAD
thanks,
Pádraig.
