Hi,
On Thu, 2 Aug 2007, Junio C Hamano wrote:
> Johannes Schindelin <Johannes.Schindelin@xxxxxx> writes:
>
> > Hi,
> >
> > On Thu, 2 Aug 2007, Alex Riesen wrote:
> >
> >> Johannes Schindelin, Thu, Aug 02, 2007 17:38:37 +0200:
> >>
> >> > On Thu, 2 Aug 2007, Alex Riesen wrote:
> >> >
> >> > >@@ -209,7 +209,8 @@ const char *set_work_tree(const char *dir)
> >> > > len = strlen(dir);
> >> > > if (len > postfix_len && !strcmp(dir + len - postfix_len,
> >> > > "/" DEFAULT_GIT_DIR_ENVIRONMENT)) {
> >> > >- strncpy(dir_buffer, dir, len - postfix_len);
> >> > >+ strncpy(dir_buffer, dir, len - postfix_len);
> >> > >+ dir_buffer[len - postfix_len] = '\0';
> >> > >
> >> > > /* are we inside the default work tree? */
> >> > > rel = get_relative_cwd(buffer, sizeof(buffer), dir_buffer);
> >> >
> >> > Darn, darn, darn. strncpy does _not_ NUL terminate. I keep forgetting
> >> > that.
> >> >
> >> > Better use strlcpy()?
> >>
> >> Of course, but it just should not be needed at all: static supposed to
> >> be zeroed.
> >
> > Certainly. But reality outweighs theory, and so I Ack either your patch
> > or replacing it by strlcpy().
>
> Static is supposed to be zeroed and also is supposed to retain
> the value from the previous call. I am guessing from the change
> to make "rel" to non-static that this function is called twice
> perhaps?
Apparently (but I would feel safer with strlcpy() anyway). git-read-tree
is the first and only offender which comes up in the test suite:
-- snipsnap --
[PATCH] read-tree: remove unnecessary call to setup_git_directory()
read-tree is already marked with RUN_SETUP in git.c, so there is
no need to call setup_git_directory() a second time.
Signed-off-by: Johannes Schindelin <johannes.schindelin@xxxxxx>
---
builtin-read-tree.c | 1 -
1 files changed, 0 insertions(+), 1 deletions(-)
diff --git a/builtin-read-tree.c b/builtin-read-tree.c
index 41f8110..a3b17a3 100644
--- a/builtin-read-tree.c
+++ b/builtin-read-tree.c
@@ -97,7 +97,6 @@ int cmd_read_tree(int argc, const char **argv, const char *unused_prefix)
memset(&opts, 0, sizeof(opts));
opts.head_idx = -1;
- setup_git_directory();
git_config(git_default_config);
newfd = hold_locked_index(&lock_file, 1);
--
1.5.3.rc3.121.g7f37
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
