Fix handle leak in write_tree
[git/mingw/j6t.git] / builtin-write-tree.c
bloba1894814f7356e5689416560aa6cda868583ce2b
1 /*
2 * GIT - The information manager from hell
4 * Copyright (C) Linus Torvalds, 2005
5 */
6 #include "builtin.h"
7 #include "cache.h"
8 #include "tree.h"
9 #include "cache-tree.h"
11 static const char write_tree_usage[] =
12 "git-write-tree [--missing-ok] [--prefix=<prefix>/]";
14 int write_tree(unsigned char *sha1, int missing_ok, const char *prefix)
16 int entries, was_valid, newfd;
18 /* We can't free this memory, it becomes part of a linked list parsed atexit() */
19 struct lock_file *lock_file = xcalloc(1, sizeof(struct lock_file));
21 newfd = hold_lock_file_for_update(lock_file, get_index_file(), 0);
23 entries = read_cache();
24 if (entries < 0)
25 die("git-write-tree: error reading cache");
27 if (!active_cache_tree)
28 active_cache_tree = cache_tree();
30 was_valid = cache_tree_fully_valid(active_cache_tree);
32 if (!was_valid) {
33 if (cache_tree_update(active_cache_tree,
34 active_cache, active_nr,
35 missing_ok, 0) < 0)
36 die("git-write-tree: error building trees");
37 if (0 <= newfd) {
38 if (!write_cache(newfd, active_cache, active_nr)
39 && !close(newfd)) {
40 commit_lock_file(lock_file);
41 newfd = -1;
44 /* Not being able to write is fine -- we are only interested
45 * in updating the cache-tree part, and if the next caller
46 * ends up using the old index with unupdated cache-tree part
47 * it misses the work we did here, but that is just a
48 * performance penalty and not a big deal.
52 if (prefix) {
53 struct cache_tree *subtree =
54 cache_tree_find(active_cache_tree, prefix);
55 hashcpy(sha1, subtree->sha1);
57 else
58 hashcpy(sha1, active_cache_tree->sha1);
60 if (0 <= newfd)
61 close(newfd);
62 rollback_lock_file(lock_file);
64 return 0;
67 int cmd_write_tree(int argc, const char **argv, const char *unused_prefix)
69 int missing_ok = 0, ret;
70 const char *prefix = NULL;
71 unsigned char sha1[20];
73 while (1 < argc) {
74 const char *arg = argv[1];
75 if (!strcmp(arg, "--missing-ok"))
76 missing_ok = 1;
77 else if (!prefixcmp(arg, "--prefix="))
78 prefix = arg + 9;
79 else
80 usage(write_tree_usage);
81 argc--; argv++;
84 if (argc > 2)
85 die("too many options");
87 ret = write_tree(sha1, missing_ok, prefix);
88 printf("%s\n", sha1_to_hex(sha1));
90 return ret;