On 6/24/22 20:49, Aaron Conole wrote:
> Ilya Maximets <[email protected]> writes:
> 
>>   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]>
>> ---
> 
> Acked-by: Aaron Conole <[email protected]>
> 

Thanks, Aaron and Dumitru!  Applied and backported down to 2.13.

Best regards, Ilya Maximets.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to