buildsys: clean: remove .buildinfo files

Fix #1492: logger: print timestamps only if we have one

There's no guarantee that there's a timestamp in an skb, so
nflog_get_timestamp can fail.

bump version to 3.0-2

Fix #1446: allow pve-firewall package install twice in a row

On packages removal (!= purge) systemd units are masked.
The postinst script has then to reenable this units at the
beginning of the 'configure' step.

Our other packages are doing this manually, or automatically
when the dh_systemd_enable helpers generated a postinst,
but this was missing here.

log errors encountered by the daemon to syslog

forbid trailing commas in lists

iptables-restore doesn't allow them

bump version to 3.0-1

buildsys: update upload target

buildsys: fix deb target dependencies

buildsys: remove fakeroot from dpkg-buildpackage

buildsys: use dpkg-architecture

logger: drop gthread dependency

g_thread_new is part of glib directly, libgthread only
contains the deprecated g_thread_init() & friends which we
do not use.
This silences a build warning.

buildsys: depend on lsb-base

simulator: make lxc/qemu optional

buildsys: make job safety

fix ambiguous if statements

the funciton nflog_bind_pf(...) returns an integer smaller 0 on a
failure, we negated that which results in 1 if no failure and 0 if
there was a failure.
This is ambiguous and as no parenthesis are set the GCC 6 warning
"logical-not-parentheses" gets triggered.

Use a simple
    nflog_bind_pf(...) < 0
check instead.

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

bump version to 2.0-33

ipset: don't allow the creation of zero-prefix entries

ipsets: catch zero-prefix entries

This way the error is visible with pve-firewall compile
without breaking the rest.

bump version to 2.0-32

improve search for local-network

Skip zero-prefix routes as they make no sense to be
considered (and ipset doesn't allow ::/0 to be added
anyway).

Support /128 local addresses by also checking for identical
addresses beside b-in-a overlapping.

bump version to 2.0-31

use new repoman for upload target

don't try to apply ports to rules which don't support them

remove redundant checks

bump version to 2.0-30

add multicast DNS to the list of Macros

multicast DNS allows to quickly access hosts without the need to
configure a DNS server

add missing parameter descriptions

build-depends: add dh-systemd

bump version to 2.0-29

prevent overwriting ipsets/sec. groups by renaming

we did not check if the target name of the group/ipset
already existed, so we overwrote them

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

fix allowed group name length

the allowed length for an iptable chain is 28 chars

we had a max set of 20 but a format of
GROUP-<name>-IN and
GROUP-<name>-OUT

where <name> is the group name

but GROUP--OUT are 10 chars so we just allow 18 chars max

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

make group digest stable

if we had mulitple security groups and wanted to
edit one, we did not have a stable digest,
because perl hashes are not sorted

this patch sorts the keys before hashing

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

bump version to 2.0-28

use pve-common's ipv4_mask_hash_localnet

bump version to 2.0-27

fix #972: make PVEFW-FWBR-* rule order stable

by sorting the VM/CT IDs and the VM/CT config keys before
iterating over them.

bump version to 2.0-26

install sysctl file set set rp_filter=2

To avoid that packet gets accepted to early in fwbr. We had the
same setting in package vzctl (Proxmox VE 3.X).

bump version to 2.0-25

fix #945: add uninitialized check in lxc ipset compilation

use pve-doc-generator, bump version to 2.0-24

move option definition to PVE::Firewall

So that we can auto-generate docs.

bump version to 2.0-23

use only the top bit for our accept marks

This way we can let the remaining 31 bits be used by the
user.
Note that the routing decision has already been made when
these tables are being traversed, so the fwmark will not be
usable for routing rules (ip-rule(8)), but the mark can
still be used for other tasks such as traffic control (tc)
which happens on the outgoing interface.

add description to DHCPv6 macro

cleanup descriptions (use single quote instead of backticks)

cleanup descriptions (correctly quote backslash)

add property descriptions to improve docs

bump version to 2.0-22

Use cfs_config_path from PVE::QemuConfig

update after refactoring

LXC refactoring

call cfs_config_path and parse_lxc_network in
PVE::LXC::Config instead of PVE::LXC

bump version to 2.0-21

whitespace cleanup

test: add test for implicitly allowed container IP

ipfilter: include configured container IPs by default

added the 'ipfilter' option

This effectively acts like adding an emtpy 'ipfilter-netX'
ipset for every firewall-enabled interface.

ipv6: fix ip_compress_address_call

This only takes an address and not a CIDR notation. It does
preserve the suffix but ended up compressing
fc00:0000::0000/64 to fc00::0000/64 instead of fc00::/64 and
thus caused the firewall to always show there are pending
changes when ipv6 addresses were available.

use systemctl reload-or-restart on update

dh_installinit's -R option uses 'restart' causing a
stop-start cycle with systemd. We really don't want that.

ipfilter: imiplicitly add the default link local address

When adding an ipset for a device via the 'ipfilter-net$NUM'
name we now implicitly add the default link local address
based on the device's MAC address and a 'nomatch' entry for
the rest of fe80::/10. This is comparable to an ARP/MAC
filter in IPv4 with the main difference that it explicitly
works at IP level.

split compile_ipsets() out of compile_iptables_filter()

compile_iptables_filter() is called twice, once to get the
ipv4 ruleset + ipsets and ones to get the ipv6 ruleset. The
second call still generates ipsets which are discarded so it
makes sense to do this in a separate step.

cleanup after old change

get_ipset_cmdlist() had a delete parameter in one commit,
removed in the one after that (dd7a13fddc) and this call
was not updated accordingly with the second patch.

ndp: use PVEFW-SET-ACCEPT-MARK and move rules further down

On host level: moved NDP to after connection tracking and
switched to RETURN instead of ACCEPT.

On VM level:
The output direction now uses the accept-mark like the dhcp
option does, too.
Also moved NDP rules below the macfilter & ipset rules.

only allow icmp names in the destination port field

We generate ICMP rules from the destination port field,
so allowing them in the source port field only confuses
people.

bump version to 2.0-20

fix 901: encode unicode characters in sha digest

if we do not do this, Digest::SHA->add croaks when it detects
wide symbols

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

bump version to 2.0-19

Add radv option to VM options.

By default firewalled VMs should not be allowed to send
router advertisement packets.

bump version to 2.0-18

Add router-solicitation to NeighborDiscovery macro

to be more consistent with the host-wide NDP option.
This macro is now mostly useful to disable NDP on VMs.

Add ndp option to host and VM firewall options

It's is enabled by default.

bump version to 2.0-17

Don't leave empty FW config files behind

Unlink FW config files instead of setting their content
to nothing.

pvefw-logger.c: remove unused var

bump version to 2.0-16

logger: basic ipv6 support

Support for:
 * IPv6 main header
 * ICMPv6:
   - echo request/reply
   - NDP
   - redirects
 * destination unreachable message
 * packet too big message
 * time exceeded message
 * parameter problem messages:
   - erroneous header
   - bad next-header
   - bad ipv6 option
 * extension headers:
   - routing
   - fragmentation
   - skipping over: hopopts, destopts and mobile home

factor out IPPROTO switch for reuse

add DHCPv6 macro

add dhcpv6 support to the dhcp option

make LEPRINT* macros safe to use with if/else pairs

set RELEASE=4.1

bump version to 2.0-15

use $security_group_name_pattern in iptables_get_chains

Fixes #859

fix some regular expressions mixups

Replacing some (:?...) with (?:...) which makes more sense
here.

bump version to 2.0-14

pve-firewall.service: WantedBy=multi-user.target

Instead of network-online.target, which is a very special systemd target
which is not always pulled.

fix typo: s/stemd-modules-load.service/systemd-modules-load.service/

bump version to 2.0-13

allow numeric icmp types

make clean fix

bump version to 2.0-12

use service class to generate pod and bash-completion files

convert pve-firewall into a PVE::Service class

add better inline documentation

bump version to 2.0-11

iptables_get_chains: fix veth device name

bump version to 2.0-10

subroutine for cloning vm's firewall config file

bump version to 2.0-9

firewall remove config file subroutine added

Signed-off-by: Alen Grizonic <a.grizonic@proxmox.com>