On Thu, Mar 27, 2025 at 12:43:49PM +0000, Johannes Schindelin via GitGitGadget wrote: > From: Johannes Schindelin <johannes.schindelin@xxxxxx> > > The `mtimes_size` variable is uninitialzed when the function errors out, > yet its value is assigned to another variable. Let's just initialize it. > > Signed-off-by: Johannes Schindelin <johannes.schindelin@xxxxxx> > --- > pack-mtimes.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/pack-mtimes.c b/pack-mtimes.c > index cdf30b8d2b0..c1f531d45a0 100644 > --- a/pack-mtimes.c > +++ b/pack-mtimes.c > @@ -29,7 +29,7 @@ static int load_pack_mtimes_file(char *mtimes_file, > int fd, ret = 0; > struct stat st; > uint32_t *data = NULL; > - size_t mtimes_size, expected_size; > + size_t mtimes_size = 0, expected_size; Hmm. This one follows an identical line of reasoning as in my previous response in the thread. So I think this one is likewise unnecessary (though not harmful, and certainly useful if it appeases static analysis tools, etc). Thanks, Taylor
