Segfault about cava

[karlstav]

woha a lot of details here. I must admit, I do not really understand what I'm looking at.

@gregf when does it segfault? Are you running any command line arguments? Anything special in the config file?

@karlstav Any command I run it segfaults. If I remove my config it appears to work. So I think it has something to do with iniparser? I don't see anything invalid in my config myself.

## Configuration file for CAVA. Default values are commented out.  Use either ; or # for commenting.

[general]
; mode = normal # smoothing mode, can be normal, scientific or waves.
; framerate = 60 # Default: 60. Accepts only non-negative values.
; sensitivity = 100 #  is sensitivity %. Accepts only non-negative values.
; bars = 0 # the number of bars. 0 sets it to auto (fil upp console). 
; bar_width = 3 # width of bars. 
; bar_spacing = 1 # space between bars.
; lower_cutoff_freq = 50 # the lower cutof frequency for the far left bar
; higher_cutoff_freq = 10000 # the higher cutoff frequency for the far right bar. Note: there is a minimum total bandwith of 43Mhz x number of bars. Cava will automaticly increase the higher cuttoff if a too low band is specified.



[input]
method = fifo # supported input methods are 'alsa' or 'fifo'.
source = /home/gregf/.mpd/mpd.fifo # ALSA device or FIFO path.

[output]
; method = ncurses # method used to draw output may be ncurses, noncurses or circle.
; style = stereo # visual styles, may be 'stereo' or 'mono'. Stereo mirrors both channels with low frequencies in center. Mono avrages both channels and outputs left to right lowest to highest frequencies

[color]
# supported colors are: red, green, yellow, magenta, cyan, white, blue, black.
; background = black
; foreground = cyan

[smoothing]
; integral = 0.7 # multiplier for the integral smoothing calculations. Takes values from 0 - 0.99. Higher values means smoother, but less precise. 0 to disable.
; monstercat = 1 # disables or enables the so-called "Monstercat smoothing". Default: 1. Set to 0 to disable. 
; gravity = 1 # Set gravity multiplier for "drop off". Higher values means bars will drop faster. Accepts only non-negative values. 0.5 means half gravity, 2 means double. Set to 0 to disable "drop off".
; ignore = 0 # set bars with height lower than this to 0

[eq]
# This one is tricky. You can have as much keys as you want. More keys = more precision. Look at readme.md on github for further explanations and examples.
; 1 = 1 # bass
; 2 = 1
; 3 = 1
; 4 = 1
; 5 = 1 # treble

[karlstav]

could you could try to comment out the two lines you changed (method and source), just to check?

I did that playing around with it, and it didn't make any difference.

On Thu, Oct 29, 2015 at 07:33:38AM -0700, karl wrote:

could you could try to comment out the two lines you changed (method and source), just to check?

---

Reply to this email directly or view it on GitHub:
#78 (comment)

--gregf

iniparser-4.0_2 if you want to try and replicate the issue.

[karlstav]

are you using a system wide iniparser or the bundled one?

I have tried with both the bundled and the newest from github without any issues.

Looks like system wide.

 ldd /usr/bin/cava
        linux-vdso.so.1 (0x00007ffc5d1a9000)
        libiniparser.so.0 => /usr/lib/libiniparser.so.0 (0x00007f59ce69f000)
        libasound.so.2 => /usr/lib/libasound.so.2 (0x00007f59ce39f000)
        libm.so.6 => /usr/lib/libm.so.6 (0x00007f59ce09f000)
        libfftw3.so.3 => /usr/lib/libfftw3.so.3 (0x00007f59cdd0f000)
        libpthread.so.0 => /usr/lib/libpthread.so.0 (0x00007f59cdaef000)
        libncursesw.so.6 => /usr/lib/libncursesw.so.6 (0x00007f59cd877000)
        libc.so.6 => /usr/lib/libc.so.6 (0x00007f59cd4cf000)
        libdl.so.2 => /usr/lib/libdl.so.2 (0x00007f59cd2c7000)
        librt.so.1 => /usr/lib/librt.so.1 (0x00007f59cd0bf000)
        /lib/ld-linux-x86-64.so.2 (0x00007f59ce8a7000)

[karlstav]

could you try to compile it with the bundled instead?

are you by any chance using the package from AUR? It is kinda out of date, although it should work.

I narrowed it down to this line in the config file.

; style = stereo # visual styles, may be 'stereo' or 'mono'. Stereo mirrors both channels with low frequencies in center. Mono avrages both channels and outputs left to right lowest to highest frequencies

The comment to the right of stereo for some reason causes the segfault.

I'm not using archlinux. I tried both the package for voidlinux.eu and I tried compiling it by hand.

I think its a line length issue with iniparser. It segfaults on the word center. In the sentence frequencies in center. If I delete the word center it then works.

This appears to be fixed with the bundled version and the latest version of cava. So i'm very sorry to have wasted your time. Voidlinux has 0.3.5 packaged and it looks like the latest version is 0.3.5.9. I'm going to submit a pull request to update there package. Thank you for your time, and again sorry to have wasted it.

[karlstav]

@gregf no problem 😉 There must have been a problem with the version of iniparser used in Voidlinux, nothing have been done with cava lately than can have caused/fixed this.

For what it's worth this made me realize that the lines on the comment in the config file are waaaaay to long, looks like a mess...

Void's iniparser-package got patched and plays nicely with cava. So yes, Voidlinux' Cava-Package is working again.