Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Non compacted IPv6 address conflict after reload #1318

Open
luizluca opened this issue Mar 5, 2024 · 1 comment
Open

Non compacted IPv6 address conflict after reload #1318

luizluca opened this issue Mar 5, 2024 · 1 comment
Labels
needinfo Needs more information before action can be taken. triage Issue needs triaged.

Comments

@luizluca
Copy link

luizluca commented Mar 5, 2024

What happened

If I have an ipset defined in files with fdc8:1324:5678:0::/64, firewalld will return an error on reload like this:

Error: INVALID_ENTRY: Entry 'fdc8:1324:5678::/64' overlaps with existing entry 'fdc8:1324:5678:0::/64'

In that case, firewalld blocks all new connections, locking the machine out of the network.

What you expected to happen

It should treat both as the same address

How to reproduce it (as minimally and precisely as possible)

Use this ipset and reload firewalld:

<?xml version="1.0" encoding="utf-8"?>
<ipset type="hash:net">
  <option name="family" value="inet6"/>
  <entry>fdc8:1324:5678:0::/64</entry>
</ipset>

Anything else we need to know?

If the ipset was created using the firewall-cmd command, it avoids the issue by compacting the address before writing to the config file.

Firewalld Version

2.1.1

Firewalld Backend

nftables

Linux distribution

OpenSUSE Tumbleweed

Linux kernel version

6.7.5-1-default

Other information

No response

@luizluca luizluca added the triage Issue needs triaged. label Mar 5, 2024
erig0 added a commit to erig0/firewalld that referenced this issue Mar 19, 2024
@erig0
Copy link
Collaborator

erig0 commented Mar 19, 2024

I was unable to reproduce this. I created a test case using your XML from above and reload works as expected.

erig0@1222d53

Do you have any more information that can be used to reproduce this?

@erig0 erig0 added the needinfo Needs more information before action can be taken. label Mar 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needinfo Needs more information before action can be taken. triage Issue needs triaged.
Projects
None yet
Development

No branches or pull requests

2 participants