Message ID | 20240909231459.GD921834@coredump.intra.peff.net (mailing list archive) |
---|---|
State | Accepted |
Commit | a2417a03c92d5a1cbd7d9b7e787e9d3882ad8842 |
Headers | show |
Series | ref-filter %(trailer) fixes | expand |
diff --git a/ref-filter.c b/ref-filter.c index e39f505a81..4d0df546da 100644 --- a/ref-filter.c +++ b/ref-filter.c @@ -578,7 +578,7 @@ static int trailers_atom_parser(struct ref_format *format UNUSED, strbuf_addf(err, _("expected %%(trailers:key=<value>)")); else strbuf_addf(err, _("unknown %%(trailers) argument: %s"), invalid_arg); - free((char *)invalid_arg); + free(invalid_arg); return -1; } }
There's no need to cast invalid_arg before freeing it. It is already a non-const pointer. Signed-off-by: Jeff King <peff@peff.net> --- ref-filter.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)