diff options
author | Junio C Hamano <gitster@pobox.com> | 2014-04-03 13:39:06 -0700 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2014-04-03 13:39:06 -0700 |
commit | 3097b687bea8817b4b0afd84db8c4b1079cea92f (patch) | |
tree | c7095afe521727f96b7ae6965cc967fc55377473 /builtin | |
parent | Merge branch 'mh/remove-subtree-long-pathname-fix' into maint (diff) | |
parent | mv: prevent mismatched data when ignoring errors. (diff) | |
download | tgif-3097b687bea8817b4b0afd84db8c4b1079cea92f.tar.xz |
Merge branch 'jk/mv-submodules-fix' into maint
* jk/mv-submodules-fix:
mv: prevent mismatched data when ignoring errors.
builtin/mv: fix out of bounds write
Conflicts:
t/t7001-mv.sh
Diffstat (limited to 'builtin')
-rw-r--r-- | builtin/mv.c | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/builtin/mv.c b/builtin/mv.c index 21c46d1636..45e57f307b 100644 --- a/builtin/mv.c +++ b/builtin/mv.c @@ -179,6 +179,9 @@ int cmd_mv(int argc, const char **argv, const char *prefix) modes = xrealloc(modes, (argc + last - first) * sizeof(enum update_mode)); + submodule_gitfile = xrealloc(submodule_gitfile, + (argc + last - first) + * sizeof(char *)); } dst = add_slash(dst); @@ -192,6 +195,7 @@ int cmd_mv(int argc, const char **argv, const char *prefix) prefix_path(dst, dst_len, path + length + 1); modes[argc + j] = INDEX; + submodule_gitfile[argc + j] = NULL; } argc += last - first; } @@ -227,6 +231,11 @@ int cmd_mv(int argc, const char **argv, const char *prefix) memmove(destination + i, destination + i + 1, (argc - i) * sizeof(char *)); + memmove(modes + i, modes + i + 1, + (argc - i) * sizeof(enum update_mode)); + memmove(submodule_gitfile + i, + submodule_gitfile + i + 1, + (argc - i) * sizeof(char *)); i--; } } else |