diff options
| author | Junio C Hamano <junkio@cox.net> | 2005-09-21 07:45:25 (GMT) |
|---|---|---|
| committer | Junio C Hamano <junkio@cox.net> | 2005-09-21 07:45:25 (GMT) |
| commit | 9f25c7bd5cf2bd6ae8e4775a4fd1a19f3eda10ea (patch) | |
| tree | be958626d1c956961e714b209a5d2894f5a1be2f | |
| parent | dc4393cba333c92873671a37e72efaf41bfa4697 (diff) | |
| parent | af61c6e008c39ee9f61af33bb48850230f503443 (diff) | |
| download | git-9f25c7bd5cf2bd6ae8e4775a4fd1a19f3eda10ea.zip git-9f25c7bd5cf2bd6ae8e4775a4fd1a19f3eda10ea.tar.gz git-9f25c7bd5cf2bd6ae8e4775a4fd1a19f3eda10ea.tar.bz2 | |
Merge branch 'fixes'
| -rw-r--r-- | sha1_name.c | 12 |
1 files changed, 5 insertions, 7 deletions
diff --git a/sha1_name.c b/sha1_name.c index 133af3b..b4fed92 100644 --- a/sha1_name.c +++ b/sha1_name.c @@ -84,19 +84,19 @@ static int find_short_packed_object(int len, const unsigned char *match, unsigne return 0; } -static int get_short_sha1(const char *name, unsigned char *sha1) +static int get_short_sha1(const char *name, int len, unsigned char *sha1) { int i; char canonical[40]; unsigned char res[20]; + if (len < 4) + return -1; memset(res, 0, 20); memset(canonical, 'x', 40); - for (i = 0;;i++) { + for (i = 0; i < len ;i++) { unsigned char c = name[i]; unsigned char val; - if (!c || i > 40) - break; if (c >= '0' && c <= '9') val = c - '0'; else if (c >= 'a' && c <= 'f') @@ -112,8 +112,6 @@ static int get_short_sha1(const char *name, unsigned char *sha1) val <<= 4; res[i >> 1] |= val; } - if (i < 4) - return -1; if (find_short_object_filename(i, canonical, sha1)) return 0; if (find_short_packed_object(i, res, sha1)) @@ -254,7 +252,7 @@ static int get_sha1_1(const char *name, int len, unsigned char *sha1) ret = get_sha1_basic(name, len, sha1); if (!ret) return 0; - return get_short_sha1(name, sha1); + return get_short_sha1(name, len, sha1); } /* |