]> git.ipfire.org Git - thirdparty/git.git/commit - commit.c
commit -S: avoid invalid pointer with empty message
authorJohannes Schindelin <johannes.schindelin@gmx.de>
Wed, 29 Jun 2016 14:14:54 +0000 (16:14 +0200)
committerJunio C Hamano <gitster@pobox.com>
Wed, 29 Jun 2016 22:07:02 +0000 (15:07 -0700)
commit3324dd8f267cb59cdd42ac33727b6844921d5017
tree40bddd3eb8bb41fdd0724d4a6d24037be083a99e
parente46579643d56162299b1756b70d418005351b256
commit -S: avoid invalid pointer with empty message

While it is not recommended, fsck.c says:

Not having a body is not a crime [...]

... which means that we cannot assume that the commit buffer
contains an empty line to separate header from body.  A commit
object with only a header without any body, not even without
a blank line after the header, is valid.

So let's tread carefully here.  strstr("\n\n") may find nothing
and return NULL.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
commit.c