diff options
| author | Jonathan Nieder <jrnieder@gmail.com> | 2010-05-26 21:50:34 (GMT) |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2010-05-28 22:08:27 (GMT) |
| commit | 0adc6a3d49a46436780b2dd636918c9840d82236 (patch) | |
| tree | d4e8e57d71509d77bd624348b5b16153c35c29f4 /fsck.c | |
| parent | daae19224a05be9efb9a39c2a2c1c9a60fe906f1 (diff) | |
| download | git-0adc6a3d49a46436780b2dd636918c9840d82236.zip git-0adc6a3d49a46436780b2dd636918c9840d82236.tar.gz git-0adc6a3d49a46436780b2dd636918c9840d82236.tar.bz2 | |
fsck: fix bogus commit header check
daae1922 (fsck: check ident lines in commit objects, 2010-04-24)
taught fsck to expect commit objects to have the form
tree <object name>
<parents>
author <valid ident string>
committer <valid ident string>
log message
The check is overly strict: for example, it errors out with the
message “expected blank line” for perfectly valid commits with an
"encoding ISO-8859-1" line.
Later it might make sense to teach fsck about the rest of the header
and warn about unrecognized header lines, but for simplicity, let’s
accept arbitrary trailing lines for now.
Reported-by: Tuncer Ayaz <tuncer.ayaz@gmail.com>
Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'fsck.c')
| -rw-r--r-- | fsck.c | 2 |
1 files changed, 0 insertions, 2 deletions
@@ -311,8 +311,6 @@ static int fsck_commit(struct commit *commit, fsck_error error_func) err = fsck_ident(&buffer, &commit->object, error_func); if (err) return err; - if (*buffer != '\n') - return error_func(&commit->object, FSCK_ERROR, "invalid format - expected blank line"); if (!commit->tree) return error_func(&commit->object, FSCK_ERROR, "could not load commit's tree %s", sha1_to_hex(tree_sha1)); |