Conky: Memory allocation error

[mir07]

Package: conky
Version: 1.10.0-1
Severity: important
Tags: upstream

When starting conky everything works but after running approximately 30 min conky crashes with the following output from DEBUG:
DEBUG(0) [/build/conky-FWDzxw/conky-1.10.0/src/specials.cc:460]: reallocing graph from 303 to 343
conky: malloc.c:3695: _int_malloc: Assertion `(unsigned long) (size) >= (unsigned long) (nb)' failed.

-- System Information:
Debian Release: stretch/sid
APT prefers unstable
APT policy: (900, 'unstable'), (1, 'experimental')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 4.0.0-2-amd64 (SMP w/6 CPU cores)
Locale: LANG=en_DK.UTF-8, LC_CTYPE=en_DK.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash
Init: systemd (via /run/systemd/system)

Versions of packages conky depends on:
ii conky-all 1.10.0-1

[corpix]

This could probably be a duplicate of https://github.com/brndnmtthws/conky/issues/103

[lasers]

Can you tell us if you're still experiencing same behavior in v1.10.9_pre? If you do, please give us a new log to work with. There are too many changes in specials.cc to follow through. Many thanks. :-)

[lasers]

3 years 1 month passed. I found a duplicate behavior https://github.com/brndnmtthws/conky/issues/421#issuecomment-33774168 with newer date Oct 18, 2017.

_Using that instead..._

10 months passed. Too much has changed. We can track changes no more than with git log -p 2600d013..HEAD --follow -- src/specials.cc. That will give us three commits to work with.

... v1.10.7
* 3 months ago 4b92556f Brenden Matthews Build improvements: clang-tidy, sonar, Dockerfile. (#488)
* 3 months ago eebc8c65 Brenden Matthews Add formatting/static analysis (#486)
... v1.10.8
* 1 year, 6 months ago a58a6d4e Darin McBride Clean up a few valgrind errors, add realloc-failure handling for graphs. (#367)

The last commit looks unrelated now giving us two commits to work with. We should only have to study few lines around current->type = t; and/or reallocing graph from to address Assertion '(unsigned long) (size) >= (unsigned long) (nb)' failed.'

[lasers]

@su8 Have you looked at this? Should be a small fix.

[su8]

I couldn't find any '(unsigned long) (size) >= (unsigned long) (nb)' traces in the current code. Otherwise current->type = t; is a special type enum, e.g. alignc, alignr, fg, bg

[lasers]

Guy got it in 1.10.6. Have you looked in that version? Checkout 4cd04f00 for 1.10.5 or e94f0494 for 1.10.6. It looks like nothing changed between commits... Is only code smellz so it had to be somewhere. I dunno.

EDIT: If nothing changed, then it should be in 1.10.9_pre too.

[lasers]

@su8 No malloc.c here too? Keep this open or close?

[su8]

Wait till mister Brenden releases a new conky version, so people can upgrade/migrate to the newer version.

[lasers]

@su8 New version went out. Can we close this now?

[su8]

Yes :octopus:

[lasers]

Code no longer exist. Closing. :elephant: