[ovs-git] [openvswitch/ovs] efa9b5: unaligned: Introduce helpers for 32-bit aligned 12...

GitHub noreply at github.com
Wed Jun 14 19:48:14 UTC 2017


  Branch: refs/heads/branch-2.7
  Home:   https://github.com/openvswitch/ovs
  Commit: efa9b5230599a772a2aa91a4852123f6217eea8b
      https://github.com/openvswitch/ovs/commit/efa9b5230599a772a2aa91a4852123f6217eea8b
  Author: Ben Pfaff <blp at ovn.org>
  Date:   2017-06-14 (Wed, 14 Jun 2017)

  Changed paths:
    M include/openvswitch/types.h
    M lib/unaligned.h

  Log Message:
  -----------
  unaligned: Introduce helpers for 32-bit aligned 128-bit integers.

These are analogous to the existing helpers for 32-bit aligned 64-bit
integers, and will have users in upcoming commits.

Signed-off-by: Ben Pfaff <blp at ovn.org>
Acked-by: Lance Richardson <lrichard at redhat.com>


  Commit: 0236ef1ea88fe057721d7f99aa23f9d0457e3aea
      https://github.com/openvswitch/ovs/commit/0236ef1ea88fe057721d7f99aa23f9d0457e3aea
  Author: Ben Pfaff <blp at ovn.org>
  Date:   2017-06-14 (Wed, 14 Jun 2017)

  Changed paths:
    M lib/dpif-netlink.c
    M lib/netlink.c
    M lib/netlink.h
    M lib/odp-util.c

  Log Message:
  -----------
  netlink: Introduce helpers for 128-bit integer attributes.

Use the helpers in appropriate places.  In most cases, this fixes a
misaligned reference, since ovs_be128 and ovs_u128 require 8-byte alignment
but Netlink only guarantees 4-byte.

Found by GCC -fsanitize=undefined.

Reported-by: Lance Richardson <lrichard at redhat.com>
Signed-off-by: Ben Pfaff <blp at ovn.org>
Acked-by: Lance Richardson <lrichard at redhat.com>


  Commit: 2ff50592e61478c1a95c0c25ae92daafcba38329
      https://github.com/openvswitch/ovs/commit/2ff50592e61478c1a95c0c25ae92daafcba38329
  Author: Ben Pfaff <blp at ovn.org>
  Date:   2017-06-14 (Wed, 14 Jun 2017)

  Changed paths:
    M lib/odp-util.c
    M ofproto/ofproto-dpif-xlate.c

  Log Message:
  -----------
  ofproto-dpif-xlate: Fix an additional misaligned reference to ovs_u128.

nl_msg_put_unspec_uninit() can return a pointer that is only 4-byte
aligned.

Signed-off-by: Ben Pfaff <blp at ovn.org>
Acked-by: Lance Richardson <lrichard at redhat.com>


Compare: https://github.com/openvswitch/ovs/compare/7110767757f9...2ff50592e614


More information about the git mailing list