git.99rst.org / git.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: ec4d77a)
attr.c: plug small leak in parse_attr_line()
authorJunio C Hamano <redacted>
Sat, 28 Jan 2017 02:01:49 +0000 (18:01 -0800)
committerJunio C Hamano <redacted>
Wed, 1 Feb 2017 21:46:52 +0000 (13:46 -0800)
commit62af896979403c6bcb441ca784121e720b161b30
tree519a9f4799c98afafa5a25275ac96b71b57d628ctree | snapshot
parentec4d77aa508ac36f1f65ca8f228d4cbac42d694ccommit | diff
attr.c: plug small leak in parse_attr_line()

If any error is noticed after the match_attr structure is allocated,
we shouldn't just return NULL from this function.

Add a fail_return label that frees the allocated structure and
returns NULL, and consistently jump there when we want to return
NULL after cleaning up.

Signed-off-by: Junio C Hamano <redacted>
Signed-off-by: Stefan Beller <redacted>
Signed-off-by: Brandon Williams <redacted>
Signed-off-by: Junio C Hamano <redacted>
attr.c diff | blob | history
git clone https://git.99rst.org/PROJECT