]> git.ipfire.org Git - thirdparty/git.git/commit - sha1_file.c
safe_create_leading_directories(): split on first of multiple slashes
authorMichael Haggerty <mhagger@alum.mit.edu>
Mon, 6 Jan 2014 13:45:23 +0000 (14:45 +0100)
committerJunio C Hamano <gitster@pobox.com>
Mon, 6 Jan 2014 17:34:20 +0000 (09:34 -0800)
commitbf10cf70ad0c777dbbbb00bbb741436e285c2181
tree25bfc27da6b48b62026facef9f872a9105159238
parent26c8ae2a577cd283ab9fa8ab5e5f0ced568dc034
safe_create_leading_directories(): split on first of multiple slashes

If the input path has multiple slashes between path components (e.g.,
"foo//bar"), then the old code was breaking the path at the last
slash, not the first one.  So in the above example, the second slash
was overwritten with NUL, resulting in the parent directory being
sought as "foo/".

When stat() is called on "foo/", it fails with ENOTDIR if "foo" exists
but is not a directory.  This caused the wrong path to be taken in the
subsequent logic.

So instead, split path components at the first intercomponent slash
rather than the last one.

Signed-off-by: Michael Haggerty <mhagger@alum.mit.edu>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
sha1_file.c