Re: [PATCH v2 0/3] Add valgrind support for the crash's custom memory

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



-----Original Message-----
> This patch set adds valgrind support for the crash's custom memory
> allocator. The motivation comes from investigation at
> https://github.com/crash-utility/crash-extensions/issues/1.
> 
> The 1st patch implements the valgrind support, while 2nd and 3rd fixes
> the actual issues on the crash's custom memory allocator detected by
> valgrind.
> 
> v2:
> - Unlink tools.o when make valgrind
> - Add description about make valgrind in README

Thanks for the update, the v2 patchset looks good.

Applied with fixing the typos Lianbo pointed and a few adjustments:
https://github.com/crash-utility/crash/compare/720279f...9c0c6c1

Kazu

> 
> HATAYAMA Daisuke (3):
>   Add valgrind support for the crash's custom memory allocator
>   symbols: Fix potential read to already freed object
>   tools: Fix potential write to object of 0 size
> 
>  Makefile    |  4 ++++
>  README      |  5 +++++
>  configure.c | 21 +++++++++++++++++-
>  help.c      |  5 +++++
>  symbols.c   | 10 +++------
>  tools.c     | 72 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++--
>  6 files changed, 107 insertions(+), 10 deletions(-)
> 
> --
> 1.8.3.1
> 
> --
> Crash-utility mailing list
> Crash-utility@xxxxxxxxxx
> https://listman.redhat.com/mailman/listinfo/crash-utility


--
Crash-utility mailing list
Crash-utility@xxxxxxxxxx
https://listman.redhat.com/mailman/listinfo/crash-utility




[Index of Archives]     [Fedora Development]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]

 

Powered by Linux