diff options
author | Andrew J. Schorr <aschorr@telemetry-investments.com> | 2017-05-24 09:49:29 -0400 |
---|---|---|
committer | Andrew J. Schorr <aschorr@telemetry-investments.com> | 2017-05-24 09:49:29 -0400 |
commit | 6163a2b1d5bdc76c395f5c7c1d0d8ef445011357 (patch) | |
tree | d03d51dc96fc1e056de342500bf28103aaaad1a8 /field.c | |
parent | 21f45911879abbaf5933c65df1beac8399b30038 (diff) | |
download | egawk-6163a2b1d5bdc76c395f5c7c1d0d8ef445011357.tar.gz egawk-6163a2b1d5bdc76c395f5c7c1d0d8ef445011357.tar.bz2 egawk-6163a2b1d5bdc76c395f5c7c1d0d8ef445011357.zip |
FIELDWIDTHS parsing should protect against blanks after skip separator, and fix field number in error message.
Diffstat (limited to 'field.c')
-rw-r--r-- | field.c | 4 |
1 files changed, 2 insertions, 2 deletions
@@ -1182,7 +1182,7 @@ set_FIELDWIDTHS() if (errno == 0 && *end == ':' && (0 < tmp && tmp <= UINT_MAX)) { FIELDWIDTHS->fields[i].skip = tmp; scan = end + 1; - if (*scan == '-') { + if (*scan == '-' || is_blank(*scan)) { fatal_error = true; break; } @@ -1223,7 +1223,7 @@ set_FIELDWIDTHS() if (fatal_error) fatal(_("invalid FIELDWIDTHS value, for field %d, near `%s'"), - i, scan); + i + 1, scan); } /* set_FS --- handle things when FS is assigned to */ |