jvoisin / fortify-headers

Standalone portable header-based implementation of FORTIFY_SOURCE=3
BSD Zero Clause License
19 stars 3 forks source link

Use const fd_set for FD_ISSET #67

Closed ncopa closed 1 month ago

ncopa commented 1 month ago

This fixes invalid conversion errors when the fd_set is defined as const.

fixes https://github.com/jvoisin/fortify-headers/issues/66

ncopa commented 1 month ago

Alternatively, we could try refactor it to be a macro so we avoid the type definitions completely.

jvoisin commented 1 month ago

A macro might be better code-wise, but let's get this in for now, as it immediately fixes the issue at hand, and it consistent with the rest of the codebase.