On 6/24/22 14:55, Ilya Maximets wrote:
>   SUMMARY: UndefinedBehaviorSanitizer:
> 
>   lib/odp-util.c:3436:32: runtime error:
>     load of misaligned address 0x624000489424 for type 'const ovs_be64'
>     (aka 'const unsigned long'), which requires 8 byte alignment 
> 0x624000489424:
>     note: pointer points here
>       0c 00 00 00 ff ff ff ff  ff ff ff ff 08 00 01 00  ...
>                   ^
>    0 0x9b13a2 in format_be64 lib/odp-util.c:3436:32
>    1 0x9b13a2 in format_odp_tun_attr lib/odp-util.c:3942:13
>    2 0x9b13a2 in format_odp_key_attr__ lib/odp-util.c:4221:9
>    3 0x9ae7a2 in odp_flow_format lib/odp-util.c:4606:17
>    4 0xee5037 in format_dpif_flow lib/dpctl.c:862:5
>    5 0xed69ed in dpctl_dump_flows lib/dpctl.c:1142:13
>    6 0xed32b3 in dpctl_unixctl_handler lib/dpctl.c:3035:17
>    7 0xc7c80b in process_command lib/unixctl.c:310:13
>    8 0xc7c80b in run_connection lib/unixctl.c:344:17
>    9 0xc7c80b in unixctl_server_run lib/unixctl.c:395:21
>    10 0x59a9a4 in main vswitchd/ovs-vswitchd.c:130:9
>    11 0x7fee2803acf2 in __libc_start_main (/lib64/libc.so.6+0x3acf2)
>    12 0x47e60d in _start (vswitchd/ovs-vswitchd+0x47e60d)
> 
> Tunnel id mask in the flow key is only 4 bytes aligned, so has to be
> accessed with appropriate unaligned read function.
> 
> Signed-off-by: Ilya Maximets <[email protected]>
> ---

Looks good to me, thanks!

Acked-by: Dumitru Ceara <[email protected]>

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to