containernetworking / plugins / 1024

54%

master: 55%

Pull #408

travis-ci

ensure iptables chain creation is idempotent

Concurrent use of the `portmap` and `firewall` plugins can result in
errors during iptables chain creation:

- The `portmap` plugin has a time-of-check-time-of-use race where it
  checks for existence of the chain but the operation isn't atomic.
- The `firewall` plugin doesn't check for existing chains and just
  returns an error.

This commit makes both operations idempotent by creating the chain and
then discarding the error if it's caused by the chain already existing.

Signed-off-by: Tim Gross <tim@0x74696d.com>

14 of 14 new or added lines in 2 files covered. (100.0%)

6588 of 12204 relevant lines covered (53.98%)

1.83 hits per line