]> granicus.if.org Git - cgit/commitdiff
cgit_parse_commit(): Add missing call to xstrdup()
authorLars Hjemli <hjemli@gmail.com>
Sat, 27 Oct 2007 11:50:18 +0000 (13:50 +0200)
committerLars Hjemli <hjemli@gmail.com>
Sat, 27 Oct 2007 11:55:12 +0000 (13:55 +0200)
It's rather silly to point into random memory-locations. Also, remove a
call to strdup() used on a literal char *.

Signed-off-by: Lars Hjemli <hjemli@gmail.com>
parsing.c

index ddbafd5df4cb5221da407ffd9c566afb0c69f8d2..30e7648958789fb72930c405d1ba8d8c5a382f27 100644 (file)
--- a/parsing.c
+++ b/parsing.c
@@ -242,14 +242,14 @@ struct commitinfo *cgit_parse_commit(struct commit *commit)
        t = strchr(p, '\n');
        if (t) {
                if (*t == '\0')
-                       ret->subject = strdup("** empty **");
+                       ret->subject = "** empty **";
                else
                        ret->subject = substr(p, t);
                p = t + 1;
 
                while (*p == '\n')
                        p = strchr(p, '\n') + 1;
-               ret->msg = p;
+               ret->msg = xstrdup(p);
        } else
                ret->subject = substr(p, p+strlen(p));