Project

General

Profile

Actions

Feature #3131

closed

Remove unneeded connection bsd filter handling for OpenBSD

Added by devnexen 5 months ago. Updated 5 months ago.

Status:
Fixed
Priority:
Low
Category:
core
Target version:
ASK QUESTIONS IN Forums:
No

Description

- Just a little nit ; OpenBSD is the only main BSD not supporting accept filter on sockets.
- Edited from master branch.


Files

Actions #1

Updated by gstrauss 5 months ago

Why was this filed as a bug? What is the impact?

If OpenBSD does not define SO_ACCEPTFILTER, then the rest of the code in that code block does not exist.

Actions #2

Updated by devnexen 5 months ago

Because it is not a new feature.

Thinking of it maybe surrounding the whole block by SO_ACCEPTFILTER instead of those BSD constants would be better ?

Actions #3

Updated by gstrauss 5 months ago

  • Category set to core
  • Status changed from New to Invalid
  • Priority changed from Normal to Low

What is the impact?

Impact is zero. Modern compiler optimizers will remove the empty code block on OpenBSD, as buffer_is_equal_string is defined to buffer_eq_slen, which is marked __attribute__(__pure__) in buffer.h.

A bug has a negative impact. If it is not a bug, you can mark it as a feature, not the other way around.

Actions #4

Updated by gstrauss 5 months ago

  • Target version deleted (1.4.xx)
Actions #5

Updated by gstrauss 5 months ago

  • Tracker changed from Bug to Feature
  • Status changed from Invalid to Patch Pending
  • Target version set to 1.4.64
Actions #6

Updated by gstrauss 5 months ago

  • Status changed from Patch Pending to Fixed
Actions

Also available in: Atom