Re: ldd.c - plug memleak

2013-07-14 Thread Jérémie Courrèges-Anglas
David Hill dh...@mindcry.org writes: Hello - Hi, doit() was not free()'ing memory or close()'ing the file descriptor if realpath() failed or dlopen() returned NULL. This diff just moves close() and free() up once we are done using them. Makes sense. ok? Index: ldd/ldd.c

Re: ldd.c - plug memleak

2013-07-14 Thread Kenneth R Westerback
On Sun, Jul 14, 2013 at 09:23:05AM +0200, J??r??mie Courr??ges-Anglas wrote: David Hill dh...@mindcry.org writes: Hello - Hi, doit() was not free()'ing memory or close()'ing the file descriptor if realpath() failed or dlopen() returned NULL. This diff just moves close() and

ldd.c - plug memleak

2013-07-05 Thread David Hill
Hello - doit() was not free()'ing memory or close()'ing the file descriptor if realpath() failed or dlopen() returned NULL. This diff just moves close() and free() up once we are done using them. Index: ldd/ldd.c === RCS file: