Skip to content
  • Yasushi SHOJI's avatar
    [PATCH] plug memory leak in diff.c::diff_free_filepair() · 068eac91
    Yasushi SHOJI authored
    
    
    When I run git-diff-tree on big change, it seems the command eats so
    much memory.  so I just put git under valgrind to see what's going on.
    diff_free_filespec_data() doesn't free diff_filespec itself.
    
    [jc: I ended up doing things slightly differently from Yasushi's
    patch.  The original idea was to use free_filespec_data() only to
    free the data portion and keep useing the filespec itself, but
    no existing code seems to do things that way, so I just yanked
    that part out.]
    
    Signed-off-by: default avatarYasushi SHOJI <yashi@atmark-techno.com>
    Signed-off-by: default avatarJunio C Hamano <junkio@cox.net>
    068eac91