Package: keepalived
Version: 1:1.3.2-1
Severity: normal

Hi,

The current squeeze version of keepalived is not parsing correctly the brackets 
to open a section.
This is related, but not the same, as 
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=656620.

This is already fixed upstream (v2.0.7 tested), therefore fixing 
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=900260 would fix also this 
bug.

IMO the most closer entry in the changelog is:
```
Release 1.4.2
[...]
* Allow more flexibility of layout of { and } in config files.
  keepalived was a bit fussy about where '{'s and '}'s (braces) could
  be placed in terms of after the keyword, or on a line on their own.
  It certainly was not possible to have multiple braces on one line.
  This commit now provides complete flexibility of where braces are, so 
  long as they occur in the correct order.
```

Here a reproducer:

Suppose the following vrrp_script
```
vrrp_script chk_apache{
    script "/bin/true"
    interval 2
    timeout 4
    fall 2
    rise 10
}
```

Please note the "chk_apache{" without whitespace.

In case a vrrp_script is written like
`vrrp_script chk_apache{`
In place of
`vrrp_script chk_apache {`

The dump reports correctly the entry
```
 VRRP Script = chk_apache{
 VRRP process priority = 0
   Command = /usr/lib/nagios/plugins/check_http -I 127.0.0.1
   Interval = 2 sec
[...]
```
But it also report errors
```
# keepalived -dDnl -f /etc/keepalived/keepalived.conf
Opening file '/etc/keepalived/keepalived.conf'.
Unknown keyword 'script'
Unknown keyword 'interval'
Unknown keyword 'timeout'
Unknown keyword 'fall'
Unknown keyword 'rise'
Unknown keyword '}'
[...]
Opening file '/etc/keepalived/keepalived.conf'.
Missing '{' at beginning of configuration block
[...]
     chk_apache no match, ignoring...
```
Seems that the `{` is not limiting the identification of a section, but the 
section is anyway skipped.

My 2 cents,
Daniele

-- System Information:
Debian Release: 9.5
  APT prefers stable-updates
  APT policy: (500, 'stable-updates'), (500, 'stable')
Architecture: amd64 (x86_64)

Kernel: Linux 4.9.0-8-amd64 (SMP w/1 CPU core)
Locale: LANG=C, LC_CTYPE=C (charmap=ANSI_X3.4-1968), LANGUAGE=C 
(charmap=ANSI_X3.4-1968)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)

Versions of packages keepalived depends on:
ii  init-system-helpers  1.48
ii  iproute2             4.9.0-1+deb9u1
ii  libc6                2.24-11+deb9u3
ii  libip4tc0            1.6.0+snapshot20161117-6
ii  libip6tc0            1.6.0+snapshot20161117-6
ii  libnl-3-200          3.2.27-2
ii  libnl-genl-3-200     3.2.27-2
ii  libnl-route-3-200    3.2.27-2
ii  libsnmp30            5.7.3+dfsg-1.7
ii  libssl1.1            1.1.0f-3+deb9u2
ii  libxtables12         1.6.0+snapshot20161117-6

Versions of packages keepalived recommends:
ii  ipvsadm  1:1.28-3+b1

keepalived suggests no packages.

-- no debconf information

Reply via email to