shortlog: do not crash on parsing "[PATCH"

Annoyingly, it looked for the closing bracket in the author name
instead of in the message, and then accessed the NULL pointer.

Signed-off-by: Johannes Schindelin <Johannes.Schindelin@gmx.de>
Signed-off-by: Junio C Hamano <junkio@cox.net>

builtin-shortlog.c

@@ -108,8 +108,9 @@ static void insert_author_oneline(struct path_list *list,
 		free(buffer);
 
 	if (!strncmp(oneline, "[PATCH", 6)) {
-		char *eob = strchr(buffer, ']');
+		char *eob = strchr(oneline, ']');
 
+		if (eob) {
 			while (isspace(eob[1]) && eob[1] != '\n')
 				eob++;
 			if (eob - oneline < onelinelen) {
@@ -117,6 +118,7 @@ static void insert_author_oneline(struct path_list *list,
 				oneline = eob;
 			}
 		}
+	}
 
 	while (onelinelen > 0 && isspace(oneline[0])) {
 		oneline++;