[Swan-dev] [PATCH libreswan v2] netlink: Silence negative shift coverity false warning
Antony Antony
antony at phenome.org
Fri Sep 22 13:21:27 UTC 2017
On Fri, Sep 15, 2017 at 11:17:43PM -0400, D. Hugh Redelmeier wrote:
> | From: Antony Antony <antony at phenome.org>
>
> coverity-detected anomalies are sometimes subtle. So I looked at this
> code and found a couple of bugs. I also did some tidying. But no
> testing!
>
> Aviv, Antony: please have a look at commit
> f7aaa80198851ef71da4be8dc54e816f7064e29a
>
> Note: % 31 is almost never correct. I think that in this code, + 31 is
> also wrong.
Thanks DHR. This commit looks good to me. It address my concerns too.
> Could someone with real hardware test this code?
not me. I have no acess to the real hardware to test.
-antony
More information about the Swan-dev
mailing list