Operator '*' will be executed prior to operator '>>', but we expect operator '>>' is executed prior to '*', this patch fixed the issue.
Signed-off-by: Yi Yang <[email protected]> --- include/openvswitch/nsh.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/openvswitch/nsh.h b/include/openvswitch/nsh.h index f4ccadc..a3611d0 100644 --- a/include/openvswitch/nsh.h +++ b/include/openvswitch/nsh.h @@ -113,7 +113,7 @@ struct nsh_hdr { static inline uint16_t nsh_hdr_len(const struct nsh_hdr *nsh) { - return 4 * (ntohs(nsh->ver_flags_len) & NSH_LEN_MASK) >> NSH_LEN_SHIFT; + return ((ntohs(nsh->ver_flags_len) & NSH_LEN_MASK) >> NSH_LEN_SHIFT) << 2; } static inline struct nsh_md1_ctx * -- 2.1.0 _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
