Bug 33349

Summary: segfaults when reading config (and is a core dump bomb)
Product: Sisyphus Reporter: Ivan Zakharyaschev <imz>
Component: keepalivedAssignee: Anton Farygin <rider>
Status: CLOSED FIXED QA Contact: qa-sisyphus
Severity: major    
Priority: P3 CC: aen, glebfm, ldv, nbr, rider, shaba
Version: unstable   
Hardware: all   
OS: Linux   

Description Ivan Zakharyaschev 2017-04-09 10:23:48 MSK 
1.3.5-alt2.S1

keepalived -d segfaults when reading the config file in a function which verifies the config (as seen in the backtrace of the core dump printed by gdb).

It's a pity that there is no keepalived-debuginfo.

And it behaves as a core dump and fork bomb, at least, when the systemd service is started: / is filled with the core dumps, and and keeps forking.

Unfortuanetly, now the system where this happened is unusable and unaccessible because of this. I'll be able to post more details on how to reproduce this when I restore this system (from scratch).
Comment 1 Anton Farygin 2017-04-09 11:30:03 MSK 
Ваня,эта сборка keepalived используется много где, если падает - у тебя что-то недонастроено.
Comment 2 nbr 2017-04-09 12:10:49 MSK 
Так дело в том, что keepalived падает с segfault  при минимальных ошибках в конфигурации! Причем в цикле.
Да, на правильных, отлаженных keepalived.conf все будет нормально. Но это не дело.


https://github.com/acassen/keepalived/issues/426

Это недоработка апстрима, но с ней надо что-то делать. Это некорректное и неробастное поведение - дампить кору при ошибках конфига. 
P.S.Я не пытаюсь закоммитить версию с keepalived в cизиф - пропускать мои таски в сизиф с ней не надо. Это чисто test-only stuff.
Comment 3 Anton Farygin 2017-04-09 14:48:16 MSK 
Ну да, падает при плохой конфигурации. 
Пишите в апстрим.
Comment 4 Anton Farygin 2018-11-19 14:31:23 MSK 
пример кривого конфига дадите ?
Comment 5 Anton Farygin 2019-01-19 22:01:38 MSK 
Кривой конфиг: 
vrrp_instance VI_1 {
    interface ens18
    state
    nopreempt
    priority 100
    virtual_router_id 1

    virtual_ipaddress {
        192.168.1.99/24 brd 192.168.1.255 dev ens18
    }
}
Comment 6 Repository Robot 2019-01-20 01:03:39 MSK 
keepalived-2.0.11-alt1 -> sisyphus:

Sat Jan 19 2019 Anton Farygin <rider@altlinux.ru> 2.0.11-alt1
- 2.0.11
- enabled strict config check (closes: #33349)