Hacker News new | ask | show | jobs
by abhorrence 3648 days ago
Operator precedence causes it to be interpreted as:

  if ((packet->flags & (DNS_RESPONSE_FLAG == 0)) || (packet->questioncount != 1))
which at least does not read as intended.
1 comments

You are correct. Has been fixed.