From: Ralph Corderoy <ralph@inputplus.co.uk>
Date: Sun, 10 Sep 2017 17:12:38 +0000 (+0100)
Subject: slocal.c: Alter trim() to return static array, not malloc(3).
X-Git-Url: https://diplodocus.org/git/nmh/commitdiff_plain/14e7f92ac37c70e4267e8602a036aa4e08a09447?hp=14e7f92ac37c70e4267e8602a036aa4e08a09447

slocal.c: Alter trim() to return static array, not malloc(3).

The callers are immediately passing the return value to printf(3) for
"%s".  There's only one call per printf().  None of the callers are
bothering to free(3) the existing return value.  Return the address of
trim()'s char array, now static, instead.  Leave trim() returning NULL
when passed NULL, even though that gives NULL to print(3) for "%s".  It,
and slocal's other bugs, remain.
---