bol-van / zapret

DPI bypass multi platform
5.84k stars 512 forks source link

nfqws: fix buffer signedness for `nfq_handle_packet()` #268

Closed LiviaMedeiros closed 1 month ago

LiviaMedeiros commented 1 month ago

Refs: https://git.netfilter.org/libnetfilter_queue/tree/src/libnetfilter_queue.c?h=396d187fc8684a49a867cdfeacf9c40baa99fdb8#n638

bol-van commented 1 month ago

in nfqws i work with uint8_t buffers i dont understand what are you fixing compiler warning ? then it should be something like (char*)buf

anphsw commented 1 month ago

Alrweady fixed here in pending commit: https://github.com/bol-van/zapret/pull/205/commits/1db0c8193066b32c19bfdda16da632fa5f4df0b5

LiviaMedeiros commented 1 month ago

Agreed, typecasting is preferable. Closing this in favor of https://github.com/bol-van/zapret/pull/205.