Skip to content

Fix bugs#23

Open
csjayp wants to merge 3 commits intomasterfrom
csjp/bugfixes
Open

Fix bugs#23
csjayp wants to merge 3 commits intomasterfrom
csjp/bugfixes

Conversation

@csjayp
Copy link
Copy Markdown
Owner

@csjayp csjayp commented Apr 14, 2026

  │             Bug             │ Location │               Fix                │
  ├─────────────────────────────┼──────────┼──────────────────────────────────┤
  │ &a[N][0] == NULL always     │ 9 sites  │ Changed to a[N] == NULL          │
  │ false                       │          │                                  │
  ├─────────────────────────────┼──────────┼──────────────────────────────────┤
  │ *m + 1 == 'x' wrong         │ line 221 │ Changed to m[1] == 'x'           │
  │ arithmetic                  │          │                                  │
  ├─────────────────────────────┼──────────┼──────────────────────────────────┤
  │ 1 << 32 UB for /0 networks  │ line 538 │ Changed to (b < 32) ? (1u << b)  │
  │                             │          │ : 0                              │
  ├─────────────────────────────┼──────────┼──────────────────────────────────┤
  │ cmask never freed           │ line 535 │ Added free(cmask) after printf   │
  └─────────────────────────────┴──────────┴──────────────────────────────────┘

csjayp added 3 commits April 13, 2026 23:59
  ┌─────────────────────────────┬──────────┬──────────────────────────────────┐
  │             Bug             │ Location │               Fix                │
  ├─────────────────────────────┼──────────┼──────────────────────────────────┤
  │ &a[N][0] == NULL always     │ 9 sites  │ Changed to a[N] == NULL          │
  │ false                       │          │                                  │
  ├─────────────────────────────┼──────────┼──────────────────────────────────┤
  │ *m + 1 == 'x' wrong         │ line 221 │ Changed to m[1] == 'x'           │
  │ arithmetic                  │          │                                  │
  ├─────────────────────────────┼──────────┼──────────────────────────────────┤
  │ 1 << 32 UB for /0 networks  │ line 538 │ Changed to (b < 32) ? (1u << b)  │
  │                             │          │ : 0                              │
  ├─────────────────────────────┼──────────┼──────────────────────────────────┤
  │ cmask never freed           │ line 535 │ Added free(cmask) after printf   │
  └─────────────────────────────┴──────────┴──────────────────────────────────┘
processing 0.0.0.0/0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant