[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#52438: 29.0.50; [PATCH] Fix off-by-one error in etags.c TeX support
From: |
Lars Ingebrigtsen |
Subject: |
bug#52438: 29.0.50; [PATCH] Fix off-by-one error in etags.c TeX support |
Date: |
Sun, 12 Dec 2021 06:41:43 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/29.0.50 (gnu/linux) |
David Fussner <dfussner@googlemail.com> writes:
> The etags program reliably crashes for me when running the test suite in
> test/manual/etags, but you need to have a long-enough TEXTAGS
> environment variable to make it happen. On my 32-bit GNU/Linux machine,
> going to that directory and running:
>
> TEXTAGS="A:B:C:D:E:F:G:H:I:J:K:L:M:N:O:P:Q:R:S:T:U:V:W:X" make check
>
> crashes the program before it finishes the first run, that is, before it
> stops because the test suite itself fails.
I'm not able to reproduce the issue on my 64-bit Debian system, but
reading the code, I think your analysis of the problem sounds correct.
The patch seemed slightly confusing, though, so I rewrote it a bit.
Could you check whether this version also fixes the problem?
diff --git a/lib-src/etags.c b/lib-src/etags.c
index bd4d4fcf53..ba66eeede4 100644
--- a/lib-src/etags.c
+++ b/lib-src/etags.c
@@ -5773,7 +5773,7 @@ TeX_commands (FILE *inf)
TEX_decode_env (const char *evarname, const char *defenv)
{
const char *env, *p;
- ptrdiff_t len;
+ ptrdiff_t len = 1;
/* Append default string to environment. */
env = getenv (evarname);
@@ -5782,8 +5782,13 @@ TEX_decode_env (const char *evarname, const char *defenv)
else
env = concat (env, defenv, "");
+ /* If the environment variable starts with a colon, increase the
+ length of the token table. */
+ if (!strneq (env, ":", 1))
+ len++;
+
/* Allocate a token table */
- for (len = 1, p = env; (p = strchr (p, ':')); )
+ for (p = env; (p = strchr (p, ':')); )
if (*++p)
len++;
TEX_toktab = xnew (len, linebuffer);
--
(domestic pets only, the antidote for overdose, milk.)
bloggy blog: http://lars.ingebrigtsen.no
- bug#52438: 29.0.50; [PATCH] Fix off-by-one error in etags.c TeX support, David Fussner, 2021/12/11
- bug#52438: 29.0.50; [PATCH] Fix off-by-one error in etags.c TeX support,
Lars Ingebrigtsen <=
- bug#52438: 29.0.50; [PATCH] Fix off-by-one error in etags.c TeX support, Andreas Schwab, 2021/12/12
- bug#52438: 29.0.50; [PATCH] Fix off-by-one error in etags.c TeX support, David Fussner, 2021/12/12
- bug#52438: 29.0.50; [PATCH] Fix off-by-one error in etags.c TeX support, Lars Ingebrigtsen, 2021/12/12
- bug#52438: 29.0.50; [PATCH] Fix off-by-one error in etags.c TeX support, David Fussner, 2021/12/12
- bug#52438: 29.0.50; [PATCH] Fix off-by-one error in etags.c TeX support, David Fussner, 2021/12/12
- bug#52438: 29.0.50; [PATCH] Fix off-by-one error in etags.c TeX support, David Fussner, 2021/12/12
- bug#52438: 29.0.50; [PATCH] Fix off-by-one error in etags.c TeX support, Lars Ingebrigtsen, 2021/12/12