diff options
| author | Panu Matilainen <pmatilai@redhat.com> | 2019-06-20 13:50:23 +0300 |
|---|---|---|
| committer | Florian Festi <ffesti@redhat.com> | 2019-06-24 17:10:33 +0200 |
| commit | 0fd720b1f0d3ec733dc1592bb3a7131e899c246b (patch) | |
| tree | a3eddb9f5b31322f3aa68a5f68eef9871ae3096b /lib/headerfmt.c | |
| parent | 4c8b584074061c606883ce3f18c8c9024d8610f5 (diff) | |
| download | rpm-0fd720b1f0d3ec733dc1592bb3a7131e899c246b.tar.gz | |
Fix excessive use of thread local storage (RhBug:1722181)
Commit 6487e873f3169c2bffbd52808b6c749e6c104ff5 introduced a thread local
BUFSIZ static buffer for header format error reporting but thread local
storage is apparently a rather scarce resource (on some architectures more
so than others) and a static buffer is highly excessive use of that resource.
Use a thread local pointer to dynamically (re)allocated buffer instead.
Diffstat (limited to 'lib/headerfmt.c')
| -rw-r--r-- | lib/headerfmt.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/lib/headerfmt.c b/lib/headerfmt.c index 1f6390b5e..7c0da1bd9 100644 --- a/lib/headerfmt.c +++ b/lib/headerfmt.c @@ -221,18 +221,18 @@ static char * hsaReserve(headerSprintfArgs hsa, size_t need) RPM_GNUC_PRINTF(2, 3) static void hsaError(headerSprintfArgs hsa, const char *fmt, ...) { - /* Use thread local static buffer as headerFormat() errmsg arg is const */ - static __thread char errbuf[BUFSIZ]; + /* Use thread local buffer as headerFormat() errmsg arg is const */ + static __thread char *errbuf = NULL; if (fmt == NULL) { hsa->errmsg = NULL; } else { va_list ap; + free(errbuf); va_start(ap, fmt); - vsnprintf(errbuf, sizeof(errbuf), fmt, ap); + rvasprintf(&errbuf, fmt, ap); va_end(ap); - hsa->errmsg = errbuf; } } |