Re: [PATCH] resolve_gitlink_ref_recursive(): verify format of symbolic refs

2014-06-27 Thread Michael Haggerty
On 06/27/2014 07:59 PM, Junio C Hamano wrote: > Junio C Hamano writes: > >> Michael Haggerty writes: >> >>> When reading a symbolic ref via resolve_gitlink_ref_recursive(), check >>> that the reference name that is pointed at is formatted correctly, >>> using the same check as resolve_ref_unsafe

Re: [PATCH] resolve_gitlink_ref_recursive(): verify format of symbolic refs

2014-06-27 Thread Junio C Hamano
Junio C Hamano writes: > Michael Haggerty writes: > >> When reading a symbolic ref via resolve_gitlink_ref_recursive(), check >> that the reference name that is pointed at is formatted correctly, >> using the same check as resolve_ref_unsafe() uses for non-gitlink >> references. This prevents b

Re: [PATCH] resolve_gitlink_ref_recursive(): verify format of symbolic refs

2014-06-27 Thread Junio C Hamano
Michael Haggerty writes: > When reading a symbolic ref via resolve_gitlink_ref_recursive(), check > that the reference name that is pointed at is formatted correctly, > using the same check as resolve_ref_unsafe() uses for non-gitlink > references. This prevents bogosity like > > ref: ../../

[PATCH] resolve_gitlink_ref_recursive(): verify format of symbolic refs

2014-06-27 Thread Michael Haggerty
When reading a symbolic ref via resolve_gitlink_ref_recursive(), check that the reference name that is pointed at is formatted correctly, using the same check as resolve_ref_unsafe() uses for non-gitlink references. This prevents bogosity like ref: ../../other/file from causing problems. Si