aboutsummaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorSergey Poznyakoff <gray@gnu.org>2019-06-05 16:34:43 +0300
committerSergey Poznyakoff <gray@gnu.org>2019-06-05 16:34:43 +0300
commitb9e95e79712c1e0021e63204e0d910da2ec2692a (patch)
treed4d70a225257807223d944972cd3a25b9bb055a4 /src
parent1fa2dd2c6587822ff682581643499c3c3b78c9ea (diff)
downloadgrecs-b9e95e79712c1e0021e63204e0d910da2ec2692a.tar.gz
grecs-b9e95e79712c1e0021e63204e0d910da2ec2692a.tar.bz2
wordsplit: special handling for WRDSE_USAGE.
* src/wordsplit.c (wordsplit_run): Initialize ws_errctx early. This ensures that wordsplit_strerror gets correct data after WRDSE_USAGE. (wordsplit_free): Return immediately if errno was WRDSE_USAGE.
Diffstat (limited to 'src')
-rw-r--r--src/wordsplit.c6
1 files changed, 6 insertions, 0 deletions
diff --git a/src/wordsplit.c b/src/wordsplit.c
index 48312bb..b38dde4 100644
--- a/src/wordsplit.c
+++ b/src/wordsplit.c
@@ -2683,12 +2683,14 @@ static int
wordsplit_run (const char *command, size_t length, struct wordsplit *wsp,
int flags, int lvl)
{
int rc;
size_t start;
+ /* Initialize error context early */
+ wsp->ws_errctx = NULL;
if (!command)
{
if (!(flags & WRDSF_INCREMENTAL))
return _wsplt_seterr (wsp, WRDSE_USAGE);
if (wsp->ws_head)
@@ -2793,12 +2795,16 @@ wordsplit_clearerr (struct wordsplit *ws)
ws->ws_errno = WRDSE_OK;
}
void
wordsplit_free (struct wordsplit *ws)
{
+ if (ws->ws_errno == WRDSE_USAGE)
+ /* Usage error: the structure is not properly initialized and there's
+ nothing to free. */
+ return;
wordsplit_clearerr (ws);
wordsplit_free_nodes (ws);
wordsplit_free_words (ws);
free (ws->ws_wordv);
ws->ws_wordv = NULL;
wordsplit_free_envbuf (ws);

Return to:

Send suggestions and report system problems to the System administrator.