bartobri / bmenu

A general purpose terminal menu that focuses on ease of configuration and use.
GNU General Public License v3.0
136 stars 11 forks source link

Segmentation fault on exit #8

Closed olafrv closed 6 years ago

olafrv commented 6 years ago

In a Linux Redhat 6.5 64 bits, the following error appears after press "exit" from bmenu prompt.

root@srvnagiosapp02:# ./config_edit.sh
./config_edit.sh: line 3: 52742 Segmentation fault      (core dumped) /root/scripts/bmenu-master/bin/bmenu -c /root/scripts/config_edit.menu

File: config_edit.sh

#!/bin/sh
/root/scripts/bmenu-master/bin/bmenu \
        -c /root/scripts/config_edit.menu

File: config_edit.menu

php.ini:/usr/bin/vim /etc/php.ini
httpd.conf:/usr/bin/vim /etc/httpd/conf/httpd.conf
nagios.cfg:/usr/bin/vim /usr/local/nagios/etc/nagios.cfg
ndo2db.cfg:/usr/bin/vim /usr/local/nagios/etc/ndo2db.cfg
npcd.cfg:/usr/bin/vim /usr/local/nagios/etc/pnp/npcd.cfg
nrdp_config.inc.php:/usr/bin/vim /usr/local/nrdp/server/config.inc.php
process_perfdata.cfg:/usr/bin/vim /usr/local/nagios/etc/pnp/process_perfdata.cfg
nagiosql_settings.php:/usr/bin/vim /var/www/html/nagiosql/config/settings.php
nagiosxi_config.inc.php:/usr/bin/vim /usr/local/nagiosxi/html/config.inc.php
nagiosmobile_include.inc.php:/usr/bin/vim /usr/local/nagiosmobile/include.inc.php
bartobri commented 6 years ago

Hi. Thanks for reporting this. Turns out I had an out-of-bounds issue when I freed memory on program exit. It only exists with 10 menu options. This should be fixed now. See commit 08c69054.