ps_matched: xcalloc() takes nmemb and then element size
authorJunio C Hamano <gitster@pobox.com>
Thu, 20 Aug 2015 16:57:32 +0000 (09:57 -0700)
committerJunio C Hamano <gitster@pobox.com>
Thu, 20 Aug 2015 16:57:38 +0000 (09:57 -0700)
Even though multiplication is commutative, the order of arguments
should be xcalloc(nmemb, size). ps_matched is an array of 1-byte
element whose size is the same as the number of pathspec elements.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin/checkout.c
builtin/ls-files.c
index 3e141fc1491949a2dbb47c59d7d156ae6fe4eb47..4f826e3b034eebcc2980f935ac4096be466d5fb1 100644 (file)
@@ -280,7 +280,7 @@ static int checkout_paths(const struct checkout_opts *opts,
        if (opts->source_tree)
                read_tree_some(opts->source_tree, &opts->pathspec);
 
-       ps_matched = xcalloc(1, opts->pathspec.nr);
+       ps_matched = xcalloc(opts->pathspec.nr, 1);
 
        /*
         * Make sure all pathspecs participated in locating the paths
index 6fa2205734e435c9db787193dcf8c0392d9d2a24..b6a7cb0c7c48293c6348806cb342a9289f146740 100644 (file)
@@ -516,7 +516,7 @@ int cmd_ls_files(int argc, const char **argv, const char *cmd_prefix)
 
        /* Treat unmatching pathspec elements as errors */
        if (pathspec.nr && error_unmatch)
-               ps_matched = xcalloc(1, pathspec.nr);
+               ps_matched = xcalloc(pathspec.nr, 1);
 
        if ((dir.flags & DIR_SHOW_IGNORED) && !exc_given)
                die("ls-files --ignored needs some exclude pattern");