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

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

 



Hi Hatayama-san,

-----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.

This series looks good to me.

Acked-by: Kazuhito Hagio <k-hagio-ab@xxxxxxx>

I hope this helps us develop/debug crash and extension modules.

Thanks,
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 ++++
>  configure.c | 15 ++++++++++++-
>  symbols.c   | 10 +++------
>  tools.c     | 72 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++--
>  4 files changed, 91 insertions(+), 10 deletions(-)
> 
> --
> 1.8.3.1
> 
> --
> Crash-utility mailing list
> Crash-utility@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/crash-utility


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




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

 

Powered by Linux