Skip to content
This repository has been archived by the owner on Apr 3, 2023. It is now read-only.

Off by one issue.. easy workaround but quite a bit broken #1

Open
mzpqnxow opened this issue Jan 7, 2019 · 2 comments
Open

Off by one issue.. easy workaround but quite a bit broken #1

mzpqnxow opened this issue Jan 7, 2019 · 2 comments

Comments

@mzpqnxow
Copy link

mzpqnxow commented Jan 7, 2019

If I use the example, and use 10.0.0.0/8, the output produced is actually for 11.0.0.0/8. Quite weird. That said, I can just specify 9.0.0.0/8 and I get 10.0.0.0/8, so it isn't a blocking bug. But seems like a simple bug somewhere.

@jamiealquiza
Copy link
Owner

Thanks for noticing, that is a weird one. I'll check this out.

@jamiealquiza
Copy link
Owner

It actually looks like there's actually a greater bug here. This transition isn't correct; the second octet shouldn't increment here:

10.0.1.251
10.0.1.252
10.0.1.253
10.0.1.254
10.1.2.1
10.1.2.2
10.1.2.3
10.1.2.4

I'll have to look closer at this later.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants