1 /*
2  * Copyright (c) 2018, 2019, 2020 Stefan Sperling <stsp@openbsd.org>
3  *
4  * Permission to use, copy, modify, and distribute this software for any
5  * purpose with or without fee is hereby granted, provided that the above
6  * copyright notice and this permission notice appear in all copies.
7  *
8  * THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES
9  * WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF
10  * MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR
11  * ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES
12  * WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN
13  * ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
14  * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
15  */
16 
17 #include <sys/types.h>
18 #include <sys/stat.h>
19 #include <sys/queue.h>
20 
21 #include <limits.h>
22 #include <stdio.h>
23 #include <stdlib.h>
24 #include <string.h>
25 #include <sha1.h>
26 #include <sha2.h>
27 #include <zlib.h>
28 #include <ctype.h>
29 
30 #include "got_compat.h"
31 
32 #include "got_error.h"
33 #include "got_object.h"
34 #include "got_cancel.h"
35 #include "got_commit_graph.h"
36 #include "got_path.h"
37 
38 #include "got_lib_delta.h"
39 #include "got_lib_inflate.h"
40 #include "got_lib_object.h"
41 #include "got_lib_object_idset.h"
42 
43 struct got_commit_graph_node {
44 	struct got_object_id id;
45 
46 	/* Used only during iteration. */
47 	time_t timestamp;
48 	TAILQ_ENTRY(got_commit_graph_node) entry;
49 };
50 
51 TAILQ_HEAD(got_commit_graph_iter_list, got_commit_graph_node);
52 
53 struct got_commit_graph_branch_tip {
54 	struct got_object_id *commit_id;
55 	struct got_commit_object *commit;
56 	struct got_commit_graph_node *new_node;
57 };
58 
59 struct got_commit_graph {
60 	/* The set of all commits we have traversed. */
61 	struct got_object_idset *node_ids;
62 
63 	int flags;
64 #define GOT_COMMIT_GRAPH_FIRST_PARENT_TRAVERSAL		0x01
65 
66 	/*
67 	 * A set of object IDs of known parent commits which we have not yet
68 	 * traversed. Each commit ID in this set represents a branch in commit
69 	 * history: Either the first-parent branch of the head node, or another
70 	 * branch corresponding to a traversed merge commit for which we have
71 	 * not traversed a branch point commit yet.
72 	 *
73 	 * Whenever we add a commit with a matching ID to the graph, we remove
74 	 * its corresponding element from this set, and add new elements for
75 	 * each of that commit's parent commits which were not traversed yet.
76 	 *
77 	 * When API users ask us to fetch more commits, we fetch commits from
78 	 * all currently open branches. This allows API users to process
79 	 * commits in linear order even though the history contains branches.
80 	 */
81 	struct got_object_idset *open_branches;
82 
83 	/* Array of branch tips for fetch_commits_from_open_branches(). */
84 	struct got_commit_graph_branch_tip *tips;
85 	int ntips;
86 
87 	/* Path of tree entry of interest to the API user. */
88 	char *path;
89 
90 	/*
91 	 * Nodes which will be passed to the API user next, sorted by
92 	 * commit timestmap.
93 	 */
94 	struct got_commit_graph_iter_list iter_list;
95 };
96 
97 static const struct got_error *
98 detect_changed_path(int *changed, struct got_commit_object *commit,
99     struct got_object_id *commit_id, const char *path,
100     struct got_repository *repo)
101 {
102 	const struct got_error *err = NULL;
103 	struct got_commit_object *pcommit = NULL;
104 	struct got_tree_object *tree = NULL, *ptree = NULL;
105 	struct got_object_qid *pid;
106 
107 	if (got_path_is_root_dir(path)) {
108 		*changed = 1;
109 		return NULL;
110 	}
111 
112 	*changed = 0;
113 
114 	pid = STAILQ_FIRST(&commit->parent_ids);
115 	if (pid == NULL) {
116 		struct got_object_id *obj_id;
117 		err = got_object_id_by_path(&obj_id, repo, commit, path);
118 		if (err) {
119 			if (err->code == GOT_ERR_NO_TREE_ENTRY)
120 				err = NULL;
121 		} else
122 			*changed = 1; /* The path was created in this commit. */
123 		free(obj_id);
124 		return err;
125 	}
126 
127 	err = got_object_open_as_tree(&tree, repo, commit->tree_id);
128 	if (err)
129 		return err;
130 
131 	err = got_object_open_as_commit(&pcommit, repo, &pid->id);
132 	if (err)
133 		goto done;
134 
135 	err = got_object_open_as_tree(&ptree, repo, pcommit->tree_id);
136 	if (err)
137 		goto done;
138 
139 	err = got_object_tree_path_changed(changed, tree, ptree, path, repo);
140 done:
141 	if (tree)
142 		got_object_tree_close(tree);
143 	if (ptree)
144 		got_object_tree_close(ptree);
145 	if (pcommit)
146 		got_object_commit_close(pcommit);
147 	return err;
148 }
149 
150 static void
151 add_node_to_iter_list(struct got_commit_graph *graph,
152     struct got_commit_graph_node *node, time_t committer_time)
153 {
154 	struct got_commit_graph_node *n, *next;
155 
156 	node->timestamp = committer_time;
157 
158 	n = TAILQ_FIRST(&graph->iter_list);
159 	while (n) {
160 		next = TAILQ_NEXT(n, entry);
161 		if (next && node->timestamp >= next->timestamp) {
162 			TAILQ_INSERT_BEFORE(next, node, entry);
163 			return;
164 		}
165 		n = next;
166 	}
167 	TAILQ_INSERT_TAIL(&graph->iter_list, node, entry);
168 }
169 
170 static const struct got_error *
171 add_node(struct got_commit_graph_node **new_node,
172     struct got_commit_graph *graph, struct got_object_id *commit_id,
173     struct got_repository *repo)
174 {
175 	const struct got_error *err = NULL;
176 	struct got_commit_graph_node *node;
177 
178 	*new_node = NULL;
179 
180 	node = calloc(1, sizeof(*node));
181 	if (node == NULL)
182 		return got_error_from_errno("calloc");
183 
184 	memcpy(&node->id, commit_id, sizeof(node->id));
185 	err = got_object_idset_add(graph->node_ids, &node->id, NULL);
186 	if (err)
187 		free(node);
188 	else
189 		*new_node = node;
190 	return err;
191 }
192 
193 /*
194  * Ask got-read-pack to traverse first-parent history until a commit is
195  * encountered which modified graph->path, or until the pack file runs
196  * out of relevant commits. This is faster than sending an individual
197  * request for each commit stored in the pack file.
198  */
199 static const struct got_error *
200 packed_first_parent_traversal(int *ncommits_traversed,
201     struct got_commit_graph *graph, struct got_object_id *commit_id,
202     struct got_repository *repo)
203 {
204 	const struct got_error *err = NULL;
205 	struct got_object_id_queue traversed_commits;
206 	struct got_object_qid *qid;
207 
208 	STAILQ_INIT(&traversed_commits);
209 	*ncommits_traversed = 0;
210 
211 	err = got_traverse_packed_commits(&traversed_commits,
212 	    commit_id, graph->path, repo);
213 	if (err)
214 		return err;
215 
216 	/* Add all traversed commits to the graph... */
217 	STAILQ_FOREACH(qid, &traversed_commits, entry) {
218 		if (got_object_idset_contains(graph->open_branches, &qid->id))
219 			continue;
220 		if (got_object_idset_contains(graph->node_ids, &qid->id))
221 			continue;
222 
223 		(*ncommits_traversed)++;
224 
225 		/* ... except the last commit is the new branch tip. */
226 		if (STAILQ_NEXT(qid, entry) == NULL) {
227 			err = got_object_idset_add(graph->open_branches,
228 			    &qid->id, NULL);
229 			break;
230 		}
231 
232 		err = got_object_idset_add(graph->node_ids, &qid->id, NULL);
233 		if (err)
234 			break;
235 	}
236 
237 	got_object_id_queue_free(&traversed_commits);
238 	return err;
239 }
240 
241 static const struct got_error *
242 close_branch(struct got_commit_graph *graph, struct got_object_id *commit_id)
243 {
244 	const struct got_error *err;
245 
246 	err = got_object_idset_remove(NULL, graph->open_branches, commit_id);
247 	if (err && err->code != GOT_ERR_NO_OBJ)
248 		return err;
249 	return NULL;
250 }
251 
252 static const struct got_error *
253 advance_branch(struct got_commit_graph *graph, struct got_object_id *commit_id,
254     struct got_commit_object *commit, struct got_repository *repo)
255 {
256 	const struct got_error *err;
257 	struct got_object_qid *qid;
258 
259 	err = close_branch(graph, commit_id);
260 	if (err)
261 		return err;
262 
263 	if (graph->flags & GOT_COMMIT_GRAPH_FIRST_PARENT_TRAVERSAL) {
264 		qid = STAILQ_FIRST(&commit->parent_ids);
265 		if (qid == NULL ||
266 		    got_object_idset_contains(graph->open_branches, &qid->id))
267 			return NULL;
268 		/*
269 		 * The root directory always changes by definition, and when
270 		 * logging the root we want to traverse consecutive commits
271 		 * even if they point at the same tree.
272 		 * But if we are looking for a specific path then we can avoid
273 		 * fetching packed commits which did not modify the path and
274 		 * only fetch their IDs. This speeds up 'got blame'.
275 		 */
276 		if (!got_path_is_root_dir(graph->path) &&
277 		    (commit->flags & GOT_COMMIT_FLAG_PACKED)) {
278 			int ncommits = 0;
279 			err = packed_first_parent_traversal(&ncommits,
280 			    graph, &qid->id, repo);
281 			if (err || ncommits > 0)
282 				return err;
283 		}
284 		return got_object_idset_add(graph->open_branches,
285 		    &qid->id, NULL);
286 	}
287 
288 	/*
289 	 * If we are graphing commits for a specific path, skip branches
290 	 * which do not contribute any content to this path.
291 	 */
292 	if (commit->nparents > 1 && !got_path_is_root_dir(graph->path)) {
293 		struct got_object_id *merged_id, *prev_id = NULL;
294 		int branches_differ = 0;
295 
296 		err = got_object_id_by_path(&merged_id, repo, commit,
297 		    graph->path);
298 		if (err)
299 			return err;
300 
301 		STAILQ_FOREACH(qid, &commit->parent_ids, entry) {
302 			struct got_object_id *id = NULL;
303 			struct got_commit_object *pcommit = NULL;
304 
305 			if (got_object_idset_contains(graph->open_branches,
306 			    &qid->id))
307 				continue;
308 
309 			err = got_object_open_as_commit(&pcommit, repo,
310 			    &qid->id);
311 			if (err) {
312 				free(merged_id);
313 				free(prev_id);
314 				return err;
315 			}
316 			err = got_object_id_by_path(&id, repo, pcommit,
317 			    graph->path);
318 			got_object_commit_close(pcommit);
319 			pcommit = NULL;
320 			if (err) {
321 				if (err->code == GOT_ERR_NO_TREE_ENTRY) {
322 					branches_differ = 1;
323 					continue;
324 				}
325 				free(merged_id);
326 				free(prev_id);
327 				return err;
328 			}
329 
330 			if (prev_id) {
331 				if (!branches_differ &&
332 				    got_object_id_cmp(id, prev_id) != 0)
333 					branches_differ = 1;
334 				free(prev_id);
335 			}
336 			prev_id = id;
337 
338 			/*
339 			 * If a branch has created the merged content we can
340 			 * skip any other branches.
341 			 */
342 			if (got_object_id_cmp(merged_id, id) == 0) {
343 				err = got_object_idset_add(graph->open_branches,
344 				    &qid->id, NULL);
345 				free(merged_id);
346 				free(id);
347 				return err;
348 			}
349 		}
350 
351 		free(prev_id);
352 		prev_id = NULL;
353 		free(merged_id);
354 		merged_id = NULL;
355 
356 		/*
357 		 * If the path's content is the same on all branches,
358 		 * follow the first parent only.
359 		 */
360 		if (!branches_differ) {
361 			qid = STAILQ_FIRST(&commit->parent_ids);
362 			if (qid == NULL)
363 				return NULL;
364 			if (got_object_idset_contains(graph->open_branches,
365 			    &qid->id))
366 				return NULL;
367 			if (got_object_idset_contains(graph->node_ids,
368 			    &qid->id))
369 				return NULL; /* parent already traversed */
370 			return got_object_idset_add(graph->open_branches,
371 			    &qid->id, NULL);
372 		}
373 	}
374 
375 	STAILQ_FOREACH(qid, &commit->parent_ids, entry) {
376 		if (got_object_idset_contains(graph->open_branches, &qid->id))
377 			continue;
378 		if (got_object_idset_contains(graph->node_ids, &qid->id))
379 			continue; /* parent already traversed */
380 		err = got_object_idset_add(graph->open_branches, &qid->id,
381 		    NULL);
382 		if (err)
383 			return err;
384 	}
385 
386 	return NULL;
387 }
388 
389 const struct got_error *
390 got_commit_graph_open(struct got_commit_graph **graph,
391     const char *path, int first_parent_traversal)
392 {
393 	const struct got_error *err = NULL;
394 
395 	*graph = calloc(1, sizeof(**graph));
396 	if (*graph == NULL)
397 		return got_error_from_errno("calloc");
398 
399 	TAILQ_INIT(&(*graph)->iter_list);
400 
401 	(*graph)->path = strdup(path);
402 	if ((*graph)->path == NULL) {
403 		err = got_error_from_errno("strdup");
404 		goto done;
405 	}
406 
407 	(*graph)->node_ids = got_object_idset_alloc();
408 	if ((*graph)->node_ids == NULL) {
409 		err = got_error_from_errno("got_object_idset_alloc");
410 		goto done;
411 	}
412 
413 	(*graph)->open_branches = got_object_idset_alloc();
414 	if ((*graph)->open_branches == NULL) {
415 		err = got_error_from_errno("got_object_idset_alloc");
416 		goto done;
417 	}
418 
419 	if (first_parent_traversal)
420 		(*graph)->flags |= GOT_COMMIT_GRAPH_FIRST_PARENT_TRAVERSAL;
421 done:
422 	if (err) {
423 		got_commit_graph_close(*graph);
424 		*graph = NULL;
425 	}
426 	return err;
427 }
428 
429 struct add_branch_tip_arg {
430 	struct got_commit_graph_branch_tip *tips;
431 	int ntips;
432 	struct got_repository *repo;
433 	struct got_commit_graph *graph;
434 };
435 
436 static const struct got_error *
437 add_branch_tip(struct got_object_id *commit_id, void *data, void *arg)
438 {
439 	const struct got_error *err;
440 	struct add_branch_tip_arg *a = arg;
441 	struct got_commit_graph_node *new_node;
442 	struct got_commit_object *commit;
443 
444 	err = got_object_open_as_commit(&commit, a->repo, commit_id);
445 	if (err)
446 		return err;
447 
448 	err = add_node(&new_node, a->graph, commit_id, a->repo);
449 	if (err) {
450 		got_object_commit_close(commit);
451 		return err;
452 	}
453 
454 	a->tips[a->ntips].commit_id = &new_node->id;
455 	a->tips[a->ntips].commit = commit;
456 	a->tips[a->ntips].new_node = new_node;
457 	a->ntips++;
458 
459 	return NULL;
460 }
461 
462 static const struct got_error *
463 fetch_commits_from_open_branches(struct got_commit_graph *graph,
464     struct got_repository *repo, got_cancel_cb cancel_cb, void *cancel_arg)
465 {
466 	const struct got_error *err;
467 	struct add_branch_tip_arg arg;
468 	int i, ntips;
469 
470 	ntips = got_object_idset_num_elements(graph->open_branches);
471 	if (ntips == 0)
472 		return NULL;
473 
474 	/* (Re-)allocate branch tips array if necessary. */
475 	if (graph->ntips < ntips) {
476 		struct got_commit_graph_branch_tip *tips;
477 		tips = recallocarray(graph->tips, graph->ntips, ntips,
478 		    sizeof(*tips));
479 		if (tips == NULL)
480 			return got_error_from_errno("recallocarray");
481 		graph->tips = tips;
482 		graph->ntips = ntips;
483 	}
484 	arg.tips = graph->tips;
485 	arg.ntips = 0; /* add_branch_tip() will increment */
486 	arg.repo = repo;
487 	arg.graph = graph;
488 	err = got_object_idset_for_each(graph->open_branches, add_branch_tip,
489 	    &arg);
490 	if (err)
491 		goto done;
492 
493 	for (i = 0; i < arg.ntips; i++) {
494 		struct got_object_id *commit_id;
495 		struct got_commit_object *commit;
496 		struct got_commit_graph_node *new_node;
497 		int changed;
498 
499 		if (cancel_cb) {
500 			err = (*cancel_cb)(cancel_arg);
501 			if (err)
502 				break;
503 		}
504 
505 		commit_id = arg.tips[i].commit_id;
506 		commit = arg.tips[i].commit;
507 		new_node = arg.tips[i].new_node;
508 
509 		err = detect_changed_path(&changed, commit, commit_id,
510 		    graph->path, repo);
511 		if (err) {
512 			if (err->code != GOT_ERR_NO_OBJ)
513 				break;
514 			/*
515 			 * History of the path stops here on the current
516 			 * branch. Keep going on other branches.
517 			 */
518 			err = close_branch(graph, commit_id);
519 			if (err)
520 				break;
521 			continue;
522 		}
523 		if (changed) {
524 			add_node_to_iter_list(graph, new_node,
525 			    got_object_commit_get_committer_time(commit));
526 			arg.tips[i].new_node = NULL;
527 		}
528 		err = advance_branch(graph, commit_id, commit, repo);
529 		if (err)
530 			break;
531 	}
532 done:
533 	for (i = 0; i < arg.ntips; i++) {
534 		got_object_commit_close(arg.tips[i].commit);
535 		free(arg.tips[i].new_node);
536 	}
537 	return err;
538 }
539 
540 void
541 got_commit_graph_close(struct got_commit_graph *graph)
542 {
543 	struct got_commit_graph_node *node;
544 
545 	while ((node = TAILQ_FIRST(&graph->iter_list))) {
546 		TAILQ_REMOVE(&graph->iter_list, node, entry);
547 		free(node);
548 	}
549 
550 	if (graph->open_branches)
551 		got_object_idset_free(graph->open_branches);
552 	if (graph->node_ids)
553 		got_object_idset_free(graph->node_ids);
554 	free(graph->tips);
555 	free(graph->path);
556 	free(graph);
557 }
558 
559 const struct got_error *
560 got_commit_graph_iter_start(struct got_commit_graph *graph,
561     struct got_object_id *id, struct got_repository *repo,
562     got_cancel_cb cancel_cb, void *cancel_arg)
563 {
564 	const struct got_error *err = NULL;
565 
566 	if (!TAILQ_EMPTY(&graph->iter_list))
567 		return got_error(GOT_ERR_ITER_BUSY);
568 
569 	err = got_object_idset_add(graph->open_branches, id, NULL);
570 	if (err)
571 		return err;
572 
573 	/* Locate first commit which changed graph->path. */
574 	while (TAILQ_EMPTY(&graph->iter_list) &&
575 	    got_object_idset_num_elements(graph->open_branches) > 0) {
576 		err = fetch_commits_from_open_branches(graph, repo,
577 		    cancel_cb, cancel_arg);
578 		if (err)
579 			return err;
580 	}
581 
582 	if (TAILQ_EMPTY(&graph->iter_list)) {
583 		const char *path;
584 		if (got_path_is_root_dir(graph->path))
585 			return got_error_no_obj(id);
586 		path = graph->path;
587 		while (path[0] == '/')
588 			path++;
589 		return got_error_path(path, GOT_ERR_NO_TREE_ENTRY);
590 	}
591 
592 	return NULL;
593 }
594 
595 const struct got_error *
596 got_commit_graph_iter_next(struct got_object_id *id,
597     struct got_commit_graph *graph, struct got_repository *repo,
598     got_cancel_cb cancel_cb, void *cancel_arg)
599 {
600 	const struct got_error *err = NULL;
601 	struct got_commit_graph_node *node;
602 
603 	node = TAILQ_FIRST(&graph->iter_list);
604 	if (node == NULL) {
605 		/* We are done iterating, or iteration was not started. */
606 		return got_error(GOT_ERR_ITER_COMPLETED);
607 	}
608 
609 	while (TAILQ_NEXT(node, entry) == NULL &&
610 	    got_object_idset_num_elements(graph->open_branches) > 0) {
611 		err = fetch_commits_from_open_branches(graph, repo,
612 		    cancel_cb, cancel_arg);
613 		if (err)
614 			return err;
615 	}
616 
617 	memcpy(id, &node->id, sizeof(*id));
618 
619 	TAILQ_REMOVE(&graph->iter_list, node, entry);
620 	free(node);
621 	return NULL;
622 }
623 
624 static const struct got_error *
625 find_yca_add_id(struct got_object_id **yca_id, struct got_commit_graph *graph,
626     struct got_object_idset *commit_ids, struct got_repository *repo,
627     got_cancel_cb cancel_cb, void *cancel_arg)
628 {
629 	const struct got_error *err = NULL;
630 	struct got_object_id id;
631 
632 	err = got_commit_graph_iter_next(&id, graph, repo, cancel_cb,
633 	    cancel_arg);
634 	if (err)
635 		return err;
636 
637 	if (got_object_idset_contains(commit_ids, &id)) {
638 		*yca_id = got_object_id_dup(&id);
639 		if (*yca_id == NULL)
640 			err = got_error_from_errno("got_object_id_dup");
641 		return err;
642 	}
643 
644 	return got_object_idset_add(commit_ids, &id, NULL);
645 }
646 
647 const struct got_error *
648 got_commit_graph_find_youngest_common_ancestor(struct got_object_id **yca_id,
649     struct got_object_id *commit_id, struct got_object_id *commit_id2,
650     int first_parent_traversal,
651     struct got_repository *repo, got_cancel_cb cancel_cb, void *cancel_arg)
652 {
653 	const struct got_error *err = NULL;
654 	struct got_commit_graph *graph = NULL, *graph2 = NULL;
655 	int completed = 0, completed2 = 0;
656 	struct got_object_idset *commit_ids;
657 
658 	*yca_id = NULL;
659 
660 	commit_ids = got_object_idset_alloc();
661 	if (commit_ids == NULL)
662 		return got_error_from_errno("got_object_idset_alloc");
663 
664 	err = got_commit_graph_open(&graph, "/", first_parent_traversal);
665 	if (err)
666 		goto done;
667 
668 	err = got_commit_graph_open(&graph2, "/", first_parent_traversal);
669 	if (err)
670 		goto done;
671 
672 	err = got_commit_graph_iter_start(graph, commit_id, repo,
673 	    cancel_cb, cancel_arg);
674 	if (err)
675 		goto done;
676 
677 	err = got_commit_graph_iter_start(graph2, commit_id2, repo,
678 	    cancel_cb, cancel_arg);
679 	if (err)
680 		goto done;
681 
682 	for (;;) {
683 		if (cancel_cb) {
684 			err = (*cancel_cb)(cancel_arg);
685 			if (err)
686 				break;
687 		}
688 
689 		if (!completed) {
690 			err = find_yca_add_id(yca_id, graph, commit_ids, repo,
691 			    cancel_cb, cancel_arg);
692 			if (err) {
693 				if (err->code != GOT_ERR_ITER_COMPLETED)
694 					break;
695 				err = NULL;
696 				completed = 1;
697 			}
698 			if (*yca_id)
699 				break;
700 		}
701 
702 		if (!completed2) {
703 			err = find_yca_add_id(yca_id, graph2, commit_ids, repo,
704 			    cancel_cb, cancel_arg);
705 			if (err) {
706 				if (err->code != GOT_ERR_ITER_COMPLETED)
707 					break;
708 				err = NULL;
709 				completed2 = 1;
710 			}
711 			if (*yca_id)
712 				break;
713 		}
714 
715 		if (completed && completed2) {
716 			err = got_error(GOT_ERR_ANCESTRY);
717 			break;
718 		}
719 	}
720 done:
721 	got_object_idset_free(commit_ids);
722 	if (graph)
723 		got_commit_graph_close(graph);
724 	if (graph2)
725 		got_commit_graph_close(graph2);
726 	return err;
727 }