Patrick Steinhardt | e7da938 | 2024-06-14 08:50:23 +0200 | [diff] [blame] | 1 | #define USE_THE_REPOSITORY_VARIABLE |
| 2 | |
Elijah Newren | d812c3b | 2023-04-11 00:41:56 -0700 | [diff] [blame] | 3 | #include "git-compat-util.h" |
Elijah Newren | d4a4f92 | 2023-04-22 20:17:26 +0000 | [diff] [blame] | 4 | #include "date.h" |
Elijah Newren | a034e91 | 2023-05-16 06:34:06 +0000 | [diff] [blame] | 5 | #include "dir.h" |
Elijah Newren | 41771fa | 2023-02-24 00:09:27 +0000 | [diff] [blame] | 6 | #include "hex.h" |
Elijah Newren | a034e91 | 2023-05-16 06:34:06 +0000 | [diff] [blame] | 7 | #include "object-store-ll.h" |
Elijah Newren | c339932 | 2023-05-16 06:33:59 +0000 | [diff] [blame] | 8 | #include "path.h" |
Stefan Beller | 109cd76 | 2018-06-28 18:21:51 -0700 | [diff] [blame] | 9 | #include "repository.h" |
Martin Koegler | 355885d | 2008-02-25 22:46:04 +0100 | [diff] [blame] | 10 | #include "object.h" |
Patrick Steinhardt | 27ab478 | 2022-12-01 15:46:09 +0100 | [diff] [blame] | 11 | #include "attr.h" |
Martin Koegler | 355885d | 2008-02-25 22:46:04 +0100 | [diff] [blame] | 12 | #include "blob.h" |
| 13 | #include "tree.h" |
| 14 | #include "tree-walk.h" |
| 15 | #include "commit.h" |
| 16 | #include "tag.h" |
| 17 | #include "fsck.h" |
Johannes Schindelin | cec097b | 2014-09-11 16:26:38 +0200 | [diff] [blame] | 18 | #include "refs.h" |
Jonathan Nieder | a2b26ff | 2020-04-18 20:52:34 -0700 | [diff] [blame] | 19 | #include "url.h" |
Jeff King | a18fcc9 | 2014-12-15 18:21:57 -0500 | [diff] [blame] | 20 | #include "utf8.h" |
Jeff King | 159e7b0 | 2018-05-02 17:20:08 -0400 | [diff] [blame] | 21 | #include "oidset.h" |
Jeff King | 2738744 | 2018-05-14 12:22:48 -0400 | [diff] [blame] | 22 | #include "packfile.h" |
Jeff King | ed8b10f | 2018-05-02 17:25:27 -0400 | [diff] [blame] | 23 | #include "submodule-config.h" |
| 24 | #include "config.h" |
Nguyễn Thái Ngọc Duy | 3ac68a9 | 2018-05-26 15:55:24 +0200 | [diff] [blame] | 25 | #include "help.h" |
Jeff King | 159e7b0 | 2018-05-02 17:20:08 -0400 | [diff] [blame] | 26 | |
Jeff King | 0fbcaef | 2023-08-31 02:20:01 -0400 | [diff] [blame] | 27 | static ssize_t max_tree_entry_len = 4096; |
| 28 | |
Johannes Schindelin | f417eed | 2015-06-22 17:25:14 +0200 | [diff] [blame] | 29 | #define STR(x) #x |
Nguyễn Thái Ngọc Duy | a4a9cc1 | 2018-05-26 15:55:25 +0200 | [diff] [blame] | 30 | #define MSG_ID(id, msg_type) { STR(id), NULL, NULL, FSCK_##msg_type }, |
Johannes Schindelin | c99ba49 | 2015-06-22 17:25:09 +0200 | [diff] [blame] | 31 | static struct { |
Johannes Schindelin | f417eed | 2015-06-22 17:25:14 +0200 | [diff] [blame] | 32 | const char *id_string; |
| 33 | const char *downcased; |
Nguyễn Thái Ngọc Duy | a4a9cc1 | 2018-05-26 15:55:25 +0200 | [diff] [blame] | 34 | const char *camelcased; |
Ævar Arnfjörð Bjarmason | 1b32b59 | 2021-03-28 15:15:40 +0200 | [diff] [blame] | 35 | enum fsck_msg_type msg_type; |
Johannes Schindelin | c99ba49 | 2015-06-22 17:25:09 +0200 | [diff] [blame] | 36 | } msg_id_info[FSCK_MSG_MAX + 1] = { |
Ævar Arnfjörð Bjarmason | 901f2f6 | 2021-03-28 15:15:44 +0200 | [diff] [blame] | 37 | FOREACH_FSCK_MSG_ID(MSG_ID) |
Nguyễn Thái Ngọc Duy | a4a9cc1 | 2018-05-26 15:55:25 +0200 | [diff] [blame] | 38 | { NULL, NULL, NULL, -1 } |
Johannes Schindelin | c99ba49 | 2015-06-22 17:25:09 +0200 | [diff] [blame] | 39 | }; |
| 40 | #undef MSG_ID |
Ævar Arnfjörð Bjarmason | b549502 | 2021-03-28 15:15:43 +0200 | [diff] [blame] | 41 | #undef STR |
Johannes Schindelin | c99ba49 | 2015-06-22 17:25:09 +0200 | [diff] [blame] | 42 | |
Nguyễn Thái Ngọc Duy | a46baac | 2018-05-26 15:55:23 +0200 | [diff] [blame] | 43 | static void prepare_msg_ids(void) |
| 44 | { |
| 45 | int i; |
| 46 | |
| 47 | if (msg_id_info[0].downcased) |
| 48 | return; |
| 49 | |
| 50 | /* convert id_string to lower case, without underscores. */ |
| 51 | for (i = 0; i < FSCK_MSG_MAX; i++) { |
| 52 | const char *p = msg_id_info[i].id_string; |
| 53 | int len = strlen(p); |
| 54 | char *q = xmalloc(len); |
| 55 | |
| 56 | msg_id_info[i].downcased = q; |
| 57 | while (*p) |
| 58 | if (*p == '_') |
| 59 | p++; |
| 60 | else |
| 61 | *(q)++ = tolower(*(p)++); |
| 62 | *q = '\0'; |
Nguyễn Thái Ngọc Duy | a4a9cc1 | 2018-05-26 15:55:25 +0200 | [diff] [blame] | 63 | |
| 64 | p = msg_id_info[i].id_string; |
| 65 | q = xmalloc(len); |
| 66 | msg_id_info[i].camelcased = q; |
| 67 | while (*p) { |
| 68 | if (*p == '_') { |
| 69 | p++; |
| 70 | if (*p) |
| 71 | *q++ = *p++; |
| 72 | } else { |
| 73 | *q++ = tolower(*p++); |
| 74 | } |
| 75 | } |
| 76 | *q = '\0'; |
Nguyễn Thái Ngọc Duy | a46baac | 2018-05-26 15:55:23 +0200 | [diff] [blame] | 77 | } |
| 78 | } |
| 79 | |
Johannes Schindelin | f417eed | 2015-06-22 17:25:14 +0200 | [diff] [blame] | 80 | static int parse_msg_id(const char *text) |
| 81 | { |
| 82 | int i; |
| 83 | |
Nguyễn Thái Ngọc Duy | a46baac | 2018-05-26 15:55:23 +0200 | [diff] [blame] | 84 | prepare_msg_ids(); |
Johannes Schindelin | f417eed | 2015-06-22 17:25:14 +0200 | [diff] [blame] | 85 | |
| 86 | for (i = 0; i < FSCK_MSG_MAX; i++) |
| 87 | if (!strcmp(text, msg_id_info[i].downcased)) |
| 88 | return i; |
| 89 | |
| 90 | return -1; |
| 91 | } |
| 92 | |
Nguyễn Thái Ngọc Duy | 3ac68a9 | 2018-05-26 15:55:24 +0200 | [diff] [blame] | 93 | void list_config_fsck_msg_ids(struct string_list *list, const char *prefix) |
| 94 | { |
| 95 | int i; |
| 96 | |
| 97 | prepare_msg_ids(); |
| 98 | |
Nguyễn Thái Ngọc Duy | 3ac68a9 | 2018-05-26 15:55:24 +0200 | [diff] [blame] | 99 | for (i = 0; i < FSCK_MSG_MAX; i++) |
Nguyễn Thái Ngọc Duy | a4a9cc1 | 2018-05-26 15:55:25 +0200 | [diff] [blame] | 100 | list_config_item(list, prefix, msg_id_info[i].camelcased); |
Nguyễn Thái Ngọc Duy | 3ac68a9 | 2018-05-26 15:55:24 +0200 | [diff] [blame] | 101 | } |
| 102 | |
Ævar Arnfjörð Bjarmason | 1b32b59 | 2021-03-28 15:15:40 +0200 | [diff] [blame] | 103 | static enum fsck_msg_type fsck_msg_type(enum fsck_msg_id msg_id, |
Johannes Schindelin | c99ba49 | 2015-06-22 17:25:09 +0200 | [diff] [blame] | 104 | struct fsck_options *options) |
| 105 | { |
Johannes Schindelin | 0282f4d | 2015-06-22 17:25:25 +0200 | [diff] [blame] | 106 | assert(msg_id >= 0 && msg_id < FSCK_MSG_MAX); |
| 107 | |
Ævar Arnfjörð Bjarmason | e35d65a | 2021-03-28 15:15:39 +0200 | [diff] [blame] | 108 | if (!options->msg_type) { |
Ævar Arnfjörð Bjarmason | 1b32b59 | 2021-03-28 15:15:40 +0200 | [diff] [blame] | 109 | enum fsck_msg_type msg_type = msg_id_info[msg_id].msg_type; |
Ævar Arnfjörð Bjarmason | e35d65a | 2021-03-28 15:15:39 +0200 | [diff] [blame] | 110 | |
Johannes Schindelin | 0282f4d | 2015-06-22 17:25:25 +0200 | [diff] [blame] | 111 | if (options->strict && msg_type == FSCK_WARN) |
| 112 | msg_type = FSCK_ERROR; |
Ævar Arnfjörð Bjarmason | e35d65a | 2021-03-28 15:15:39 +0200 | [diff] [blame] | 113 | return msg_type; |
Johannes Schindelin | 0282f4d | 2015-06-22 17:25:25 +0200 | [diff] [blame] | 114 | } |
Johannes Schindelin | c99ba49 | 2015-06-22 17:25:09 +0200 | [diff] [blame] | 115 | |
Ævar Arnfjörð Bjarmason | e35d65a | 2021-03-28 15:15:39 +0200 | [diff] [blame] | 116 | return options->msg_type[msg_id]; |
Johannes Schindelin | c99ba49 | 2015-06-22 17:25:09 +0200 | [diff] [blame] | 117 | } |
| 118 | |
Ævar Arnfjörð Bjarmason | 1b32b59 | 2021-03-28 15:15:40 +0200 | [diff] [blame] | 119 | static enum fsck_msg_type parse_msg_type(const char *str) |
Johannes Schindelin | 0282f4d | 2015-06-22 17:25:25 +0200 | [diff] [blame] | 120 | { |
| 121 | if (!strcmp(str, "error")) |
| 122 | return FSCK_ERROR; |
| 123 | else if (!strcmp(str, "warn")) |
| 124 | return FSCK_WARN; |
Johannes Schindelin | efaba7c | 2015-06-22 17:26:48 +0200 | [diff] [blame] | 125 | else if (!strcmp(str, "ignore")) |
| 126 | return FSCK_IGNORE; |
Johannes Schindelin | 0282f4d | 2015-06-22 17:25:25 +0200 | [diff] [blame] | 127 | else |
| 128 | die("Unknown fsck message type: '%s'", str); |
| 129 | } |
| 130 | |
Johannes Schindelin | 5d477a3 | 2015-06-22 17:25:31 +0200 | [diff] [blame] | 131 | int is_valid_msg_type(const char *msg_id, const char *msg_type) |
| 132 | { |
| 133 | if (parse_msg_id(msg_id) < 0) |
| 134 | return 0; |
| 135 | parse_msg_type(msg_type); |
| 136 | return 1; |
| 137 | } |
| 138 | |
Ævar Arnfjörð Bjarmason | 53692df | 2021-03-28 15:15:47 +0200 | [diff] [blame] | 139 | void fsck_set_msg_type_from_ids(struct fsck_options *options, |
| 140 | enum fsck_msg_id msg_id, |
| 141 | enum fsck_msg_type msg_type) |
| 142 | { |
| 143 | if (!options->msg_type) { |
| 144 | int i; |
| 145 | enum fsck_msg_type *severity; |
| 146 | ALLOC_ARRAY(severity, FSCK_MSG_MAX); |
| 147 | for (i = 0; i < FSCK_MSG_MAX; i++) |
| 148 | severity[i] = fsck_msg_type(i, options); |
| 149 | options->msg_type = severity; |
| 150 | } |
| 151 | |
| 152 | options->msg_type[msg_id] = msg_type; |
| 153 | } |
| 154 | |
Johannes Schindelin | 0282f4d | 2015-06-22 17:25:25 +0200 | [diff] [blame] | 155 | void fsck_set_msg_type(struct fsck_options *options, |
Ævar Arnfjörð Bjarmason | f1abc2d | 2021-03-28 15:15:36 +0200 | [diff] [blame] | 156 | const char *msg_id_str, const char *msg_type_str) |
Johannes Schindelin | 0282f4d | 2015-06-22 17:25:25 +0200 | [diff] [blame] | 157 | { |
Ævar Arnfjörð Bjarmason | 1b32b59 | 2021-03-28 15:15:40 +0200 | [diff] [blame] | 158 | int msg_id = parse_msg_id(msg_id_str); |
Jeff King | 0fbcaef | 2023-08-31 02:20:01 -0400 | [diff] [blame] | 159 | char *to_free = NULL; |
| 160 | enum fsck_msg_type msg_type; |
Johannes Schindelin | 0282f4d | 2015-06-22 17:25:25 +0200 | [diff] [blame] | 161 | |
Ævar Arnfjörð Bjarmason | f1abc2d | 2021-03-28 15:15:36 +0200 | [diff] [blame] | 162 | if (msg_id < 0) |
| 163 | die("Unhandled message id: %s", msg_id_str); |
Johannes Schindelin | 0282f4d | 2015-06-22 17:25:25 +0200 | [diff] [blame] | 164 | |
Jeff King | 0fbcaef | 2023-08-31 02:20:01 -0400 | [diff] [blame] | 165 | if (msg_id == FSCK_MSG_LARGE_PATHNAME) { |
| 166 | const char *colon = strchr(msg_type_str, ':'); |
| 167 | if (colon) { |
| 168 | msg_type_str = to_free = |
| 169 | xmemdupz(msg_type_str, colon - msg_type_str); |
| 170 | colon++; |
| 171 | if (!git_parse_ssize_t(colon, &max_tree_entry_len)) |
| 172 | die("unable to parse max tree entry len: %s", colon); |
| 173 | } |
| 174 | } |
| 175 | msg_type = parse_msg_type(msg_type_str); |
| 176 | |
Ævar Arnfjörð Bjarmason | f1abc2d | 2021-03-28 15:15:36 +0200 | [diff] [blame] | 177 | if (msg_type != FSCK_ERROR && msg_id_info[msg_id].msg_type == FSCK_FATAL) |
| 178 | die("Cannot demote %s to %s", msg_id_str, msg_type_str); |
Johannes Schindelin | f50c440 | 2015-06-22 17:26:42 +0200 | [diff] [blame] | 179 | |
Ævar Arnfjörð Bjarmason | 53692df | 2021-03-28 15:15:47 +0200 | [diff] [blame] | 180 | fsck_set_msg_type_from_ids(options, msg_id, msg_type); |
Jeff King | 0fbcaef | 2023-08-31 02:20:01 -0400 | [diff] [blame] | 181 | free(to_free); |
Johannes Schindelin | 0282f4d | 2015-06-22 17:25:25 +0200 | [diff] [blame] | 182 | } |
| 183 | |
| 184 | void fsck_set_msg_types(struct fsck_options *options, const char *values) |
| 185 | { |
| 186 | char *buf = xstrdup(values), *to_free = buf; |
| 187 | int done = 0; |
| 188 | |
| 189 | while (!done) { |
| 190 | int len = strcspn(buf, " ,|"), equal; |
| 191 | |
| 192 | done = !buf[len]; |
| 193 | if (!len) { |
| 194 | buf++; |
| 195 | continue; |
| 196 | } |
| 197 | buf[len] = '\0'; |
| 198 | |
| 199 | for (equal = 0; |
| 200 | equal < len && buf[equal] != '=' && buf[equal] != ':'; |
| 201 | equal++) |
| 202 | buf[equal] = tolower(buf[equal]); |
| 203 | buf[equal] = '\0'; |
| 204 | |
Johannes Schindelin | cd94c6f | 2015-06-22 17:27:18 +0200 | [diff] [blame] | 205 | if (!strcmp(buf, "skiplist")) { |
| 206 | if (equal == len) |
| 207 | die("skiplist requires a path"); |
Patrick Steinhardt | f2c32a6 | 2024-06-14 08:50:42 +0200 | [diff] [blame] | 208 | oidset_parse_file(&options->skiplist, buf + equal + 1, |
| 209 | the_repository->hash_algo); |
Johannes Schindelin | cd94c6f | 2015-06-22 17:27:18 +0200 | [diff] [blame] | 210 | buf += len + 1; |
| 211 | continue; |
| 212 | } |
| 213 | |
Johannes Schindelin | 0282f4d | 2015-06-22 17:25:25 +0200 | [diff] [blame] | 214 | if (equal == len) |
| 215 | die("Missing '=': '%s'", buf); |
| 216 | |
| 217 | fsck_set_msg_type(options, buf, buf + equal + 1); |
| 218 | buf += len + 1; |
| 219 | } |
| 220 | free(to_free); |
| 221 | } |
| 222 | |
Jeff King | f597937 | 2019-10-18 00:58:51 -0400 | [diff] [blame] | 223 | static int object_on_skiplist(struct fsck_options *opts, |
| 224 | const struct object_id *oid) |
Ramsay Jones | fb16287 | 2018-06-27 19:39:53 +0100 | [diff] [blame] | 225 | { |
Jeff King | f597937 | 2019-10-18 00:58:51 -0400 | [diff] [blame] | 226 | return opts && oid && oidset_contains(&opts->skiplist, oid); |
Ramsay Jones | fb16287 | 2018-06-27 19:39:53 +0100 | [diff] [blame] | 227 | } |
| 228 | |
Jeff King | 3837025 | 2019-10-18 00:59:15 -0400 | [diff] [blame] | 229 | __attribute__((format (printf, 5, 6))) |
| 230 | static int report(struct fsck_options *options, |
| 231 | const struct object_id *oid, enum object_type object_type, |
Ævar Arnfjörð Bjarmason | 35af754 | 2021-03-28 15:15:38 +0200 | [diff] [blame] | 232 | enum fsck_msg_id msg_id, const char *fmt, ...) |
Johannes Schindelin | c99ba49 | 2015-06-22 17:25:09 +0200 | [diff] [blame] | 233 | { |
| 234 | va_list ap; |
| 235 | struct strbuf sb = STRBUF_INIT; |
Ævar Arnfjörð Bjarmason | 1b32b59 | 2021-03-28 15:15:40 +0200 | [diff] [blame] | 236 | enum fsck_msg_type msg_type = fsck_msg_type(msg_id, options); |
| 237 | int result; |
Johannes Schindelin | c99ba49 | 2015-06-22 17:25:09 +0200 | [diff] [blame] | 238 | |
Johannes Schindelin | efaba7c | 2015-06-22 17:26:48 +0200 | [diff] [blame] | 239 | if (msg_type == FSCK_IGNORE) |
| 240 | return 0; |
| 241 | |
Jeff King | 3837025 | 2019-10-18 00:59:15 -0400 | [diff] [blame] | 242 | if (object_on_skiplist(options, oid)) |
Johannes Schindelin | cd94c6f | 2015-06-22 17:27:18 +0200 | [diff] [blame] | 243 | return 0; |
| 244 | |
Johannes Schindelin | f50c440 | 2015-06-22 17:26:42 +0200 | [diff] [blame] | 245 | if (msg_type == FSCK_FATAL) |
| 246 | msg_type = FSCK_ERROR; |
Johannes Schindelin | f27d05b | 2015-06-22 17:26:54 +0200 | [diff] [blame] | 247 | else if (msg_type == FSCK_INFO) |
| 248 | msg_type = FSCK_WARN; |
Johannes Schindelin | f50c440 | 2015-06-22 17:26:42 +0200 | [diff] [blame] | 249 | |
Ævar Arnfjörð Bjarmason | 034a7b7 | 2021-03-28 15:15:37 +0200 | [diff] [blame] | 250 | prepare_msg_ids(); |
Ævar Arnfjörð Bjarmason | 35af754 | 2021-03-28 15:15:38 +0200 | [diff] [blame] | 251 | strbuf_addf(&sb, "%s: ", msg_id_info[msg_id].camelcased); |
Johannes Schindelin | 71ab8fa | 2015-06-22 17:25:52 +0200 | [diff] [blame] | 252 | |
Johannes Schindelin | c99ba49 | 2015-06-22 17:25:09 +0200 | [diff] [blame] | 253 | va_start(ap, fmt); |
| 254 | strbuf_vaddf(&sb, fmt, ap); |
Jeff King | 3837025 | 2019-10-18 00:59:15 -0400 | [diff] [blame] | 255 | result = options->error_func(options, oid, object_type, |
Ævar Arnfjörð Bjarmason | 394d5d3 | 2021-03-28 15:15:46 +0200 | [diff] [blame] | 256 | msg_type, msg_id, sb.buf); |
Johannes Schindelin | c99ba49 | 2015-06-22 17:25:09 +0200 | [diff] [blame] | 257 | strbuf_release(&sb); |
| 258 | va_end(ap); |
| 259 | |
| 260 | return result; |
| 261 | } |
| 262 | |
Jeff King | a59cfb3 | 2019-10-18 00:56:13 -0400 | [diff] [blame] | 263 | void fsck_enable_object_names(struct fsck_options *options) |
| 264 | { |
| 265 | if (!options->object_names) |
Jeff King | 7339029 | 2019-10-18 00:57:37 -0400 | [diff] [blame] | 266 | options->object_names = kh_init_oid_map(); |
Jeff King | a59cfb3 | 2019-10-18 00:56:13 -0400 | [diff] [blame] | 267 | } |
| 268 | |
Jeff King | 7339029 | 2019-10-18 00:57:37 -0400 | [diff] [blame] | 269 | const char *fsck_get_object_name(struct fsck_options *options, |
| 270 | const struct object_id *oid) |
Johannes Schindelin | 7b35efd | 2016-07-17 12:59:49 +0200 | [diff] [blame] | 271 | { |
Jeff King | 7339029 | 2019-10-18 00:57:37 -0400 | [diff] [blame] | 272 | khiter_t pos; |
Johannes Schindelin | 7b35efd | 2016-07-17 12:59:49 +0200 | [diff] [blame] | 273 | if (!options->object_names) |
| 274 | return NULL; |
Jeff King | 7339029 | 2019-10-18 00:57:37 -0400 | [diff] [blame] | 275 | pos = kh_get_oid_map(options->object_names, *oid); |
| 276 | if (pos >= kh_end(options->object_names)) |
| 277 | return NULL; |
| 278 | return kh_value(options->object_names, pos); |
Johannes Schindelin | 7b35efd | 2016-07-17 12:59:49 +0200 | [diff] [blame] | 279 | } |
| 280 | |
Jeff King | 7339029 | 2019-10-18 00:57:37 -0400 | [diff] [blame] | 281 | void fsck_put_object_name(struct fsck_options *options, |
| 282 | const struct object_id *oid, |
Jeff King | a59cfb3 | 2019-10-18 00:56:13 -0400 | [diff] [blame] | 283 | const char *fmt, ...) |
Johannes Schindelin | 7b35efd | 2016-07-17 12:59:49 +0200 | [diff] [blame] | 284 | { |
| 285 | va_list ap; |
| 286 | struct strbuf buf = STRBUF_INIT; |
Jeff King | 7339029 | 2019-10-18 00:57:37 -0400 | [diff] [blame] | 287 | khiter_t pos; |
| 288 | int hashret; |
Johannes Schindelin | 7b35efd | 2016-07-17 12:59:49 +0200 | [diff] [blame] | 289 | |
| 290 | if (!options->object_names) |
| 291 | return; |
Jeff King | 7339029 | 2019-10-18 00:57:37 -0400 | [diff] [blame] | 292 | |
| 293 | pos = kh_put_oid_map(options->object_names, *oid, &hashret); |
| 294 | if (!hashret) |
Johannes Schindelin | 7b35efd | 2016-07-17 12:59:49 +0200 | [diff] [blame] | 295 | return; |
| 296 | va_start(ap, fmt); |
| 297 | strbuf_vaddf(&buf, fmt, ap); |
Jeff King | 7339029 | 2019-10-18 00:57:37 -0400 | [diff] [blame] | 298 | kh_value(options->object_names, pos) = strbuf_detach(&buf, NULL); |
Johannes Schindelin | 7b35efd | 2016-07-17 12:59:49 +0200 | [diff] [blame] | 299 | va_end(ap); |
| 300 | } |
| 301 | |
Jeff King | a59cfb3 | 2019-10-18 00:56:13 -0400 | [diff] [blame] | 302 | const char *fsck_describe_object(struct fsck_options *options, |
Jeff King | 7339029 | 2019-10-18 00:57:37 -0400 | [diff] [blame] | 303 | const struct object_id *oid) |
Johannes Schindelin | 90cf590 | 2016-07-17 13:00:02 +0200 | [diff] [blame] | 304 | { |
Jeff King | a59cfb3 | 2019-10-18 00:56:13 -0400 | [diff] [blame] | 305 | static struct strbuf bufs[] = { |
| 306 | STRBUF_INIT, STRBUF_INIT, STRBUF_INIT, STRBUF_INIT |
| 307 | }; |
| 308 | static int b = 0; |
| 309 | struct strbuf *buf; |
Jeff King | 7339029 | 2019-10-18 00:57:37 -0400 | [diff] [blame] | 310 | const char *name = fsck_get_object_name(options, oid); |
Johannes Schindelin | 90cf590 | 2016-07-17 13:00:02 +0200 | [diff] [blame] | 311 | |
Jeff King | a59cfb3 | 2019-10-18 00:56:13 -0400 | [diff] [blame] | 312 | buf = bufs + b; |
| 313 | b = (b + 1) % ARRAY_SIZE(bufs); |
| 314 | strbuf_reset(buf); |
Jeff King | 7339029 | 2019-10-18 00:57:37 -0400 | [diff] [blame] | 315 | strbuf_addstr(buf, oid_to_hex(oid)); |
Jeff King | a59cfb3 | 2019-10-18 00:56:13 -0400 | [diff] [blame] | 316 | if (name) |
| 317 | strbuf_addf(buf, " (%s)", name); |
| 318 | |
| 319 | return buf->buf; |
Johannes Schindelin | 90cf590 | 2016-07-17 13:00:02 +0200 | [diff] [blame] | 320 | } |
| 321 | |
Johannes Schindelin | 2241054 | 2015-06-22 17:25:00 +0200 | [diff] [blame] | 322 | static int fsck_walk_tree(struct tree *tree, void *data, struct fsck_options *options) |
Martin Koegler | 355885d | 2008-02-25 22:46:04 +0100 | [diff] [blame] | 323 | { |
| 324 | struct tree_desc desc; |
| 325 | struct name_entry entry; |
| 326 | int res = 0; |
Johannes Schindelin | 7b35efd | 2016-07-17 12:59:49 +0200 | [diff] [blame] | 327 | const char *name; |
Martin Koegler | 355885d | 2008-02-25 22:46:04 +0100 | [diff] [blame] | 328 | |
| 329 | if (parse_tree(tree)) |
| 330 | return -1; |
| 331 | |
Jeff King | 7339029 | 2019-10-18 00:57:37 -0400 | [diff] [blame] | 332 | name = fsck_get_object_name(options, &tree->object.oid); |
Eric W. Biederman | efed687 | 2023-10-01 21:40:28 -0500 | [diff] [blame] | 333 | if (init_tree_desc_gently(&desc, &tree->object.oid, |
| 334 | tree->buffer, tree->size, 0)) |
David Turner | 8354fa3 | 2016-09-27 16:59:51 -0400 | [diff] [blame] | 335 | return -1; |
| 336 | while (tree_entry_gently(&desc, &entry)) { |
Johannes Schindelin | 7b35efd | 2016-07-17 12:59:49 +0200 | [diff] [blame] | 337 | struct object *obj; |
Martin Koegler | 355885d | 2008-02-25 22:46:04 +0100 | [diff] [blame] | 338 | int result; |
| 339 | |
| 340 | if (S_ISGITLINK(entry.mode)) |
| 341 | continue; |
Johannes Schindelin | 7b35efd | 2016-07-17 12:59:49 +0200 | [diff] [blame] | 342 | |
| 343 | if (S_ISDIR(entry.mode)) { |
brian m. carlson | ea82b2a | 2019-01-15 00:39:44 +0000 | [diff] [blame] | 344 | obj = (struct object *)lookup_tree(the_repository, &entry.oid); |
René Scharfe | 2720f6d | 2017-10-05 21:41:26 +0200 | [diff] [blame] | 345 | if (name && obj) |
Jeff King | 7339029 | 2019-10-18 00:57:37 -0400 | [diff] [blame] | 346 | fsck_put_object_name(options, &entry.oid, "%s%s/", |
Jeff King | a59cfb3 | 2019-10-18 00:56:13 -0400 | [diff] [blame] | 347 | name, entry.path); |
Johannes Schindelin | 7b35efd | 2016-07-17 12:59:49 +0200 | [diff] [blame] | 348 | result = options->walk(obj, OBJ_TREE, data, options); |
| 349 | } |
| 350 | else if (S_ISREG(entry.mode) || S_ISLNK(entry.mode)) { |
brian m. carlson | ea82b2a | 2019-01-15 00:39:44 +0000 | [diff] [blame] | 351 | obj = (struct object *)lookup_blob(the_repository, &entry.oid); |
René Scharfe | 2720f6d | 2017-10-05 21:41:26 +0200 | [diff] [blame] | 352 | if (name && obj) |
Jeff King | 7339029 | 2019-10-18 00:57:37 -0400 | [diff] [blame] | 353 | fsck_put_object_name(options, &entry.oid, "%s%s", |
Jeff King | a59cfb3 | 2019-10-18 00:56:13 -0400 | [diff] [blame] | 354 | name, entry.path); |
Johannes Schindelin | 7b35efd | 2016-07-17 12:59:49 +0200 | [diff] [blame] | 355 | result = options->walk(obj, OBJ_BLOB, data, options); |
| 356 | } |
Martin Koegler | 355885d | 2008-02-25 22:46:04 +0100 | [diff] [blame] | 357 | else { |
Pete Wyckoff | 82247e9 | 2012-04-29 20:28:45 -0400 | [diff] [blame] | 358 | result = error("in tree %s: entry %s has bad mode %.6o", |
Jeff King | 7339029 | 2019-10-18 00:57:37 -0400 | [diff] [blame] | 359 | fsck_describe_object(options, &tree->object.oid), |
Jeff King | a59cfb3 | 2019-10-18 00:56:13 -0400 | [diff] [blame] | 360 | entry.path, entry.mode); |
Martin Koegler | 355885d | 2008-02-25 22:46:04 +0100 | [diff] [blame] | 361 | } |
| 362 | if (result < 0) |
| 363 | return result; |
| 364 | if (!res) |
| 365 | res = result; |
| 366 | } |
| 367 | return res; |
| 368 | } |
| 369 | |
Johannes Schindelin | 2241054 | 2015-06-22 17:25:00 +0200 | [diff] [blame] | 370 | static int fsck_walk_commit(struct commit *commit, void *data, struct fsck_options *options) |
Martin Koegler | 355885d | 2008-02-25 22:46:04 +0100 | [diff] [blame] | 371 | { |
Johannes Schindelin | 7b35efd | 2016-07-17 12:59:49 +0200 | [diff] [blame] | 372 | int counter = 0, generation = 0, name_prefix_len = 0; |
Martin Koegler | 355885d | 2008-02-25 22:46:04 +0100 | [diff] [blame] | 373 | struct commit_list *parents; |
| 374 | int res; |
| 375 | int result; |
Johannes Schindelin | 7b35efd | 2016-07-17 12:59:49 +0200 | [diff] [blame] | 376 | const char *name; |
Martin Koegler | 355885d | 2008-02-25 22:46:04 +0100 | [diff] [blame] | 377 | |
Ævar Arnfjörð Bjarmason | ecb5091 | 2023-03-28 15:58:48 +0200 | [diff] [blame] | 378 | if (repo_parse_commit(the_repository, commit)) |
Martin Koegler | 355885d | 2008-02-25 22:46:04 +0100 | [diff] [blame] | 379 | return -1; |
| 380 | |
Jeff King | 7339029 | 2019-10-18 00:57:37 -0400 | [diff] [blame] | 381 | name = fsck_get_object_name(options, &commit->object.oid); |
Johannes Schindelin | 7b35efd | 2016-07-17 12:59:49 +0200 | [diff] [blame] | 382 | if (name) |
Jeff King | 7339029 | 2019-10-18 00:57:37 -0400 | [diff] [blame] | 383 | fsck_put_object_name(options, get_commit_tree_oid(commit), |
Jeff King | a59cfb3 | 2019-10-18 00:56:13 -0400 | [diff] [blame] | 384 | "%s:", name); |
Johannes Schindelin | 7b35efd | 2016-07-17 12:59:49 +0200 | [diff] [blame] | 385 | |
Ævar Arnfjörð Bjarmason | ecb5091 | 2023-03-28 15:58:48 +0200 | [diff] [blame] | 386 | result = options->walk((struct object *) repo_get_commit_tree(the_repository, commit), |
Derrick Stolee | 2e27bd7 | 2018-04-06 19:09:38 +0000 | [diff] [blame] | 387 | OBJ_TREE, data, options); |
Martin Koegler | 355885d | 2008-02-25 22:46:04 +0100 | [diff] [blame] | 388 | if (result < 0) |
| 389 | return result; |
| 390 | res = result; |
| 391 | |
| 392 | parents = commit->parents; |
Johannes Schindelin | 7b35efd | 2016-07-17 12:59:49 +0200 | [diff] [blame] | 393 | if (name && parents) { |
| 394 | int len = strlen(name), power; |
| 395 | |
| 396 | if (len && name[len - 1] == '^') { |
| 397 | generation = 1; |
| 398 | name_prefix_len = len - 1; |
| 399 | } |
| 400 | else { /* parse ~<generation> suffix */ |
| 401 | for (generation = 0, power = 1; |
| 402 | len && isdigit(name[len - 1]); |
| 403 | power *= 10) |
| 404 | generation += power * (name[--len] - '0'); |
| 405 | if (power > 1 && len && name[len - 1] == '~') |
| 406 | name_prefix_len = len - 1; |
Johannes Schindelin | e89f893 | 2021-02-10 18:01:30 +0000 | [diff] [blame] | 407 | else { |
| 408 | /* Maybe a non-first parent, e.g. HEAD^2 */ |
| 409 | generation = 0; |
| 410 | name_prefix_len = len; |
| 411 | } |
Johannes Schindelin | 7b35efd | 2016-07-17 12:59:49 +0200 | [diff] [blame] | 412 | } |
| 413 | } |
| 414 | |
Martin Koegler | 355885d | 2008-02-25 22:46:04 +0100 | [diff] [blame] | 415 | while (parents) { |
Johannes Schindelin | 7b35efd | 2016-07-17 12:59:49 +0200 | [diff] [blame] | 416 | if (name) { |
Jeff King | 7339029 | 2019-10-18 00:57:37 -0400 | [diff] [blame] | 417 | struct object_id *oid = &parents->item->object.oid; |
Johannes Schindelin | 7b35efd | 2016-07-17 12:59:49 +0200 | [diff] [blame] | 418 | |
Junio C Hamano | b84c7838 | 2018-10-24 10:25:12 +0900 | [diff] [blame] | 419 | if (counter++) |
Jeff King | 7339029 | 2019-10-18 00:57:37 -0400 | [diff] [blame] | 420 | fsck_put_object_name(options, oid, "%s^%d", |
Jeff King | a59cfb3 | 2019-10-18 00:56:13 -0400 | [diff] [blame] | 421 | name, counter); |
Johannes Schindelin | 7b35efd | 2016-07-17 12:59:49 +0200 | [diff] [blame] | 422 | else if (generation > 0) |
Jeff King | 7339029 | 2019-10-18 00:57:37 -0400 | [diff] [blame] | 423 | fsck_put_object_name(options, oid, "%.*s~%d", |
Jeff King | a59cfb3 | 2019-10-18 00:56:13 -0400 | [diff] [blame] | 424 | name_prefix_len, name, |
| 425 | generation + 1); |
Johannes Schindelin | 7b35efd | 2016-07-17 12:59:49 +0200 | [diff] [blame] | 426 | else |
Jeff King | 7339029 | 2019-10-18 00:57:37 -0400 | [diff] [blame] | 427 | fsck_put_object_name(options, oid, "%s^", name); |
Johannes Schindelin | 7b35efd | 2016-07-17 12:59:49 +0200 | [diff] [blame] | 428 | } |
Johannes Schindelin | 2241054 | 2015-06-22 17:25:00 +0200 | [diff] [blame] | 429 | result = options->walk((struct object *)parents->item, OBJ_COMMIT, data, options); |
Martin Koegler | 355885d | 2008-02-25 22:46:04 +0100 | [diff] [blame] | 430 | if (result < 0) |
| 431 | return result; |
| 432 | if (!res) |
| 433 | res = result; |
| 434 | parents = parents->next; |
| 435 | } |
| 436 | return res; |
| 437 | } |
| 438 | |
Johannes Schindelin | 2241054 | 2015-06-22 17:25:00 +0200 | [diff] [blame] | 439 | static int fsck_walk_tag(struct tag *tag, void *data, struct fsck_options *options) |
Martin Koegler | 355885d | 2008-02-25 22:46:04 +0100 | [diff] [blame] | 440 | { |
Jeff King | 7339029 | 2019-10-18 00:57:37 -0400 | [diff] [blame] | 441 | const char *name = fsck_get_object_name(options, &tag->object.oid); |
Johannes Schindelin | 7b35efd | 2016-07-17 12:59:49 +0200 | [diff] [blame] | 442 | |
Martin Koegler | 355885d | 2008-02-25 22:46:04 +0100 | [diff] [blame] | 443 | if (parse_tag(tag)) |
| 444 | return -1; |
Johannes Schindelin | 7b35efd | 2016-07-17 12:59:49 +0200 | [diff] [blame] | 445 | if (name) |
Jeff King | 7339029 | 2019-10-18 00:57:37 -0400 | [diff] [blame] | 446 | fsck_put_object_name(options, &tag->tagged->oid, "%s", name); |
Johannes Schindelin | 2241054 | 2015-06-22 17:25:00 +0200 | [diff] [blame] | 447 | return options->walk(tag->tagged, OBJ_ANY, data, options); |
Martin Koegler | 355885d | 2008-02-25 22:46:04 +0100 | [diff] [blame] | 448 | } |
| 449 | |
Johannes Schindelin | 2241054 | 2015-06-22 17:25:00 +0200 | [diff] [blame] | 450 | int fsck_walk(struct object *obj, void *data, struct fsck_options *options) |
Martin Koegler | 355885d | 2008-02-25 22:46:04 +0100 | [diff] [blame] | 451 | { |
| 452 | if (!obj) |
| 453 | return -1; |
Jeff King | a2b2285 | 2017-01-25 23:12:07 -0500 | [diff] [blame] | 454 | |
| 455 | if (obj->type == OBJ_NONE) |
Stefan Beller | 109cd76 | 2018-06-28 18:21:51 -0700 | [diff] [blame] | 456 | parse_object(the_repository, &obj->oid); |
Jeff King | a2b2285 | 2017-01-25 23:12:07 -0500 | [diff] [blame] | 457 | |
Martin Koegler | 355885d | 2008-02-25 22:46:04 +0100 | [diff] [blame] | 458 | switch (obj->type) { |
| 459 | case OBJ_BLOB: |
| 460 | return 0; |
| 461 | case OBJ_TREE: |
Johannes Schindelin | 2241054 | 2015-06-22 17:25:00 +0200 | [diff] [blame] | 462 | return fsck_walk_tree((struct tree *)obj, data, options); |
Martin Koegler | 355885d | 2008-02-25 22:46:04 +0100 | [diff] [blame] | 463 | case OBJ_COMMIT: |
Johannes Schindelin | 2241054 | 2015-06-22 17:25:00 +0200 | [diff] [blame] | 464 | return fsck_walk_commit((struct commit *)obj, data, options); |
Martin Koegler | 355885d | 2008-02-25 22:46:04 +0100 | [diff] [blame] | 465 | case OBJ_TAG: |
Johannes Schindelin | 2241054 | 2015-06-22 17:25:00 +0200 | [diff] [blame] | 466 | return fsck_walk_tag((struct tag *)obj, data, options); |
Martin Koegler | 355885d | 2008-02-25 22:46:04 +0100 | [diff] [blame] | 467 | default: |
Jeff King | a59cfb3 | 2019-10-18 00:56:13 -0400 | [diff] [blame] | 468 | error("Unknown object type for %s", |
Jeff King | 7339029 | 2019-10-18 00:57:37 -0400 | [diff] [blame] | 469 | fsck_describe_object(options, &obj->oid)); |
Martin Koegler | 355885d | 2008-02-25 22:46:04 +0100 | [diff] [blame] | 470 | return -1; |
| 471 | } |
| 472 | } |
Martin Koegler | ba002f3 | 2008-02-25 22:46:08 +0100 | [diff] [blame] | 473 | |
René Scharfe | 9068cfb | 2020-05-10 18:12:16 +0200 | [diff] [blame] | 474 | struct name_stack { |
| 475 | const char **names; |
| 476 | size_t nr, alloc; |
| 477 | }; |
| 478 | |
| 479 | static void name_stack_push(struct name_stack *stack, const char *name) |
| 480 | { |
| 481 | ALLOC_GROW(stack->names, stack->nr + 1, stack->alloc); |
| 482 | stack->names[stack->nr++] = name; |
| 483 | } |
| 484 | |
| 485 | static const char *name_stack_pop(struct name_stack *stack) |
| 486 | { |
| 487 | return stack->nr ? stack->names[--stack->nr] : NULL; |
| 488 | } |
| 489 | |
| 490 | static void name_stack_clear(struct name_stack *stack) |
| 491 | { |
| 492 | FREE_AND_NULL(stack->names); |
| 493 | stack->nr = stack->alloc = 0; |
| 494 | } |
| 495 | |
Martin Koegler | ba002f3 | 2008-02-25 22:46:08 +0100 | [diff] [blame] | 496 | /* |
| 497 | * The entries in a tree are ordered in the _path_ order, |
| 498 | * which means that a directory entry is ordered by adding |
| 499 | * a slash to the end of it. |
| 500 | * |
| 501 | * So a directory called "a" is ordered _after_ a file |
| 502 | * called "a.c", because "a/" sorts after "a.c". |
| 503 | */ |
| 504 | #define TREE_UNORDERED (-1) |
| 505 | #define TREE_HAS_DUPS (-2) |
| 506 | |
René Scharfe | 9068cfb | 2020-05-10 18:12:16 +0200 | [diff] [blame] | 507 | static int is_less_than_slash(unsigned char c) |
| 508 | { |
| 509 | return '\0' < c && c < '/'; |
| 510 | } |
| 511 | |
| 512 | static int verify_ordered(unsigned mode1, const char *name1, |
| 513 | unsigned mode2, const char *name2, |
| 514 | struct name_stack *candidates) |
Martin Koegler | ba002f3 | 2008-02-25 22:46:08 +0100 | [diff] [blame] | 515 | { |
| 516 | int len1 = strlen(name1); |
| 517 | int len2 = strlen(name2); |
| 518 | int len = len1 < len2 ? len1 : len2; |
| 519 | unsigned char c1, c2; |
| 520 | int cmp; |
| 521 | |
| 522 | cmp = memcmp(name1, name2, len); |
| 523 | if (cmp < 0) |
| 524 | return 0; |
| 525 | if (cmp > 0) |
| 526 | return TREE_UNORDERED; |
| 527 | |
| 528 | /* |
| 529 | * Ok, the first <len> characters are the same. |
| 530 | * Now we need to order the next one, but turn |
| 531 | * a '\0' into a '/' for a directory entry. |
| 532 | */ |
| 533 | c1 = name1[len]; |
| 534 | c2 = name2[len]; |
| 535 | if (!c1 && !c2) |
| 536 | /* |
| 537 | * git-write-tree used to write out a nonsense tree that has |
| 538 | * entries with the same name, one blob and one tree. Make |
| 539 | * sure we do not have duplicate entries. |
| 540 | */ |
| 541 | return TREE_HAS_DUPS; |
| 542 | if (!c1 && S_ISDIR(mode1)) |
| 543 | c1 = '/'; |
| 544 | if (!c2 && S_ISDIR(mode2)) |
| 545 | c2 = '/'; |
René Scharfe | 9068cfb | 2020-05-10 18:12:16 +0200 | [diff] [blame] | 546 | |
| 547 | /* |
| 548 | * There can be non-consecutive duplicates due to the implicitly |
René Scharfe | 8671559 | 2020-05-21 11:52:04 +0200 | [diff] [blame] | 549 | * added slash, e.g.: |
René Scharfe | 9068cfb | 2020-05-10 18:12:16 +0200 | [diff] [blame] | 550 | * |
| 551 | * foo |
| 552 | * foo.bar |
| 553 | * foo.bar.baz |
| 554 | * foo.bar/ |
| 555 | * foo/ |
| 556 | * |
| 557 | * Record non-directory candidates (like "foo" and "foo.bar" in |
| 558 | * the example) on a stack and check directory candidates (like |
| 559 | * foo/" and "foo.bar/") against that stack. |
| 560 | */ |
| 561 | if (!c1 && is_less_than_slash(c2)) { |
| 562 | name_stack_push(candidates, name1); |
| 563 | } else if (c2 == '/' && is_less_than_slash(c1)) { |
| 564 | for (;;) { |
| 565 | const char *p; |
| 566 | const char *f_name = name_stack_pop(candidates); |
| 567 | |
| 568 | if (!f_name) |
| 569 | break; |
| 570 | if (!skip_prefix(name2, f_name, &p)) |
René Scharfe | fe74704 | 2020-05-21 11:52:54 +0200 | [diff] [blame] | 571 | continue; |
René Scharfe | 9068cfb | 2020-05-10 18:12:16 +0200 | [diff] [blame] | 572 | if (!*p) |
| 573 | return TREE_HAS_DUPS; |
| 574 | if (is_less_than_slash(*p)) { |
| 575 | name_stack_push(candidates, f_name); |
| 576 | break; |
| 577 | } |
| 578 | } |
| 579 | } |
| 580 | |
Martin Koegler | ba002f3 | 2008-02-25 22:46:08 +0100 | [diff] [blame] | 581 | return c1 < c2 ? 0 : TREE_UNORDERED; |
| 582 | } |
| 583 | |
Jeff King | 9e1947c | 2021-05-01 11:41:38 -0400 | [diff] [blame] | 584 | static int fsck_tree(const struct object_id *tree_oid, |
Jeff King | 23a173a | 2019-10-18 00:54:12 -0400 | [diff] [blame] | 585 | const char *buffer, unsigned long size, |
| 586 | struct fsck_options *options) |
Martin Koegler | ba002f3 | 2008-02-25 22:46:08 +0100 | [diff] [blame] | 587 | { |
David Turner | 8354fa3 | 2016-09-27 16:59:51 -0400 | [diff] [blame] | 588 | int retval = 0; |
Jeff King | c479d14 | 2012-07-28 11:06:29 -0400 | [diff] [blame] | 589 | int has_null_sha1 = 0; |
Martin Koegler | ba002f3 | 2008-02-25 22:46:08 +0100 | [diff] [blame] | 590 | int has_full_path = 0; |
| 591 | int has_empty_name = 0; |
Jeff King | 5d34a43 | 2012-11-27 21:27:37 -0500 | [diff] [blame] | 592 | int has_dot = 0; |
| 593 | int has_dotdot = 0; |
Jeff King | 5c17f51 | 2012-11-28 16:35:29 -0500 | [diff] [blame] | 594 | int has_dotgit = 0; |
Martin Koegler | ba002f3 | 2008-02-25 22:46:08 +0100 | [diff] [blame] | 595 | int has_zero_pad = 0; |
| 596 | int has_bad_modes = 0; |
| 597 | int has_dup_entries = 0; |
| 598 | int not_properly_sorted = 0; |
Jeff King | 0fbcaef | 2023-08-31 02:20:01 -0400 | [diff] [blame] | 599 | int has_large_name = 0; |
Martin Koegler | ba002f3 | 2008-02-25 22:46:08 +0100 | [diff] [blame] | 600 | struct tree_desc desc; |
| 601 | unsigned o_mode; |
| 602 | const char *o_name; |
René Scharfe | 9068cfb | 2020-05-10 18:12:16 +0200 | [diff] [blame] | 603 | struct name_stack df_dup_candidates = { NULL }; |
Martin Koegler | ba002f3 | 2008-02-25 22:46:08 +0100 | [diff] [blame] | 604 | |
Eric W. Biederman | efed687 | 2023-10-01 21:40:28 -0500 | [diff] [blame] | 605 | if (init_tree_desc_gently(&desc, tree_oid, buffer, size, |
| 606 | TREE_DESC_RAW_MODES)) { |
Jeff King | 0282f67 | 2021-05-01 11:41:43 -0400 | [diff] [blame] | 607 | retval += report(options, tree_oid, OBJ_TREE, |
| 608 | FSCK_MSG_BAD_TREE, |
| 609 | "cannot be parsed as a tree"); |
David Turner | 8354fa3 | 2016-09-27 16:59:51 -0400 | [diff] [blame] | 610 | return retval; |
| 611 | } |
Martin Koegler | ba002f3 | 2008-02-25 22:46:08 +0100 | [diff] [blame] | 612 | |
| 613 | o_mode = 0; |
| 614 | o_name = NULL; |
Martin Koegler | ba002f3 | 2008-02-25 22:46:08 +0100 | [diff] [blame] | 615 | |
| 616 | while (desc.size) { |
Elijah Newren | 5ec1e72 | 2019-04-05 08:00:12 -0700 | [diff] [blame] | 617 | unsigned short mode; |
Johannes Schindelin | 288a74b | 2019-09-23 08:58:11 +0200 | [diff] [blame] | 618 | const char *name, *backslash; |
Jeff King | 9e1947c | 2021-05-01 11:41:38 -0400 | [diff] [blame] | 619 | const struct object_id *entry_oid; |
Martin Koegler | ba002f3 | 2008-02-25 22:46:08 +0100 | [diff] [blame] | 620 | |
Jeff King | 9e1947c | 2021-05-01 11:41:38 -0400 | [diff] [blame] | 621 | entry_oid = tree_entry_extract(&desc, &name, &mode); |
Martin Koegler | ba002f3 | 2008-02-25 22:46:08 +0100 | [diff] [blame] | 622 | |
Jeff King | 9e1947c | 2021-05-01 11:41:38 -0400 | [diff] [blame] | 623 | has_null_sha1 |= is_null_oid(entry_oid); |
Hiroyuki Sano | effd12e | 2014-03-20 08:02:04 +0900 | [diff] [blame] | 624 | has_full_path |= !!strchr(name, '/'); |
| 625 | has_empty_name |= !*name; |
| 626 | has_dot |= !strcmp(name, "."); |
| 627 | has_dotdot |= !strcmp(name, ".."); |
Jeff King | ed9c322 | 2018-05-13 12:35:37 -0400 | [diff] [blame] | 628 | has_dotgit |= is_hfs_dotgit(name) || is_ntfs_dotgit(name); |
Martin Koegler | ba002f3 | 2008-02-25 22:46:08 +0100 | [diff] [blame] | 629 | has_zero_pad |= *(char *)desc.buffer == '0'; |
Jeff King | 0fbcaef | 2023-08-31 02:20:01 -0400 | [diff] [blame] | 630 | has_large_name |= tree_entry_len(&desc.entry) > max_tree_entry_len; |
Jeff King | 159e7b0 | 2018-05-02 17:20:08 -0400 | [diff] [blame] | 631 | |
Jeff King | b7b1fca | 2018-05-04 20:03:35 -0400 | [diff] [blame] | 632 | if (is_hfs_dotgitmodules(name) || is_ntfs_dotgitmodules(name)) { |
| 633 | if (!S_ISLNK(mode)) |
Jeff King | 9e1947c | 2021-05-01 11:41:38 -0400 | [diff] [blame] | 634 | oidset_insert(&options->gitmodules_found, |
| 635 | entry_oid); |
Jeff King | b7b1fca | 2018-05-04 20:03:35 -0400 | [diff] [blame] | 636 | else |
Jeff King | 3837025 | 2019-10-18 00:59:15 -0400 | [diff] [blame] | 637 | retval += report(options, |
Jeff King | 9e1947c | 2021-05-01 11:41:38 -0400 | [diff] [blame] | 638 | tree_oid, OBJ_TREE, |
Jeff King | b7b1fca | 2018-05-04 20:03:35 -0400 | [diff] [blame] | 639 | FSCK_MSG_GITMODULES_SYMLINK, |
| 640 | ".gitmodules is a symbolic link"); |
| 641 | } |
Jeff King | 159e7b0 | 2018-05-02 17:20:08 -0400 | [diff] [blame] | 642 | |
Patrick Steinhardt | f8587c3 | 2022-12-01 15:46:05 +0100 | [diff] [blame] | 643 | if (is_hfs_dotgitattributes(name) || is_ntfs_dotgitattributes(name)) { |
Patrick Steinhardt | 27ab478 | 2022-12-01 15:46:09 +0100 | [diff] [blame] | 644 | if (!S_ISLNK(mode)) |
| 645 | oidset_insert(&options->gitattributes_found, |
| 646 | entry_oid); |
| 647 | else |
Patrick Steinhardt | f8587c3 | 2022-12-01 15:46:05 +0100 | [diff] [blame] | 648 | retval += report(options, tree_oid, OBJ_TREE, |
| 649 | FSCK_MSG_GITATTRIBUTES_SYMLINK, |
| 650 | ".gitattributes is a symlink"); |
| 651 | } |
| 652 | |
Jeff King | bb6832d | 2021-05-03 16:43:25 -0400 | [diff] [blame] | 653 | if (S_ISLNK(mode)) { |
| 654 | if (is_hfs_dotgitignore(name) || |
| 655 | is_ntfs_dotgitignore(name)) |
| 656 | retval += report(options, tree_oid, OBJ_TREE, |
| 657 | FSCK_MSG_GITIGNORE_SYMLINK, |
| 658 | ".gitignore is a symlink"); |
Jeff King | bb6832d | 2021-05-03 16:43:25 -0400 | [diff] [blame] | 659 | if (is_hfs_dotmailmap(name) || |
| 660 | is_ntfs_dotmailmap(name)) |
| 661 | retval += report(options, tree_oid, OBJ_TREE, |
| 662 | FSCK_MSG_MAILMAP_SYMLINK, |
| 663 | ".mailmap is a symlink"); |
| 664 | } |
| 665 | |
Johannes Schindelin | 288a74b | 2019-09-23 08:58:11 +0200 | [diff] [blame] | 666 | if ((backslash = strchr(name, '\\'))) { |
| 667 | while (backslash) { |
| 668 | backslash++; |
| 669 | has_dotgit |= is_ntfs_dotgit(backslash); |
Johannes Schindelin | bdfef04 | 2019-12-04 21:52:10 +0100 | [diff] [blame] | 670 | if (is_ntfs_dotgitmodules(backslash)) { |
| 671 | if (!S_ISLNK(mode)) |
Jeff King | 9e1947c | 2021-05-01 11:41:38 -0400 | [diff] [blame] | 672 | oidset_insert(&options->gitmodules_found, |
| 673 | entry_oid); |
Johannes Schindelin | bdfef04 | 2019-12-04 21:52:10 +0100 | [diff] [blame] | 674 | else |
Jeff King | 9e1947c | 2021-05-01 11:41:38 -0400 | [diff] [blame] | 675 | retval += report(options, tree_oid, OBJ_TREE, |
Johannes Schindelin | bdfef04 | 2019-12-04 21:52:10 +0100 | [diff] [blame] | 676 | FSCK_MSG_GITMODULES_SYMLINK, |
| 677 | ".gitmodules is a symbolic link"); |
| 678 | } |
Johannes Schindelin | 288a74b | 2019-09-23 08:58:11 +0200 | [diff] [blame] | 679 | backslash = strchr(backslash, '\\'); |
| 680 | } |
| 681 | } |
| 682 | |
David Turner | 8354fa3 | 2016-09-27 16:59:51 -0400 | [diff] [blame] | 683 | if (update_tree_entry_gently(&desc)) { |
Jeff King | 0282f67 | 2021-05-01 11:41:43 -0400 | [diff] [blame] | 684 | retval += report(options, tree_oid, OBJ_TREE, |
| 685 | FSCK_MSG_BAD_TREE, |
| 686 | "cannot be parsed as a tree"); |
David Turner | 8354fa3 | 2016-09-27 16:59:51 -0400 | [diff] [blame] | 687 | break; |
| 688 | } |
Martin Koegler | ba002f3 | 2008-02-25 22:46:08 +0100 | [diff] [blame] | 689 | |
| 690 | switch (mode) { |
| 691 | /* |
| 692 | * Standard modes.. |
| 693 | */ |
| 694 | case S_IFREG | 0755: |
| 695 | case S_IFREG | 0644: |
| 696 | case S_IFLNK: |
| 697 | case S_IFDIR: |
| 698 | case S_IFGITLINK: |
| 699 | break; |
| 700 | /* |
| 701 | * This is nonstandard, but we had a few of these |
| 702 | * early on when we honored the full set of mode |
| 703 | * bits.. |
| 704 | */ |
| 705 | case S_IFREG | 0664: |
Johannes Schindelin | 2241054 | 2015-06-22 17:25:00 +0200 | [diff] [blame] | 706 | if (!options->strict) |
Martin Koegler | ba002f3 | 2008-02-25 22:46:08 +0100 | [diff] [blame] | 707 | break; |
Jeff King | 1cf01a3 | 2017-09-21 02:25:41 -0400 | [diff] [blame] | 708 | /* fallthrough */ |
Martin Koegler | ba002f3 | 2008-02-25 22:46:08 +0100 | [diff] [blame] | 709 | default: |
| 710 | has_bad_modes = 1; |
| 711 | } |
| 712 | |
| 713 | if (o_name) { |
René Scharfe | 9068cfb | 2020-05-10 18:12:16 +0200 | [diff] [blame] | 714 | switch (verify_ordered(o_mode, o_name, mode, name, |
| 715 | &df_dup_candidates)) { |
Martin Koegler | ba002f3 | 2008-02-25 22:46:08 +0100 | [diff] [blame] | 716 | case TREE_UNORDERED: |
| 717 | not_properly_sorted = 1; |
| 718 | break; |
| 719 | case TREE_HAS_DUPS: |
| 720 | has_dup_entries = 1; |
| 721 | break; |
| 722 | default: |
| 723 | break; |
| 724 | } |
| 725 | } |
| 726 | |
| 727 | o_mode = mode; |
| 728 | o_name = name; |
Martin Koegler | ba002f3 | 2008-02-25 22:46:08 +0100 | [diff] [blame] | 729 | } |
| 730 | |
René Scharfe | 9068cfb | 2020-05-10 18:12:16 +0200 | [diff] [blame] | 731 | name_stack_clear(&df_dup_candidates); |
| 732 | |
Jeff King | c479d14 | 2012-07-28 11:06:29 -0400 | [diff] [blame] | 733 | if (has_null_sha1) |
Jeff King | 0282f67 | 2021-05-01 11:41:43 -0400 | [diff] [blame] | 734 | retval += report(options, tree_oid, OBJ_TREE, |
| 735 | FSCK_MSG_NULL_SHA1, |
| 736 | "contains entries pointing to null sha1"); |
Martin Koegler | ba002f3 | 2008-02-25 22:46:08 +0100 | [diff] [blame] | 737 | if (has_full_path) |
Jeff King | 0282f67 | 2021-05-01 11:41:43 -0400 | [diff] [blame] | 738 | retval += report(options, tree_oid, OBJ_TREE, |
| 739 | FSCK_MSG_FULL_PATHNAME, |
| 740 | "contains full pathnames"); |
Martin Koegler | ba002f3 | 2008-02-25 22:46:08 +0100 | [diff] [blame] | 741 | if (has_empty_name) |
Jeff King | 0282f67 | 2021-05-01 11:41:43 -0400 | [diff] [blame] | 742 | retval += report(options, tree_oid, OBJ_TREE, |
| 743 | FSCK_MSG_EMPTY_NAME, |
| 744 | "contains empty pathname"); |
Jeff King | 5d34a43 | 2012-11-27 21:27:37 -0500 | [diff] [blame] | 745 | if (has_dot) |
Jeff King | 0282f67 | 2021-05-01 11:41:43 -0400 | [diff] [blame] | 746 | retval += report(options, tree_oid, OBJ_TREE, |
| 747 | FSCK_MSG_HAS_DOT, |
| 748 | "contains '.'"); |
Jeff King | 5d34a43 | 2012-11-27 21:27:37 -0500 | [diff] [blame] | 749 | if (has_dotdot) |
Jeff King | 0282f67 | 2021-05-01 11:41:43 -0400 | [diff] [blame] | 750 | retval += report(options, tree_oid, OBJ_TREE, |
| 751 | FSCK_MSG_HAS_DOTDOT, |
| 752 | "contains '..'"); |
Jeff King | 5c17f51 | 2012-11-28 16:35:29 -0500 | [diff] [blame] | 753 | if (has_dotgit) |
Jeff King | 0282f67 | 2021-05-01 11:41:43 -0400 | [diff] [blame] | 754 | retval += report(options, tree_oid, OBJ_TREE, |
| 755 | FSCK_MSG_HAS_DOTGIT, |
| 756 | "contains '.git'"); |
Martin Koegler | ba002f3 | 2008-02-25 22:46:08 +0100 | [diff] [blame] | 757 | if (has_zero_pad) |
Jeff King | 0282f67 | 2021-05-01 11:41:43 -0400 | [diff] [blame] | 758 | retval += report(options, tree_oid, OBJ_TREE, |
| 759 | FSCK_MSG_ZERO_PADDED_FILEMODE, |
| 760 | "contains zero-padded file modes"); |
Martin Koegler | ba002f3 | 2008-02-25 22:46:08 +0100 | [diff] [blame] | 761 | if (has_bad_modes) |
Jeff King | 0282f67 | 2021-05-01 11:41:43 -0400 | [diff] [blame] | 762 | retval += report(options, tree_oid, OBJ_TREE, |
| 763 | FSCK_MSG_BAD_FILEMODE, |
| 764 | "contains bad file modes"); |
Martin Koegler | ba002f3 | 2008-02-25 22:46:08 +0100 | [diff] [blame] | 765 | if (has_dup_entries) |
Jeff King | 0282f67 | 2021-05-01 11:41:43 -0400 | [diff] [blame] | 766 | retval += report(options, tree_oid, OBJ_TREE, |
| 767 | FSCK_MSG_DUPLICATE_ENTRIES, |
| 768 | "contains duplicate file entries"); |
Martin Koegler | ba002f3 | 2008-02-25 22:46:08 +0100 | [diff] [blame] | 769 | if (not_properly_sorted) |
Jeff King | 0282f67 | 2021-05-01 11:41:43 -0400 | [diff] [blame] | 770 | retval += report(options, tree_oid, OBJ_TREE, |
| 771 | FSCK_MSG_TREE_NOT_SORTED, |
| 772 | "not properly sorted"); |
Jeff King | 0fbcaef | 2023-08-31 02:20:01 -0400 | [diff] [blame] | 773 | if (has_large_name) |
| 774 | retval += report(options, tree_oid, OBJ_TREE, |
| 775 | FSCK_MSG_LARGE_PATHNAME, |
| 776 | "contains excessively large pathname"); |
Martin Koegler | ba002f3 | 2008-02-25 22:46:08 +0100 | [diff] [blame] | 777 | return retval; |
| 778 | } |
| 779 | |
Jeff King | 8e43090 | 2023-01-19 18:13:29 -0500 | [diff] [blame] | 780 | /* |
| 781 | * Confirm that the headers of a commit or tag object end in a reasonable way, |
| 782 | * either with the usual "\n\n" separator, or at least with a trailing newline |
| 783 | * on the final header line. |
| 784 | * |
| 785 | * This property is important for the memory safety of our callers. It allows |
| 786 | * them to scan the buffer linewise without constantly checking the remaining |
| 787 | * size as long as: |
| 788 | * |
| 789 | * - they check that there are bytes left in the buffer at the start of any |
| 790 | * line (i.e., that the last newline they saw was not the final one we |
| 791 | * found here) |
| 792 | * |
| 793 | * - any intra-line scanning they do will stop at a newline, which will worst |
| 794 | * case hit the newline we found here as the end-of-header. This makes it |
| 795 | * OK for them to use helpers like parse_oid_hex(), or even skip_prefix(). |
| 796 | */ |
Junio C Hamano | 84d18c0 | 2015-06-28 11:18:31 -0700 | [diff] [blame] | 797 | static int verify_headers(const void *data, unsigned long size, |
Jeff King | cc57900 | 2019-10-18 01:00:50 -0400 | [diff] [blame] | 798 | const struct object_id *oid, enum object_type type, |
| 799 | struct fsck_options *options) |
Johannes Schindelin | 4d0d897 | 2014-09-11 16:26:33 +0200 | [diff] [blame] | 800 | { |
| 801 | const char *buffer = (const char *)data; |
| 802 | unsigned long i; |
| 803 | |
| 804 | for (i = 0; i < size; i++) { |
| 805 | switch (buffer[i]) { |
| 806 | case '\0': |
Jeff King | cc57900 | 2019-10-18 01:00:50 -0400 | [diff] [blame] | 807 | return report(options, oid, type, |
Johannes Schindelin | c99ba49 | 2015-06-22 17:25:09 +0200 | [diff] [blame] | 808 | FSCK_MSG_NUL_IN_HEADER, |
| 809 | "unterminated header: NUL at offset %ld", i); |
Johannes Schindelin | 4d0d897 | 2014-09-11 16:26:33 +0200 | [diff] [blame] | 810 | case '\n': |
| 811 | if (i + 1 < size && buffer[i + 1] == '\n') |
| 812 | return 0; |
| 813 | } |
| 814 | } |
| 815 | |
Junio C Hamano | 84d18c0 | 2015-06-28 11:18:31 -0700 | [diff] [blame] | 816 | /* |
| 817 | * We did not find double-LF that separates the header |
| 818 | * and the body. Not having a body is not a crime but |
| 819 | * we do want to see the terminating LF for the last header |
| 820 | * line. |
| 821 | */ |
| 822 | if (size && buffer[size - 1] == '\n') |
| 823 | return 0; |
| 824 | |
Jeff King | cc57900 | 2019-10-18 01:00:50 -0400 | [diff] [blame] | 825 | return report(options, oid, type, |
Johannes Schindelin | c99ba49 | 2015-06-22 17:25:09 +0200 | [diff] [blame] | 826 | FSCK_MSG_UNTERMINATED_HEADER, "unterminated header"); |
Johannes Schindelin | 4d0d897 | 2014-09-11 16:26:33 +0200 | [diff] [blame] | 827 | } |
| 828 | |
Jeff King | 7854399 | 2019-10-18 01:00:04 -0400 | [diff] [blame] | 829 | static int fsck_ident(const char **ident, |
| 830 | const struct object_id *oid, enum object_type type, |
| 831 | struct fsck_options *options) |
Jonathan Nieder | daae192 | 2010-04-24 11:06:08 -0500 | [diff] [blame] | 832 | { |
Johannes Schindelin | e6826e3 | 2015-06-22 17:26:03 +0200 | [diff] [blame] | 833 | const char *p = *ident; |
Jeff King | d4b8de0 | 2014-02-24 02:39:04 -0500 | [diff] [blame] | 834 | char *end; |
| 835 | |
Johannes Schindelin | e6826e3 | 2015-06-22 17:26:03 +0200 | [diff] [blame] | 836 | *ident = strchrnul(*ident, '\n'); |
| 837 | if (**ident == '\n') |
| 838 | (*ident)++; |
| 839 | |
| 840 | if (*p == '<') |
Jeff King | 7854399 | 2019-10-18 01:00:04 -0400 | [diff] [blame] | 841 | return report(options, oid, type, FSCK_MSG_MISSING_NAME_BEFORE_EMAIL, "invalid author/committer line - missing space before email"); |
Johannes Schindelin | e6826e3 | 2015-06-22 17:26:03 +0200 | [diff] [blame] | 842 | p += strcspn(p, "<>\n"); |
| 843 | if (*p == '>') |
Jeff King | 7854399 | 2019-10-18 01:00:04 -0400 | [diff] [blame] | 844 | return report(options, oid, type, FSCK_MSG_BAD_NAME, "invalid author/committer line - bad name"); |
Johannes Schindelin | e6826e3 | 2015-06-22 17:26:03 +0200 | [diff] [blame] | 845 | if (*p != '<') |
Jeff King | 7854399 | 2019-10-18 01:00:04 -0400 | [diff] [blame] | 846 | return report(options, oid, type, FSCK_MSG_MISSING_EMAIL, "invalid author/committer line - missing email"); |
Johannes Schindelin | e6826e3 | 2015-06-22 17:26:03 +0200 | [diff] [blame] | 847 | if (p[-1] != ' ') |
Jeff King | 7854399 | 2019-10-18 01:00:04 -0400 | [diff] [blame] | 848 | return report(options, oid, type, FSCK_MSG_MISSING_SPACE_BEFORE_EMAIL, "invalid author/committer line - missing space before email"); |
Johannes Schindelin | e6826e3 | 2015-06-22 17:26:03 +0200 | [diff] [blame] | 849 | p++; |
| 850 | p += strcspn(p, "<>\n"); |
| 851 | if (*p != '>') |
Jeff King | 7854399 | 2019-10-18 01:00:04 -0400 | [diff] [blame] | 852 | return report(options, oid, type, FSCK_MSG_BAD_EMAIL, "invalid author/committer line - bad email"); |
Johannes Schindelin | e6826e3 | 2015-06-22 17:26:03 +0200 | [diff] [blame] | 853 | p++; |
| 854 | if (*p != ' ') |
Jeff King | 7854399 | 2019-10-18 01:00:04 -0400 | [diff] [blame] | 855 | return report(options, oid, type, FSCK_MSG_MISSING_SPACE_BEFORE_DATE, "invalid author/committer line - missing space before date"); |
Johannes Schindelin | e6826e3 | 2015-06-22 17:26:03 +0200 | [diff] [blame] | 856 | p++; |
Jeff King | 8e43090 | 2023-01-19 18:13:29 -0500 | [diff] [blame] | 857 | /* |
| 858 | * Our timestamp parser is based on the C strto*() functions, which |
| 859 | * will happily eat whitespace, including the newline that is supposed |
| 860 | * to prevent us walking past the end of the buffer. So do our own |
| 861 | * scan, skipping linear whitespace but not newlines, and then |
| 862 | * confirming we found a digit. We _could_ be even more strict here, |
| 863 | * as we really expect only a single space, but since we have |
| 864 | * traditionally allowed extra whitespace, we'll continue to do so. |
| 865 | */ |
| 866 | while (*p == ' ' || *p == '\t') |
| 867 | p++; |
| 868 | if (!isdigit(*p)) |
| 869 | return report(options, oid, type, FSCK_MSG_BAD_DATE, |
| 870 | "invalid author/committer line - bad date"); |
Johannes Schindelin | e6826e3 | 2015-06-22 17:26:03 +0200 | [diff] [blame] | 871 | if (*p == '0' && p[1] != ' ') |
Jeff King | 7854399 | 2019-10-18 01:00:04 -0400 | [diff] [blame] | 872 | return report(options, oid, type, FSCK_MSG_ZERO_PADDED_DATE, "invalid author/committer line - zero-padded date"); |
Johannes Schindelin | 1aeb7e7 | 2017-04-21 12:45:44 +0200 | [diff] [blame] | 873 | if (date_overflows(parse_timestamp(p, &end, 10))) |
Jeff King | 7854399 | 2019-10-18 01:00:04 -0400 | [diff] [blame] | 874 | return report(options, oid, type, FSCK_MSG_BAD_DATE_OVERFLOW, "invalid author/committer line - date causes integer overflow"); |
Johannes Schindelin | e6826e3 | 2015-06-22 17:26:03 +0200 | [diff] [blame] | 875 | if ((end == p || *end != ' ')) |
Jeff King | 7854399 | 2019-10-18 01:00:04 -0400 | [diff] [blame] | 876 | return report(options, oid, type, FSCK_MSG_BAD_DATE, "invalid author/committer line - bad date"); |
Johannes Schindelin | e6826e3 | 2015-06-22 17:26:03 +0200 | [diff] [blame] | 877 | p = end + 1; |
| 878 | if ((*p != '+' && *p != '-') || |
| 879 | !isdigit(p[1]) || |
| 880 | !isdigit(p[2]) || |
| 881 | !isdigit(p[3]) || |
| 882 | !isdigit(p[4]) || |
| 883 | (p[5] != '\n')) |
Jeff King | 7854399 | 2019-10-18 01:00:04 -0400 | [diff] [blame] | 884 | return report(options, oid, type, FSCK_MSG_BAD_TIMEZONE, "invalid author/committer line - bad time zone"); |
Johannes Schindelin | e6826e3 | 2015-06-22 17:26:03 +0200 | [diff] [blame] | 885 | p += 6; |
Jonathan Nieder | daae192 | 2010-04-24 11:06:08 -0500 | [diff] [blame] | 886 | return 0; |
| 887 | } |
| 888 | |
Jeff King | c5b4269 | 2019-10-18 01:01:48 -0400 | [diff] [blame] | 889 | static int fsck_commit(const struct object_id *oid, |
| 890 | const char *buffer, unsigned long size, |
| 891 | struct fsck_options *options) |
Martin Koegler | ba002f3 | 2008-02-25 22:46:08 +0100 | [diff] [blame] | 892 | { |
Jeff King | f648ee7 | 2019-10-18 01:00:59 -0400 | [diff] [blame] | 893 | struct object_id tree_oid, parent_oid; |
Jeff King | ec65231 | 2019-10-18 00:49:10 -0400 | [diff] [blame] | 894 | unsigned author_count; |
Jonathan Nieder | daae192 | 2010-04-24 11:06:08 -0500 | [diff] [blame] | 895 | int err; |
Junio C Hamano | 6d2d780 | 2016-04-14 10:58:22 -0700 | [diff] [blame] | 896 | const char *buffer_begin = buffer; |
Jeff King | 8e43090 | 2023-01-19 18:13:29 -0500 | [diff] [blame] | 897 | const char *buffer_end = buffer + size; |
brian m. carlson | c54f5ca | 2018-05-02 00:25:41 +0000 | [diff] [blame] | 898 | const char *p; |
Martin Koegler | ba002f3 | 2008-02-25 22:46:08 +0100 | [diff] [blame] | 899 | |
Jeff King | 8e43090 | 2023-01-19 18:13:29 -0500 | [diff] [blame] | 900 | /* |
| 901 | * We _must_ stop parsing immediately if this reports failure, as the |
| 902 | * memory safety of the rest of the function depends on it. See the |
| 903 | * comment above the definition of verify_headers() for more details. |
| 904 | */ |
Jeff King | c5b4269 | 2019-10-18 01:01:48 -0400 | [diff] [blame] | 905 | if (verify_headers(buffer, size, oid, OBJ_COMMIT, options)) |
Johannes Schindelin | 4d0d897 | 2014-09-11 16:26:33 +0200 | [diff] [blame] | 906 | return -1; |
| 907 | |
Jeff King | 8e43090 | 2023-01-19 18:13:29 -0500 | [diff] [blame] | 908 | if (buffer >= buffer_end || !skip_prefix(buffer, "tree ", &buffer)) |
Jeff King | c5b4269 | 2019-10-18 01:01:48 -0400 | [diff] [blame] | 909 | return report(options, oid, OBJ_COMMIT, FSCK_MSG_MISSING_TREE, "invalid format - expected 'tree' line"); |
brian m. carlson | c54f5ca | 2018-05-02 00:25:41 +0000 | [diff] [blame] | 910 | if (parse_oid_hex(buffer, &tree_oid, &p) || *p != '\n') { |
Jeff King | c5b4269 | 2019-10-18 01:01:48 -0400 | [diff] [blame] | 911 | err = report(options, oid, OBJ_COMMIT, FSCK_MSG_BAD_TREE_SHA1, "invalid 'tree' line format - bad sha1"); |
Johannes Schindelin | b358476 | 2015-06-22 17:26:11 +0200 | [diff] [blame] | 912 | if (err) |
| 913 | return err; |
| 914 | } |
brian m. carlson | c54f5ca | 2018-05-02 00:25:41 +0000 | [diff] [blame] | 915 | buffer = p + 1; |
Jeff King | 8e43090 | 2023-01-19 18:13:29 -0500 | [diff] [blame] | 916 | while (buffer < buffer_end && skip_prefix(buffer, "parent ", &buffer)) { |
Jeff King | f648ee7 | 2019-10-18 01:00:59 -0400 | [diff] [blame] | 917 | if (parse_oid_hex(buffer, &parent_oid, &p) || *p != '\n') { |
Jeff King | c5b4269 | 2019-10-18 01:01:48 -0400 | [diff] [blame] | 918 | err = report(options, oid, OBJ_COMMIT, FSCK_MSG_BAD_PARENT_SHA1, "invalid 'parent' line format - bad sha1"); |
Johannes Schindelin | b358476 | 2015-06-22 17:26:11 +0200 | [diff] [blame] | 919 | if (err) |
| 920 | return err; |
| 921 | } |
brian m. carlson | c54f5ca | 2018-05-02 00:25:41 +0000 | [diff] [blame] | 922 | buffer = p + 1; |
Martin Koegler | ba002f3 | 2008-02-25 22:46:08 +0100 | [diff] [blame] | 923 | } |
Johannes Schindelin | c9ad147 | 2015-06-22 17:26:23 +0200 | [diff] [blame] | 924 | author_count = 0; |
Jeff King | 8e43090 | 2023-01-19 18:13:29 -0500 | [diff] [blame] | 925 | while (buffer < buffer_end && skip_prefix(buffer, "author ", &buffer)) { |
Johannes Schindelin | c9ad147 | 2015-06-22 17:26:23 +0200 | [diff] [blame] | 926 | author_count++; |
Jeff King | c5b4269 | 2019-10-18 01:01:48 -0400 | [diff] [blame] | 927 | err = fsck_ident(&buffer, oid, OBJ_COMMIT, options); |
Johannes Schindelin | c9ad147 | 2015-06-22 17:26:23 +0200 | [diff] [blame] | 928 | if (err) |
| 929 | return err; |
| 930 | } |
| 931 | if (author_count < 1) |
Jeff King | c5b4269 | 2019-10-18 01:01:48 -0400 | [diff] [blame] | 932 | err = report(options, oid, OBJ_COMMIT, FSCK_MSG_MISSING_AUTHOR, "invalid format - expected 'author' line"); |
Johannes Schindelin | c9ad147 | 2015-06-22 17:26:23 +0200 | [diff] [blame] | 933 | else if (author_count > 1) |
Jeff King | c5b4269 | 2019-10-18 01:01:48 -0400 | [diff] [blame] | 934 | err = report(options, oid, OBJ_COMMIT, FSCK_MSG_MULTIPLE_AUTHORS, "invalid format - multiple 'author' lines"); |
Jonathan Nieder | daae192 | 2010-04-24 11:06:08 -0500 | [diff] [blame] | 935 | if (err) |
| 936 | return err; |
Jeff King | 8e43090 | 2023-01-19 18:13:29 -0500 | [diff] [blame] | 937 | if (buffer >= buffer_end || !skip_prefix(buffer, "committer ", &buffer)) |
Jeff King | c5b4269 | 2019-10-18 01:01:48 -0400 | [diff] [blame] | 938 | return report(options, oid, OBJ_COMMIT, FSCK_MSG_MISSING_COMMITTER, "invalid format - expected 'committer' line"); |
| 939 | err = fsck_ident(&buffer, oid, OBJ_COMMIT, options); |
Jonathan Nieder | daae192 | 2010-04-24 11:06:08 -0500 | [diff] [blame] | 940 | if (err) |
| 941 | return err; |
Junio C Hamano | 6d2d780 | 2016-04-14 10:58:22 -0700 | [diff] [blame] | 942 | if (memchr(buffer_begin, '\0', size)) { |
Jeff King | c5b4269 | 2019-10-18 01:01:48 -0400 | [diff] [blame] | 943 | err = report(options, oid, OBJ_COMMIT, FSCK_MSG_NUL_IN_COMMIT, |
Junio C Hamano | 6d2d780 | 2016-04-14 10:58:22 -0700 | [diff] [blame] | 944 | "NUL byte in the commit object body"); |
| 945 | if (err) |
| 946 | return err; |
| 947 | } |
Martin Koegler | ba002f3 | 2008-02-25 22:46:08 +0100 | [diff] [blame] | 948 | return 0; |
| 949 | } |
| 950 | |
Jeff King | 103fb6d | 2019-10-18 01:01:26 -0400 | [diff] [blame] | 951 | static int fsck_tag(const struct object_id *oid, const char *buffer, |
Jeff King | 2175a0c | 2019-10-18 00:51:19 -0400 | [diff] [blame] | 952 | unsigned long size, struct fsck_options *options) |
Johannes Schindelin | cec097b | 2014-09-11 16:26:38 +0200 | [diff] [blame] | 953 | { |
Jeff King | f648ee7 | 2019-10-18 01:00:59 -0400 | [diff] [blame] | 954 | struct object_id tagged_oid; |
Ævar Arnfjörð Bjarmason | acf9de4 | 2021-01-05 20:42:46 +0100 | [diff] [blame] | 955 | int tagged_type; |
| 956 | return fsck_tag_standalone(oid, buffer, size, options, &tagged_oid, |
| 957 | &tagged_type); |
| 958 | } |
| 959 | |
| 960 | int fsck_tag_standalone(const struct object_id *oid, const char *buffer, |
| 961 | unsigned long size, struct fsck_options *options, |
| 962 | struct object_id *tagged_oid, |
| 963 | int *tagged_type) |
| 964 | { |
Johannes Schindelin | cec097b | 2014-09-11 16:26:38 +0200 | [diff] [blame] | 965 | int ret = 0; |
Jeff King | 23a173a | 2019-10-18 00:54:12 -0400 | [diff] [blame] | 966 | char *eol; |
Johannes Schindelin | cec097b | 2014-09-11 16:26:38 +0200 | [diff] [blame] | 967 | struct strbuf sb = STRBUF_INIT; |
Jeff King | 8e43090 | 2023-01-19 18:13:29 -0500 | [diff] [blame] | 968 | const char *buffer_end = buffer + size; |
brian m. carlson | c54f5ca | 2018-05-02 00:25:41 +0000 | [diff] [blame] | 969 | const char *p; |
Johannes Schindelin | cec097b | 2014-09-11 16:26:38 +0200 | [diff] [blame] | 970 | |
Jeff King | 8e43090 | 2023-01-19 18:13:29 -0500 | [diff] [blame] | 971 | /* |
| 972 | * We _must_ stop parsing immediately if this reports failure, as the |
| 973 | * memory safety of the rest of the function depends on it. See the |
| 974 | * comment above the definition of verify_headers() for more details. |
| 975 | */ |
Jeff King | 103fb6d | 2019-10-18 01:01:26 -0400 | [diff] [blame] | 976 | ret = verify_headers(buffer, size, oid, OBJ_TAG, options); |
René Scharfe | 8a272f2 | 2015-11-19 17:25:31 +0100 | [diff] [blame] | 977 | if (ret) |
Johannes Schindelin | cec097b | 2014-09-11 16:26:38 +0200 | [diff] [blame] | 978 | goto done; |
| 979 | |
Jeff King | 8e43090 | 2023-01-19 18:13:29 -0500 | [diff] [blame] | 980 | if (buffer >= buffer_end || !skip_prefix(buffer, "object ", &buffer)) { |
Jeff King | 103fb6d | 2019-10-18 01:01:26 -0400 | [diff] [blame] | 981 | ret = report(options, oid, OBJ_TAG, FSCK_MSG_MISSING_OBJECT, "invalid format - expected 'object' line"); |
Johannes Schindelin | cec097b | 2014-09-11 16:26:38 +0200 | [diff] [blame] | 982 | goto done; |
| 983 | } |
Ævar Arnfjörð Bjarmason | acf9de4 | 2021-01-05 20:42:46 +0100 | [diff] [blame] | 984 | if (parse_oid_hex(buffer, tagged_oid, &p) || *p != '\n') { |
Jeff King | 103fb6d | 2019-10-18 01:01:26 -0400 | [diff] [blame] | 985 | ret = report(options, oid, OBJ_TAG, FSCK_MSG_BAD_OBJECT_SHA1, "invalid 'object' line format - bad sha1"); |
Johannes Schindelin | 7d7d5b0 | 2015-06-22 17:26:30 +0200 | [diff] [blame] | 986 | if (ret) |
| 987 | goto done; |
Johannes Schindelin | cec097b | 2014-09-11 16:26:38 +0200 | [diff] [blame] | 988 | } |
brian m. carlson | c54f5ca | 2018-05-02 00:25:41 +0000 | [diff] [blame] | 989 | buffer = p + 1; |
Johannes Schindelin | cec097b | 2014-09-11 16:26:38 +0200 | [diff] [blame] | 990 | |
Jeff King | 8e43090 | 2023-01-19 18:13:29 -0500 | [diff] [blame] | 991 | if (buffer >= buffer_end || !skip_prefix(buffer, "type ", &buffer)) { |
Jeff King | 103fb6d | 2019-10-18 01:01:26 -0400 | [diff] [blame] | 992 | ret = report(options, oid, OBJ_TAG, FSCK_MSG_MISSING_TYPE_ENTRY, "invalid format - expected 'type' line"); |
Johannes Schindelin | cec097b | 2014-09-11 16:26:38 +0200 | [diff] [blame] | 993 | goto done; |
| 994 | } |
Jeff King | 8e43090 | 2023-01-19 18:13:29 -0500 | [diff] [blame] | 995 | eol = memchr(buffer, '\n', buffer_end - buffer); |
Johannes Schindelin | cec097b | 2014-09-11 16:26:38 +0200 | [diff] [blame] | 996 | if (!eol) { |
Jeff King | 103fb6d | 2019-10-18 01:01:26 -0400 | [diff] [blame] | 997 | ret = report(options, oid, OBJ_TAG, FSCK_MSG_MISSING_TYPE, "invalid format - unexpected end after 'type' line"); |
Johannes Schindelin | cec097b | 2014-09-11 16:26:38 +0200 | [diff] [blame] | 998 | goto done; |
| 999 | } |
Ævar Arnfjörð Bjarmason | acf9de4 | 2021-01-05 20:42:46 +0100 | [diff] [blame] | 1000 | *tagged_type = type_from_string_gently(buffer, eol - buffer, 1); |
| 1001 | if (*tagged_type < 0) |
Jeff King | 103fb6d | 2019-10-18 01:01:26 -0400 | [diff] [blame] | 1002 | ret = report(options, oid, OBJ_TAG, FSCK_MSG_BAD_TYPE, "invalid 'type' value"); |
Johannes Schindelin | cec097b | 2014-09-11 16:26:38 +0200 | [diff] [blame] | 1003 | if (ret) |
| 1004 | goto done; |
| 1005 | buffer = eol + 1; |
| 1006 | |
Jeff King | 8e43090 | 2023-01-19 18:13:29 -0500 | [diff] [blame] | 1007 | if (buffer >= buffer_end || !skip_prefix(buffer, "tag ", &buffer)) { |
Jeff King | 103fb6d | 2019-10-18 01:01:26 -0400 | [diff] [blame] | 1008 | ret = report(options, oid, OBJ_TAG, FSCK_MSG_MISSING_TAG_ENTRY, "invalid format - expected 'tag' line"); |
Johannes Schindelin | cec097b | 2014-09-11 16:26:38 +0200 | [diff] [blame] | 1009 | goto done; |
| 1010 | } |
Jeff King | 8e43090 | 2023-01-19 18:13:29 -0500 | [diff] [blame] | 1011 | eol = memchr(buffer, '\n', buffer_end - buffer); |
Johannes Schindelin | cec097b | 2014-09-11 16:26:38 +0200 | [diff] [blame] | 1012 | if (!eol) { |
Jeff King | 103fb6d | 2019-10-18 01:01:26 -0400 | [diff] [blame] | 1013 | ret = report(options, oid, OBJ_TAG, FSCK_MSG_MISSING_TAG, "invalid format - unexpected end after 'type' line"); |
Johannes Schindelin | cec097b | 2014-09-11 16:26:38 +0200 | [diff] [blame] | 1014 | goto done; |
| 1015 | } |
| 1016 | strbuf_addf(&sb, "refs/tags/%.*s", (int)(eol - buffer), buffer); |
Johannes Schindelin | f27d05b | 2015-06-22 17:26:54 +0200 | [diff] [blame] | 1017 | if (check_refname_format(sb.buf, 0)) { |
Jeff King | 103fb6d | 2019-10-18 01:01:26 -0400 | [diff] [blame] | 1018 | ret = report(options, oid, OBJ_TAG, |
Jeff King | 3837025 | 2019-10-18 00:59:15 -0400 | [diff] [blame] | 1019 | FSCK_MSG_BAD_TAG_NAME, |
| 1020 | "invalid 'tag' name: %.*s", |
| 1021 | (int)(eol - buffer), buffer); |
Johannes Schindelin | f27d05b | 2015-06-22 17:26:54 +0200 | [diff] [blame] | 1022 | if (ret) |
| 1023 | goto done; |
| 1024 | } |
Johannes Schindelin | cec097b | 2014-09-11 16:26:38 +0200 | [diff] [blame] | 1025 | buffer = eol + 1; |
| 1026 | |
Jeff King | 8e43090 | 2023-01-19 18:13:29 -0500 | [diff] [blame] | 1027 | if (buffer >= buffer_end || !skip_prefix(buffer, "tagger ", &buffer)) { |
Johannes Schindelin | cec097b | 2014-09-11 16:26:38 +0200 | [diff] [blame] | 1028 | /* early tags do not contain 'tagger' lines; warn only */ |
Jeff King | 103fb6d | 2019-10-18 01:01:26 -0400 | [diff] [blame] | 1029 | ret = report(options, oid, OBJ_TAG, FSCK_MSG_MISSING_TAGGER_ENTRY, "invalid format - expected 'tagger' line"); |
Johannes Schindelin | f27d05b | 2015-06-22 17:26:54 +0200 | [diff] [blame] | 1030 | if (ret) |
| 1031 | goto done; |
| 1032 | } |
Johannes Schindelin | cec097b | 2014-09-11 16:26:38 +0200 | [diff] [blame] | 1033 | else |
Jeff King | 103fb6d | 2019-10-18 01:01:26 -0400 | [diff] [blame] | 1034 | ret = fsck_ident(&buffer, oid, OBJ_TAG, options); |
Johannes Schindelin | cec097b | 2014-09-11 16:26:38 +0200 | [diff] [blame] | 1035 | |
Jeff King | 8e43090 | 2023-01-19 18:13:29 -0500 | [diff] [blame] | 1036 | if (buffer < buffer_end && !starts_with(buffer, "\n")) { |
Ævar Arnfjörð Bjarmason | acf9de4 | 2021-01-05 20:42:46 +0100 | [diff] [blame] | 1037 | /* |
| 1038 | * The verify_headers() check will allow |
| 1039 | * e.g. "[...]tagger <tagger>\nsome |
| 1040 | * garbage\n\nmessage" to pass, thinking "some |
| 1041 | * garbage" could be a custom header. E.g. "mktag" |
| 1042 | * doesn't want any unknown headers. |
| 1043 | */ |
| 1044 | ret = report(options, oid, OBJ_TAG, FSCK_MSG_EXTRA_HEADER_ENTRY, "invalid format - extra header(s) after 'tagger'"); |
| 1045 | if (ret) |
| 1046 | goto done; |
| 1047 | } |
| 1048 | |
Johannes Schindelin | cec097b | 2014-09-11 16:26:38 +0200 | [diff] [blame] | 1049 | done: |
| 1050 | strbuf_release(&sb); |
Johannes Schindelin | cec097b | 2014-09-11 16:26:38 +0200 | [diff] [blame] | 1051 | return ret; |
| 1052 | } |
| 1053 | |
Jeff King | ed8b10f | 2018-05-02 17:25:27 -0400 | [diff] [blame] | 1054 | struct fsck_gitmodules_data { |
Jeff King | 6da40b2 | 2019-10-18 00:59:29 -0400 | [diff] [blame] | 1055 | const struct object_id *oid; |
Jeff King | ed8b10f | 2018-05-02 17:25:27 -0400 | [diff] [blame] | 1056 | struct fsck_options *options; |
| 1057 | int ret; |
| 1058 | }; |
| 1059 | |
Glen Choo | a4e7e31 | 2023-06-28 19:26:22 +0000 | [diff] [blame] | 1060 | static int fsck_gitmodules_fn(const char *var, const char *value, |
| 1061 | const struct config_context *ctx UNUSED, |
| 1062 | void *vdata) |
Jeff King | ed8b10f | 2018-05-02 17:25:27 -0400 | [diff] [blame] | 1063 | { |
| 1064 | struct fsck_gitmodules_data *data = vdata; |
| 1065 | const char *subsection, *key; |
Jeff King | f5914f4 | 2020-04-10 15:44:28 -0400 | [diff] [blame] | 1066 | size_t subsection_len; |
Jeff King | ed8b10f | 2018-05-02 17:25:27 -0400 | [diff] [blame] | 1067 | char *name; |
| 1068 | |
| 1069 | if (parse_config_key(var, "submodule", &subsection, &subsection_len, &key) < 0 || |
| 1070 | !subsection) |
| 1071 | return 0; |
| 1072 | |
| 1073 | name = xmemdupz(subsection, subsection_len); |
| 1074 | if (check_submodule_name(name) < 0) |
Jeff King | 3837025 | 2019-10-18 00:59:15 -0400 | [diff] [blame] | 1075 | data->ret |= report(data->options, |
Jeff King | 6da40b2 | 2019-10-18 00:59:29 -0400 | [diff] [blame] | 1076 | data->oid, OBJ_BLOB, |
Jeff King | ed8b10f | 2018-05-02 17:25:27 -0400 | [diff] [blame] | 1077 | FSCK_MSG_GITMODULES_NAME, |
| 1078 | "disallowed submodule name: %s", |
| 1079 | name); |
Jeff King | a124133 | 2018-09-24 04:37:17 -0400 | [diff] [blame] | 1080 | if (!strcmp(key, "url") && value && |
Jeff King | 07259e7 | 2020-03-11 18:48:24 -0400 | [diff] [blame] | 1081 | check_submodule_url(value) < 0) |
Jeff King | 3837025 | 2019-10-18 00:59:15 -0400 | [diff] [blame] | 1082 | data->ret |= report(data->options, |
Jeff King | 6da40b2 | 2019-10-18 00:59:29 -0400 | [diff] [blame] | 1083 | data->oid, OBJ_BLOB, |
Jeff King | a124133 | 2018-09-24 04:37:17 -0400 | [diff] [blame] | 1084 | FSCK_MSG_GITMODULES_URL, |
| 1085 | "disallowed submodule url: %s", |
| 1086 | value); |
Jeff King | 1a7fd1f | 2018-09-24 04:42:19 -0400 | [diff] [blame] | 1087 | if (!strcmp(key, "path") && value && |
| 1088 | looks_like_command_line_option(value)) |
Jeff King | 3837025 | 2019-10-18 00:59:15 -0400 | [diff] [blame] | 1089 | data->ret |= report(data->options, |
Jeff King | 6da40b2 | 2019-10-18 00:59:29 -0400 | [diff] [blame] | 1090 | data->oid, OBJ_BLOB, |
Jeff King | 1a7fd1f | 2018-09-24 04:42:19 -0400 | [diff] [blame] | 1091 | FSCK_MSG_GITMODULES_PATH, |
| 1092 | "disallowed submodule path: %s", |
| 1093 | value); |
Jonathan Nieder | bb92255 | 2019-12-05 01:30:43 -0800 | [diff] [blame] | 1094 | if (!strcmp(key, "update") && value && |
| 1095 | parse_submodule_update_type(value) == SM_UPDATE_COMMAND) |
Junio C Hamano | 7034cd0 | 2019-12-09 22:17:55 -0800 | [diff] [blame] | 1096 | data->ret |= report(data->options, data->oid, OBJ_BLOB, |
Jonathan Nieder | bb92255 | 2019-12-05 01:30:43 -0800 | [diff] [blame] | 1097 | FSCK_MSG_GITMODULES_UPDATE, |
| 1098 | "disallowed submodule update setting: %s", |
| 1099 | value); |
Jeff King | ed8b10f | 2018-05-02 17:25:27 -0400 | [diff] [blame] | 1100 | free(name); |
| 1101 | |
| 1102 | return 0; |
| 1103 | } |
| 1104 | |
Jeff King | 6da40b2 | 2019-10-18 00:59:29 -0400 | [diff] [blame] | 1105 | static int fsck_blob(const struct object_id *oid, const char *buf, |
Jeff King | 7ac4f3a | 2018-05-02 15:44:51 -0400 | [diff] [blame] | 1106 | unsigned long size, struct fsck_options *options) |
| 1107 | { |
Patrick Steinhardt | bb3a926 | 2022-12-01 15:45:57 +0100 | [diff] [blame] | 1108 | int ret = 0; |
Jeff King | ed8b10f | 2018-05-02 17:25:27 -0400 | [diff] [blame] | 1109 | |
Jeff King | 6da40b2 | 2019-10-18 00:59:29 -0400 | [diff] [blame] | 1110 | if (object_on_skiplist(options, oid)) |
Ramsay Jones | fb16287 | 2018-06-27 19:39:53 +0100 | [diff] [blame] | 1111 | return 0; |
| 1112 | |
Patrick Steinhardt | bb3a926 | 2022-12-01 15:45:57 +0100 | [diff] [blame] | 1113 | if (oidset_contains(&options->gitmodules_found, oid)) { |
| 1114 | struct config_options config_opts = { 0 }; |
| 1115 | struct fsck_gitmodules_data data; |
| 1116 | |
| 1117 | oidset_insert(&options->gitmodules_done, oid); |
| 1118 | |
| 1119 | if (!buf) { |
| 1120 | /* |
| 1121 | * A missing buffer here is a sign that the caller found the |
| 1122 | * blob too gigantic to load into memory. Let's just consider |
| 1123 | * that an error. |
| 1124 | */ |
| 1125 | return report(options, oid, OBJ_BLOB, |
| 1126 | FSCK_MSG_GITMODULES_LARGE, |
| 1127 | ".gitmodules too large to parse"); |
| 1128 | } |
| 1129 | |
| 1130 | data.oid = oid; |
| 1131 | data.options = options; |
| 1132 | data.ret = 0; |
| 1133 | config_opts.error_action = CONFIG_ERROR_SILENT; |
| 1134 | if (git_config_from_mem(fsck_gitmodules_fn, CONFIG_ORIGIN_BLOB, |
Glen Choo | 809d868 | 2023-06-28 19:26:24 +0000 | [diff] [blame] | 1135 | ".gitmodules", buf, size, &data, |
| 1136 | CONFIG_SCOPE_UNKNOWN, &config_opts)) |
Patrick Steinhardt | bb3a926 | 2022-12-01 15:45:57 +0100 | [diff] [blame] | 1137 | data.ret |= report(options, oid, OBJ_BLOB, |
| 1138 | FSCK_MSG_GITMODULES_PARSE, |
| 1139 | "could not parse gitmodules blob"); |
| 1140 | ret |= data.ret; |
Jeff King | ed8b10f | 2018-05-02 17:25:27 -0400 | [diff] [blame] | 1141 | } |
| 1142 | |
Patrick Steinhardt | 27ab478 | 2022-12-01 15:46:09 +0100 | [diff] [blame] | 1143 | if (oidset_contains(&options->gitattributes_found, oid)) { |
| 1144 | const char *ptr; |
Jeff King | ed8b10f | 2018-05-02 17:25:27 -0400 | [diff] [blame] | 1145 | |
Patrick Steinhardt | 27ab478 | 2022-12-01 15:46:09 +0100 | [diff] [blame] | 1146 | oidset_insert(&options->gitattributes_done, oid); |
| 1147 | |
| 1148 | if (!buf || size > ATTR_MAX_FILE_SIZE) { |
| 1149 | /* |
| 1150 | * A missing buffer here is a sign that the caller found the |
| 1151 | * blob too gigantic to load into memory. Let's just consider |
| 1152 | * that an error. |
| 1153 | */ |
| 1154 | return report(options, oid, OBJ_BLOB, |
| 1155 | FSCK_MSG_GITATTRIBUTES_LARGE, |
| 1156 | ".gitattributes too large to parse"); |
| 1157 | } |
| 1158 | |
| 1159 | for (ptr = buf; *ptr; ) { |
| 1160 | const char *eol = strchrnul(ptr, '\n'); |
| 1161 | if (eol - ptr >= ATTR_MAX_LINE_LENGTH) { |
| 1162 | ret |= report(options, oid, OBJ_BLOB, |
| 1163 | FSCK_MSG_GITATTRIBUTES_LINE_LENGTH, |
| 1164 | ".gitattributes has too long lines to parse"); |
| 1165 | break; |
| 1166 | } |
| 1167 | |
| 1168 | ptr = *eol ? eol + 1 : eol; |
| 1169 | } |
| 1170 | } |
| 1171 | |
Patrick Steinhardt | bb3a926 | 2022-12-01 15:45:57 +0100 | [diff] [blame] | 1172 | return ret; |
Jeff King | 7ac4f3a | 2018-05-02 15:44:51 -0400 | [diff] [blame] | 1173 | } |
| 1174 | |
Johannes Schindelin | 90a398b | 2014-09-10 15:52:51 +0200 | [diff] [blame] | 1175 | int fsck_object(struct object *obj, void *data, unsigned long size, |
Johannes Schindelin | 2241054 | 2015-06-22 17:25:00 +0200 | [diff] [blame] | 1176 | struct fsck_options *options) |
Martin Koegler | ba002f3 | 2008-02-25 22:46:08 +0100 | [diff] [blame] | 1177 | { |
| 1178 | if (!obj) |
Jeff King | 3837025 | 2019-10-18 00:59:15 -0400 | [diff] [blame] | 1179 | return report(options, NULL, OBJ_NONE, FSCK_MSG_BAD_OBJECT_SHA1, "no valid object to fsck"); |
Martin Koegler | ba002f3 | 2008-02-25 22:46:08 +0100 | [diff] [blame] | 1180 | |
Jeff King | 35ff327 | 2023-01-18 15:43:53 -0500 | [diff] [blame] | 1181 | return fsck_buffer(&obj->oid, obj->type, data, size, options); |
| 1182 | } |
Martin Koegler | ba002f3 | 2008-02-25 22:46:08 +0100 | [diff] [blame] | 1183 | |
Jeff King | 35ff327 | 2023-01-18 15:43:53 -0500 | [diff] [blame] | 1184 | int fsck_buffer(const struct object_id *oid, enum object_type type, |
Patrick Steinhardt | b567004 | 2024-06-07 08:37:39 +0200 | [diff] [blame] | 1185 | const void *data, unsigned long size, |
Jeff King | 35ff327 | 2023-01-18 15:43:53 -0500 | [diff] [blame] | 1186 | struct fsck_options *options) |
| 1187 | { |
| 1188 | if (type == OBJ_BLOB) |
| 1189 | return fsck_blob(oid, data, size, options); |
| 1190 | if (type == OBJ_TREE) |
| 1191 | return fsck_tree(oid, data, size, options); |
| 1192 | if (type == OBJ_COMMIT) |
| 1193 | return fsck_commit(oid, data, size, options); |
| 1194 | if (type == OBJ_TAG) |
| 1195 | return fsck_tag(oid, data, size, options); |
| 1196 | |
| 1197 | return report(options, oid, type, |
Jeff King | 3837025 | 2019-10-18 00:59:15 -0400 | [diff] [blame] | 1198 | FSCK_MSG_UNKNOWN_TYPE, |
| 1199 | "unknown type '%d' (internal fsck error)", |
Jeff King | 35ff327 | 2023-01-18 15:43:53 -0500 | [diff] [blame] | 1200 | type); |
Martin Koegler | ba002f3 | 2008-02-25 22:46:08 +0100 | [diff] [blame] | 1201 | } |
Martin Koegler | d6ffc8d | 2008-02-25 22:46:09 +0100 | [diff] [blame] | 1202 | |
Johannes Schindelin | 1cd772c | 2016-07-17 12:59:57 +0200 | [diff] [blame] | 1203 | int fsck_error_function(struct fsck_options *o, |
Jeff King | 5afc4b1 | 2019-10-18 00:58:40 -0400 | [diff] [blame] | 1204 | const struct object_id *oid, |
Jeff King | 0b4e901 | 2023-07-03 02:44:18 -0400 | [diff] [blame] | 1205 | enum object_type object_type UNUSED, |
Ævar Arnfjörð Bjarmason | 394d5d3 | 2021-03-28 15:15:46 +0200 | [diff] [blame] | 1206 | enum fsck_msg_type msg_type, |
Jeff King | 0b4e901 | 2023-07-03 02:44:18 -0400 | [diff] [blame] | 1207 | enum fsck_msg_id msg_id UNUSED, |
Ævar Arnfjörð Bjarmason | 394d5d3 | 2021-03-28 15:15:46 +0200 | [diff] [blame] | 1208 | const char *message) |
Martin Koegler | d6ffc8d | 2008-02-25 22:46:09 +0100 | [diff] [blame] | 1209 | { |
Johannes Schindelin | 0282f4d | 2015-06-22 17:25:25 +0200 | [diff] [blame] | 1210 | if (msg_type == FSCK_WARN) { |
Jeff King | 5afc4b1 | 2019-10-18 00:58:40 -0400 | [diff] [blame] | 1211 | warning("object %s: %s", fsck_describe_object(o, oid), message); |
Johannes Schindelin | 0282f4d | 2015-06-22 17:25:25 +0200 | [diff] [blame] | 1212 | return 0; |
| 1213 | } |
Jeff King | 5afc4b1 | 2019-10-18 00:58:40 -0400 | [diff] [blame] | 1214 | error("object %s: %s", fsck_describe_object(o, oid), message); |
Martin Koegler | d6ffc8d | 2008-02-25 22:46:09 +0100 | [diff] [blame] | 1215 | return 1; |
| 1216 | } |
Jeff King | 159e7b0 | 2018-05-02 17:20:08 -0400 | [diff] [blame] | 1217 | |
Patrick Steinhardt | a59a8c6 | 2022-12-01 15:46:01 +0100 | [diff] [blame] | 1218 | static int fsck_blobs(struct oidset *blobs_found, struct oidset *blobs_done, |
| 1219 | enum fsck_msg_id msg_missing, enum fsck_msg_id msg_type, |
| 1220 | struct fsck_options *options, const char *blob_type) |
Jeff King | 159e7b0 | 2018-05-02 17:20:08 -0400 | [diff] [blame] | 1221 | { |
| 1222 | int ret = 0; |
| 1223 | struct oidset_iter iter; |
| 1224 | const struct object_id *oid; |
| 1225 | |
Patrick Steinhardt | a59a8c6 | 2022-12-01 15:46:01 +0100 | [diff] [blame] | 1226 | oidset_iter_init(blobs_found, &iter); |
Jeff King | 159e7b0 | 2018-05-02 17:20:08 -0400 | [diff] [blame] | 1227 | while ((oid = oidset_iter_next(&iter))) { |
Jeff King | 159e7b0 | 2018-05-02 17:20:08 -0400 | [diff] [blame] | 1228 | enum object_type type; |
| 1229 | unsigned long size; |
| 1230 | char *buf; |
| 1231 | |
Patrick Steinhardt | a59a8c6 | 2022-12-01 15:46:01 +0100 | [diff] [blame] | 1232 | if (oidset_contains(blobs_done, oid)) |
Jeff King | 159e7b0 | 2018-05-02 17:20:08 -0400 | [diff] [blame] | 1233 | continue; |
| 1234 | |
Ævar Arnfjörð Bjarmason | bc726bd | 2023-03-28 15:58:50 +0200 | [diff] [blame] | 1235 | buf = repo_read_object_file(the_repository, oid, &type, &size); |
Jeff King | 159e7b0 | 2018-05-02 17:20:08 -0400 | [diff] [blame] | 1236 | if (!buf) { |
Jeff King | b8b00f1 | 2019-10-18 00:59:54 -0400 | [diff] [blame] | 1237 | if (is_promisor_object(oid)) |
Jeff King | 2738744 | 2018-05-14 12:22:48 -0400 | [diff] [blame] | 1238 | continue; |
Jeff King | 3837025 | 2019-10-18 00:59:15 -0400 | [diff] [blame] | 1239 | ret |= report(options, |
Patrick Steinhardt | a59a8c6 | 2022-12-01 15:46:01 +0100 | [diff] [blame] | 1240 | oid, OBJ_BLOB, msg_missing, |
| 1241 | "unable to read %s blob", blob_type); |
Jeff King | 159e7b0 | 2018-05-02 17:20:08 -0400 | [diff] [blame] | 1242 | continue; |
| 1243 | } |
| 1244 | |
| 1245 | if (type == OBJ_BLOB) |
Jeff King | b8b00f1 | 2019-10-18 00:59:54 -0400 | [diff] [blame] | 1246 | ret |= fsck_blob(oid, buf, size, options); |
Jeff King | 159e7b0 | 2018-05-02 17:20:08 -0400 | [diff] [blame] | 1247 | else |
Patrick Steinhardt | a59a8c6 | 2022-12-01 15:46:01 +0100 | [diff] [blame] | 1248 | ret |= report(options, oid, type, msg_type, |
| 1249 | "non-blob found at %s", blob_type); |
Jeff King | 159e7b0 | 2018-05-02 17:20:08 -0400 | [diff] [blame] | 1250 | free(buf); |
| 1251 | } |
| 1252 | |
Patrick Steinhardt | a59a8c6 | 2022-12-01 15:46:01 +0100 | [diff] [blame] | 1253 | oidset_clear(blobs_found); |
| 1254 | oidset_clear(blobs_done); |
Jeff King | 159e7b0 | 2018-05-02 17:20:08 -0400 | [diff] [blame] | 1255 | |
Patrick Steinhardt | a59a8c6 | 2022-12-01 15:46:01 +0100 | [diff] [blame] | 1256 | return ret; |
| 1257 | } |
| 1258 | |
| 1259 | int fsck_finish(struct fsck_options *options) |
| 1260 | { |
| 1261 | int ret = 0; |
| 1262 | |
| 1263 | ret |= fsck_blobs(&options->gitmodules_found, &options->gitmodules_done, |
| 1264 | FSCK_MSG_GITMODULES_MISSING, FSCK_MSG_GITMODULES_BLOB, |
| 1265 | options, ".gitmodules"); |
Patrick Steinhardt | 27ab478 | 2022-12-01 15:46:09 +0100 | [diff] [blame] | 1266 | ret |= fsck_blobs(&options->gitattributes_found, &options->gitattributes_done, |
| 1267 | FSCK_MSG_GITATTRIBUTES_MISSING, FSCK_MSG_GITATTRIBUTES_BLOB, |
| 1268 | options, ".gitattributes"); |
Patrick Steinhardt | a59a8c6 | 2022-12-01 15:46:01 +0100 | [diff] [blame] | 1269 | |
Jeff King | 159e7b0 | 2018-05-02 17:20:08 -0400 | [diff] [blame] | 1270 | return ret; |
| 1271 | } |
Ævar Arnfjörð Bjarmason | 1f3299f | 2021-01-05 20:42:47 +0100 | [diff] [blame] | 1272 | |
Glen Choo | a4e7e31 | 2023-06-28 19:26:22 +0000 | [diff] [blame] | 1273 | int git_fsck_config(const char *var, const char *value, |
| 1274 | const struct config_context *ctx, void *cb) |
Ævar Arnfjörð Bjarmason | 1f3299f | 2021-01-05 20:42:47 +0100 | [diff] [blame] | 1275 | { |
Ævar Arnfjörð Bjarmason | fb79f5b | 2021-03-17 19:20:36 +0100 | [diff] [blame] | 1276 | struct fsck_options *options = cb; |
Jeff King | d49cb16 | 2023-12-07 02:11:35 -0500 | [diff] [blame] | 1277 | const char *msg_id; |
| 1278 | |
Ævar Arnfjörð Bjarmason | 1f3299f | 2021-01-05 20:42:47 +0100 | [diff] [blame] | 1279 | if (strcmp(var, "fsck.skiplist") == 0) { |
Patrick Steinhardt | 6073b3b | 2024-05-27 13:46:15 +0200 | [diff] [blame] | 1280 | char *path; |
Ævar Arnfjörð Bjarmason | 1f3299f | 2021-01-05 20:42:47 +0100 | [diff] [blame] | 1281 | struct strbuf sb = STRBUF_INIT; |
| 1282 | |
| 1283 | if (git_config_pathname(&path, var, value)) |
| 1284 | return 1; |
| 1285 | strbuf_addf(&sb, "skiplist=%s", path); |
Patrick Steinhardt | 6073b3b | 2024-05-27 13:46:15 +0200 | [diff] [blame] | 1286 | free(path); |
Ævar Arnfjörð Bjarmason | 1f3299f | 2021-01-05 20:42:47 +0100 | [diff] [blame] | 1287 | fsck_set_msg_types(options, sb.buf); |
| 1288 | strbuf_release(&sb); |
| 1289 | return 0; |
| 1290 | } |
| 1291 | |
Jeff King | d49cb16 | 2023-12-07 02:11:35 -0500 | [diff] [blame] | 1292 | if (skip_prefix(var, "fsck.", &msg_id)) { |
| 1293 | if (!value) |
| 1294 | return config_error_nonbool(var); |
| 1295 | fsck_set_msg_type(options, msg_id, value); |
Ævar Arnfjörð Bjarmason | 1f3299f | 2021-01-05 20:42:47 +0100 | [diff] [blame] | 1296 | return 0; |
| 1297 | } |
| 1298 | |
Glen Choo | a4e7e31 | 2023-06-28 19:26:22 +0000 | [diff] [blame] | 1299 | return git_default_config(var, value, ctx, cb); |
Ævar Arnfjörð Bjarmason | 1f3299f | 2021-01-05 20:42:47 +0100 | [diff] [blame] | 1300 | } |
Ævar Arnfjörð Bjarmason | 3745e26 | 2021-03-28 15:15:51 +0200 | [diff] [blame] | 1301 | |
| 1302 | /* |
| 1303 | * Custom error callbacks that are used in more than one place. |
| 1304 | */ |
| 1305 | |
| 1306 | int fsck_error_cb_print_missing_gitmodules(struct fsck_options *o, |
| 1307 | const struct object_id *oid, |
| 1308 | enum object_type object_type, |
| 1309 | enum fsck_msg_type msg_type, |
| 1310 | enum fsck_msg_id msg_id, |
| 1311 | const char *message) |
| 1312 | { |
| 1313 | if (msg_id == FSCK_MSG_GITMODULES_MISSING) { |
| 1314 | puts(oid_to_hex(oid)); |
| 1315 | return 0; |
| 1316 | } |
| 1317 | return fsck_error_function(o, oid, object_type, msg_type, msg_id, message); |
| 1318 | } |