Re: Bug: segfault during git-prune
- From
- Andy Parkins <andyparkins@gmail.com>
- Date
- Jul 2, 2007, 13:25 UTC
- Message-ID
- <200707021425.38367.andyparkins@gmail.com>
- In-Reply-To
- <alpine.LFD.0.98.0707020441460.3323@woody.linux-foundation.org>
On Monday 2007 July 02, Linus Torvalds wrote:
> Just for clarity, I'd use "process_gitlink()" here (and it would probably > be a good idea to use process_blob() on the blob we look up too), and the > proper tab indentation, but yes, I think you're otherwise exactly right.
I've pushed it out as-is as a bug fix patch now; and will happilly make the above change on top. However, I've not understood what all the parameters are for in the process_XXXX() functions:
static void process_gitlink(const unsigned char *sha1, struct object_array *p, struct name_path *path, const char *name)
the sha1 and name I can read out of the cache_entry structure, but I'm not sure about the object_array and name_path structures in the context of add_cache_refs().
Andy
-- Dr Andy Parkins, M Eng (hons), MIET andyparkins@gmail.com