Em Mon, Feb 08, 2021 at 09:08:59PM +0100, Jiri Olsa escreveu: > + > + if (!pollfd.revents & POLLIN) { > + pr_err("failed: did not received an ack\n"); > + goto out; > + } > +
Fixed up this, pointed out by clang on many build containers, including fedora:34: Committer notes: Fixed up bug pointed by clang: Buggy: if (!pollfd.revents & POLLIN) Correct code: if (!(pollfd.revents & POLLIN)) clang warning: builtin-daemon.c:560:6: error: logical not is only applied to the left hand side of this bitwise operator [-Werror,-Wlogical-not-parentheses] if (!pollfd.revents & POLLIN) { ^ ~ builtin-daemon.c:560:6: note: add parentheses after the '!' to evaluate the bitwise operator first - Arnaldo