X-Git-Url: https://git.octo.it/?a=blobdiff_plain;f=sha1_name.c;h=faac158b16ca978914696ed5d2801770a122cce2;hb=ed24928e122bad83e62d161087f806fa21c46a59;hp=be1755a70b2c53fc102db41a62e6099fa0adac1b;hpb=8fc66df237afce0b4318657f166b3583831949f3;p=git.git diff --git a/sha1_name.c b/sha1_name.c index be1755a7..faac158b 100644 --- a/sha1_name.c +++ b/sha1_name.c @@ -236,6 +236,7 @@ static int get_sha1_basic(const char *str, int len, unsigned char *sha1) NULL }; const char **p; + int found = 0; if (len == 40 && !get_sha1_hex(str, sha1)) return 0; @@ -246,10 +247,20 @@ static int get_sha1_basic(const char *str, int len, unsigned char *sha1) for (p = prefix; *p; p++) { char *pathname = git_path("%s/%.*s", *p, len, str); - if (!read_ref(pathname, sha1)) - return 0; + if (!read_ref(pathname, sha1)) { + /* Must be unique; i.e. when heads/foo and + * tags/foo are both present, reject "foo". + * Note that read_ref() eventually calls + * get_sha1_hex() which can smudge initial + * part of the buffer even if what is read + * is found to be invalid halfway. + */ + if (1 < found++) + return -1; + } } - + if (found == 1) + return 0; return -1; }