There's no need to cast invalid_arg before freeing it. It is already a non-const pointer. Signed-off-by: Jeff King <peff@xxxxxxxx> --- ref-filter.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) 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; } } -- 2.46.0.867.gf99b2b8e0f