Code

Fix memory allocation check in open_commit_parent_menu
authorJonas Fonseca <fonseca@diku.dk>
Sun, 22 Feb 2009 17:58:25 +0000 (18:58 +0100)
committerJonas Fonseca <fonseca@diku.dk>
Sun, 22 Feb 2009 17:58:25 +0000 (18:58 +0100)
The check was not converted when the code was moved to a separate
function before the commit. To test the menu using the tig repository
start tig using:

tig blame cf63d30cbc4efb2bb87e52c4e54366751bb958fa Makefile

Then go to line 80 (type ":80<enter>") and press the key for moving to
the parent (bound to ',' by default).

tig.c

diff --git a/tig.c b/tig.c
index 60d73d52cdb0e8d58629c41b5bfa798268e1a883..58d7bd09137dd73ac495d2079eed24874934e97f 100644 (file)
--- a/tig.c
+++ b/tig.c
@@ -3530,7 +3530,7 @@ open_commit_parent_menu(char buf[SIZEOF_STR], int *parents)
        int i;
 
        items = calloc(*parents + 1, sizeof(*items));
-       if (items)
+       if (!items)
                return FALSE;
 
        for (i = 0; i < *parents; i++) {