Is it a typo in check the validity of nat->logical_ip? #42

ateleshev · 2020-05-07T10:24:15Z

https://github.com/ovn-org/ovn/blob/master/northd/ovn-northd.c#L8831

For ipv4 used nat->logical_ip, but for ipv6 nat->external_ip:

            /* Check the validity of nat->logical_ip. 'logical_ip' can
             * be a subnet when the type is "snat". */
            if (is_v6) {
                error = ipv6_parse_masked(nat->external_ip, &ipv6, &mask_v6);
            } else {
                error = ip_parse_masked(nat->logical_ip, &ip, &mask);
            }

Can some one explain, why do we use nat->external_ip here?

The text was updated successfully, but these errors were encountered:

Reported-at: #42 Submitted-at: #43 Signed-off-by: Artem Teleshev <artem.teleshev@gmail.com> Signed-off-by: Numan Siddique <numans@ovn.org>

Reported-at: #42 Submitted-at: #43 Signed-off-by: Artem Teleshev <artem.teleshev@gmail.com> Signed-off-by: Numan Siddique <numans@ovn.org> (cherry-picked from master commit 5d80bb5)

@pg1

Avoid the following crash in ovn-trace due to a reject action infinite loop: AddressSanitizer:DEADLYSIGNAL ==569410==ERROR: AddressSanitizer: stack-overflow on address 0x7ffc0a991bc4 (pc 0x000000415081 bp 0x7ffc0a993fa0 sp 0x7ffc0a991b30 T0) #0 0x415081 in trace_actions utilities/ovn-trace.c:2617 ovn-org#1 0x41d878 in trace_actions utilities/ovn-trace.c:2622 ovn-org#2 0x41d878 in trace__ utilities/ovn-trace.c:3007 ovn-org#3 0x41e3a8 in execute_output utilities/ovn-trace.c:1700 ovn-org#4 0x416fcf in trace_actions utilities/ovn-trace.c:2640 ovn-org#5 0x41d878 in trace_actions utilities/ovn-trace.c:2622 ovn-org#6 0x41d878 in trace__ utilities/ovn-trace.c:3007 ovn-org#7 0x41788d in execute_next utilities/ovn-trace.c:2307 ovn-org#8 0x41788d in trace_actions utilities/ovn-trace.c:2644 ovn-org#9 0x41a7c0 in trace_actions utilities/ovn-trace.c:2622 ovn-org#10 0x41a7c0 in execute_sctp4_abort utilities/ovn-trace.c:1985 ovn-org#11 0x419671 in execute_reject utilities/ovn-trace.c:2049 ovn-org#12 0x419671 in trace_actions utilities/ovn-trace.c:2860 ovn-org#13 0x41d878 in trace_actions utilities/ovn-trace.c:2622 ovn-org#14 0x41d878 in trace__ utilities/ovn-trace.c:3007 ovn-org#15 0x41e3a8 in execute_output utilities/ovn-trace.c:1700 ovn-org#16 0x416fcf in trace_actions utilities/ovn-trace.c:2640 ovn-org#17 0x41d878 in trace_actions utilities/ovn-trace.c:2622 ovn-org#18 0x41d878 in trace__ utilities/ovn-trace.c:3007 ovn-org#19 0x41788d in execute_next utilities/ovn-trace.c:2307 ovn-org#20 0x41788d in trace_actions utilities/ovn-trace.c:2644 ovn-org#21 0x41a7c0 in trace_actions utilities/ovn-trace.c:2622 ovn-org#22 0x41a7c0 in execute_sctp4_abort utilities/ovn-trace.c:1985 ovn-org#23 0x419671 in execute_reject utilities/ovn-trace.c:2049 ovn-org#24 0x419671 in trace_actions utilities/ovn-trace.c:2860 ovn-org#25 0x41d878 in trace_actions utilities/ovn-trace.c:2622 ovn-org#26 0x41d878 in trace__ utilities/ovn-trace.c:3007 ovn-org#27 0x41e3a8 in execute_output utilities/ovn-trace.c:1700 ovn-org#28 0x416fcf in trace_actions utilities/ovn-trace.c:2640 ovn-org#29 0x41d878 in trace_actions utilities/ovn-trace.c:2622 ovn-org#30 0x41d878 in trace__ utilities/ovn-trace.c:3007 ovn-org#31 0x41788d in execute_next utilities/ovn-trace.c:2307 ovn-org#32 0x41788d in trace_actions utilities/ovn-trace.c:2644 ovn-org#33 0x41a7c0 in trace_actions utilities/ovn-trace.c:2622 ovn-org#34 0x41a7c0 in execute_sctp4_abort utilities/ovn-trace.c:1985 ovn-org#35 0x419671 in execute_reject utilities/ovn-trace.c:2049 ovn-org#36 0x419671 in trace_actions utilities/ovn-trace.c:2860 ovn-org#37 0x41d878 in trace_actions utilities/ovn-trace.c:2622 ovn-org#38 0x41d878 in trace__ utilities/ovn-trace.c:3007 ovn-org#39 0x41e3a8 in execute_output utilities/ovn-trace.c:1700 ovn-org#40 0x416fcf in trace_actions utilities/ovn-trace.c:2640 ovn-org#41 0x41d878 in trace_actions utilities/ovn-trace.c:2622 ovn-org#42 0x41d878 in trace__ utilities/ovn-trace.c:3007 ovn-org#43 0x41788d in execute_next utilities/ovn-trace.c:2307 .... The issue can be triggered with the following reproducer: $ovn-nbctl ls-add sw $ovn-nbctl lsp-add sw p1 $ovn-nbctl lsp-set-addresses p1 "00:00:00:00:00:02 192.168.0.2" $ovn-nbctl lsp-add sw p2 $ovn-nbctl lsp-set-addresses p2 "00:00:00:00:00:03 192.168.0.3" $ovn-nbctl pg-add pg1 p1 p2 $ovn-nbctl acl-add pg1 to-lport 2003 "inport==@pg1 && ip4 && ip4.src == 192.168.0.0/16 && udp && udp.dst == 9000" allow $ovn-nbctl acl-add pg1 to-lport 2001 "inport==@pg1" reject $ovn-trace 'inport == "p2" && eth.src == 00:00:00:00:00:03 && eth.dst == 00:00:00:00:00:02 && ip.ttl==42 && ip4.src == 192.168.0.3' Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2074537

@pg1

Avoid the following crash in ovn-trace due to a reject action infinite loop: AddressSanitizer:DEADLYSIGNAL ==569410==ERROR: AddressSanitizer: stack-overflow on address 0x7ffc0a991bc4 (pc 0x000000415081 bp 0x7ffc0a993fa0 sp 0x7ffc0a991b30 T0) #0 0x415081 in trace_actions utilities/ovn-trace.c:2617 ovn-org#1 0x41d878 in trace_actions utilities/ovn-trace.c:2622 ovn-org#2 0x41d878 in trace__ utilities/ovn-trace.c:3007 ovn-org#3 0x41e3a8 in execute_output utilities/ovn-trace.c:1700 ovn-org#4 0x416fcf in trace_actions utilities/ovn-trace.c:2640 ovn-org#5 0x41d878 in trace_actions utilities/ovn-trace.c:2622 ovn-org#6 0x41d878 in trace__ utilities/ovn-trace.c:3007 ovn-org#7 0x41788d in execute_next utilities/ovn-trace.c:2307 ovn-org#8 0x41788d in trace_actions utilities/ovn-trace.c:2644 ovn-org#9 0x41a7c0 in trace_actions utilities/ovn-trace.c:2622 ovn-org#10 0x41a7c0 in execute_sctp4_abort utilities/ovn-trace.c:1985 ovn-org#11 0x419671 in execute_reject utilities/ovn-trace.c:2049 ovn-org#12 0x419671 in trace_actions utilities/ovn-trace.c:2860 ovn-org#13 0x41d878 in trace_actions utilities/ovn-trace.c:2622 ovn-org#14 0x41d878 in trace__ utilities/ovn-trace.c:3007 ovn-org#15 0x41e3a8 in execute_output utilities/ovn-trace.c:1700 ovn-org#16 0x416fcf in trace_actions utilities/ovn-trace.c:2640 ovn-org#17 0x41d878 in trace_actions utilities/ovn-trace.c:2622 ovn-org#18 0x41d878 in trace__ utilities/ovn-trace.c:3007 ovn-org#19 0x41788d in execute_next utilities/ovn-trace.c:2307 ovn-org#20 0x41788d in trace_actions utilities/ovn-trace.c:2644 ovn-org#21 0x41a7c0 in trace_actions utilities/ovn-trace.c:2622 ovn-org#22 0x41a7c0 in execute_sctp4_abort utilities/ovn-trace.c:1985 ovn-org#23 0x419671 in execute_reject utilities/ovn-trace.c:2049 ovn-org#24 0x419671 in trace_actions utilities/ovn-trace.c:2860 ovn-org#25 0x41d878 in trace_actions utilities/ovn-trace.c:2622 ovn-org#26 0x41d878 in trace__ utilities/ovn-trace.c:3007 ovn-org#27 0x41e3a8 in execute_output utilities/ovn-trace.c:1700 ovn-org#28 0x416fcf in trace_actions utilities/ovn-trace.c:2640 ovn-org#29 0x41d878 in trace_actions utilities/ovn-trace.c:2622 ovn-org#30 0x41d878 in trace__ utilities/ovn-trace.c:3007 ovn-org#31 0x41788d in execute_next utilities/ovn-trace.c:2307 ovn-org#32 0x41788d in trace_actions utilities/ovn-trace.c:2644 ovn-org#33 0x41a7c0 in trace_actions utilities/ovn-trace.c:2622 ovn-org#34 0x41a7c0 in execute_sctp4_abort utilities/ovn-trace.c:1985 ovn-org#35 0x419671 in execute_reject utilities/ovn-trace.c:2049 ovn-org#36 0x419671 in trace_actions utilities/ovn-trace.c:2860 ovn-org#37 0x41d878 in trace_actions utilities/ovn-trace.c:2622 ovn-org#38 0x41d878 in trace__ utilities/ovn-trace.c:3007 ovn-org#39 0x41e3a8 in execute_output utilities/ovn-trace.c:1700 ovn-org#40 0x416fcf in trace_actions utilities/ovn-trace.c:2640 ovn-org#41 0x41d878 in trace_actions utilities/ovn-trace.c:2622 ovn-org#42 0x41d878 in trace__ utilities/ovn-trace.c:3007 ovn-org#43 0x41788d in execute_next utilities/ovn-trace.c:2307 .... The issue can be triggered with the following reproducer: $ovn-nbctl ls-add sw $ovn-nbctl lsp-add sw p1 $ovn-nbctl lsp-set-addresses p1 "00:00:00:00:00:02 192.168.0.2" $ovn-nbctl lsp-add sw p2 $ovn-nbctl lsp-set-addresses p2 "00:00:00:00:00:03 192.168.0.3" $ovn-nbctl pg-add pg1 p1 p2 $ovn-nbctl acl-add pg1 to-lport 2003 "inport==@pg1 && ip4 && ip4.src == 192.168.0.0/16 && udp && udp.dst == 9000" allow $ovn-nbctl acl-add pg1 to-lport 2001 "inport==@pg1" reject $ovn-trace 'inport == "p2" && eth.src == 00:00:00:00:00:03 && eth.dst == 00:00:00:00:00:02 && ip.ttl==42 && ip4.src == 192.168.0.3' Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2074537 Signed-off-by: Lorenzo Bianconi <lorenzo.bianconi@redhat.com> Signed-off-by: 0-day Robot <robot@bytheb.org>

@pg1

Avoid the following crash in ovn-trace due to a reject action infinite loop: AddressSanitizer:DEADLYSIGNAL ==569410==ERROR: AddressSanitizer: stack-overflow on address 0x7ffc0a991bc4 (pc 0x000000415081 bp 0x7ffc0a993fa0 sp 0x7ffc0a991b30 T0) #0 0x415081 in trace_actions utilities/ovn-trace.c:2617 #1 0x41d878 in trace_actions utilities/ovn-trace.c:2622 #2 0x41d878 in trace__ utilities/ovn-trace.c:3007 #3 0x41e3a8 in execute_output utilities/ovn-trace.c:1700 #4 0x416fcf in trace_actions utilities/ovn-trace.c:2640 #5 0x41d878 in trace_actions utilities/ovn-trace.c:2622 #6 0x41d878 in trace__ utilities/ovn-trace.c:3007 #7 0x41788d in execute_next utilities/ovn-trace.c:2307 #8 0x41788d in trace_actions utilities/ovn-trace.c:2644 #9 0x41a7c0 in trace_actions utilities/ovn-trace.c:2622 #10 0x41a7c0 in execute_sctp4_abort utilities/ovn-trace.c:1985 #11 0x419671 in execute_reject utilities/ovn-trace.c:2049 #12 0x419671 in trace_actions utilities/ovn-trace.c:2860 #13 0x41d878 in trace_actions utilities/ovn-trace.c:2622 #14 0x41d878 in trace__ utilities/ovn-trace.c:3007 #15 0x41e3a8 in execute_output utilities/ovn-trace.c:1700 #16 0x416fcf in trace_actions utilities/ovn-trace.c:2640 #17 0x41d878 in trace_actions utilities/ovn-trace.c:2622 #18 0x41d878 in trace__ utilities/ovn-trace.c:3007 #19 0x41788d in execute_next utilities/ovn-trace.c:2307 #20 0x41788d in trace_actions utilities/ovn-trace.c:2644 #21 0x41a7c0 in trace_actions utilities/ovn-trace.c:2622 #22 0x41a7c0 in execute_sctp4_abort utilities/ovn-trace.c:1985 #23 0x419671 in execute_reject utilities/ovn-trace.c:2049 #24 0x419671 in trace_actions utilities/ovn-trace.c:2860 #25 0x41d878 in trace_actions utilities/ovn-trace.c:2622 #26 0x41d878 in trace__ utilities/ovn-trace.c:3007 #27 0x41e3a8 in execute_output utilities/ovn-trace.c:1700 #28 0x416fcf in trace_actions utilities/ovn-trace.c:2640 #29 0x41d878 in trace_actions utilities/ovn-trace.c:2622 #30 0x41d878 in trace__ utilities/ovn-trace.c:3007 #31 0x41788d in execute_next utilities/ovn-trace.c:2307 #32 0x41788d in trace_actions utilities/ovn-trace.c:2644 #33 0x41a7c0 in trace_actions utilities/ovn-trace.c:2622 #34 0x41a7c0 in execute_sctp4_abort utilities/ovn-trace.c:1985 #35 0x419671 in execute_reject utilities/ovn-trace.c:2049 #36 0x419671 in trace_actions utilities/ovn-trace.c:2860 #37 0x41d878 in trace_actions utilities/ovn-trace.c:2622 #38 0x41d878 in trace__ utilities/ovn-trace.c:3007 #39 0x41e3a8 in execute_output utilities/ovn-trace.c:1700 #40 0x416fcf in trace_actions utilities/ovn-trace.c:2640 #41 0x41d878 in trace_actions utilities/ovn-trace.c:2622 #42 0x41d878 in trace__ utilities/ovn-trace.c:3007 #43 0x41788d in execute_next utilities/ovn-trace.c:2307 .... The issue can be triggered with the following reproducer: $ovn-nbctl ls-add sw $ovn-nbctl lsp-add sw p1 $ovn-nbctl lsp-set-addresses p1 "00:00:00:00:00:02 192.168.0.2" $ovn-nbctl lsp-add sw p2 $ovn-nbctl lsp-set-addresses p2 "00:00:00:00:00:03 192.168.0.3" $ovn-nbctl pg-add pg1 p1 p2 $ovn-nbctl acl-add pg1 to-lport 2003 "inport==@pg1 && ip4 && ip4.src == 192.168.0.0/16 && udp && udp.dst == 9000" allow $ovn-nbctl acl-add pg1 to-lport 2001 "inport==@pg1" reject $ovn-trace 'inport == "p2" && eth.src == 00:00:00:00:00:03 && eth.dst == 00:00:00:00:00:02 && ip.ttl==42 && ip4.src == 192.168.0.3' Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2074537 Signed-off-by: Lorenzo Bianconi <lorenzo.bianconi@redhat.com> Acked-by: Mark Michelson <mmichels@redhat.com> Signed-off-by: Mark Michelson <mmichels@redhat.com>

@pg1

Avoid the following crash in ovn-trace due to a reject action infinite loop: AddressSanitizer:DEADLYSIGNAL ==569410==ERROR: AddressSanitizer: stack-overflow on address 0x7ffc0a991bc4 (pc 0x000000415081 bp 0x7ffc0a993fa0 sp 0x7ffc0a991b30 T0) #0 0x415081 in trace_actions utilities/ovn-trace.c:2617 #1 0x41d878 in trace_actions utilities/ovn-trace.c:2622 #2 0x41d878 in trace__ utilities/ovn-trace.c:3007 #3 0x41e3a8 in execute_output utilities/ovn-trace.c:1700 #4 0x416fcf in trace_actions utilities/ovn-trace.c:2640 #5 0x41d878 in trace_actions utilities/ovn-trace.c:2622 #6 0x41d878 in trace__ utilities/ovn-trace.c:3007 #7 0x41788d in execute_next utilities/ovn-trace.c:2307 #8 0x41788d in trace_actions utilities/ovn-trace.c:2644 #9 0x41a7c0 in trace_actions utilities/ovn-trace.c:2622 #10 0x41a7c0 in execute_sctp4_abort utilities/ovn-trace.c:1985 #11 0x419671 in execute_reject utilities/ovn-trace.c:2049 #12 0x419671 in trace_actions utilities/ovn-trace.c:2860 #13 0x41d878 in trace_actions utilities/ovn-trace.c:2622 #14 0x41d878 in trace__ utilities/ovn-trace.c:3007 #15 0x41e3a8 in execute_output utilities/ovn-trace.c:1700 #16 0x416fcf in trace_actions utilities/ovn-trace.c:2640 #17 0x41d878 in trace_actions utilities/ovn-trace.c:2622 #18 0x41d878 in trace__ utilities/ovn-trace.c:3007 #19 0x41788d in execute_next utilities/ovn-trace.c:2307 #20 0x41788d in trace_actions utilities/ovn-trace.c:2644 #21 0x41a7c0 in trace_actions utilities/ovn-trace.c:2622 #22 0x41a7c0 in execute_sctp4_abort utilities/ovn-trace.c:1985 #23 0x419671 in execute_reject utilities/ovn-trace.c:2049 #24 0x419671 in trace_actions utilities/ovn-trace.c:2860 #25 0x41d878 in trace_actions utilities/ovn-trace.c:2622 #26 0x41d878 in trace__ utilities/ovn-trace.c:3007 #27 0x41e3a8 in execute_output utilities/ovn-trace.c:1700 #28 0x416fcf in trace_actions utilities/ovn-trace.c:2640 #29 0x41d878 in trace_actions utilities/ovn-trace.c:2622 #30 0x41d878 in trace__ utilities/ovn-trace.c:3007 #31 0x41788d in execute_next utilities/ovn-trace.c:2307 #32 0x41788d in trace_actions utilities/ovn-trace.c:2644 #33 0x41a7c0 in trace_actions utilities/ovn-trace.c:2622 #34 0x41a7c0 in execute_sctp4_abort utilities/ovn-trace.c:1985 #35 0x419671 in execute_reject utilities/ovn-trace.c:2049 #36 0x419671 in trace_actions utilities/ovn-trace.c:2860 #37 0x41d878 in trace_actions utilities/ovn-trace.c:2622 #38 0x41d878 in trace__ utilities/ovn-trace.c:3007 #39 0x41e3a8 in execute_output utilities/ovn-trace.c:1700 #40 0x416fcf in trace_actions utilities/ovn-trace.c:2640 #41 0x41d878 in trace_actions utilities/ovn-trace.c:2622 #42 0x41d878 in trace__ utilities/ovn-trace.c:3007 #43 0x41788d in execute_next utilities/ovn-trace.c:2307 .... The issue can be triggered with the following reproducer: $ovn-nbctl ls-add sw $ovn-nbctl lsp-add sw p1 $ovn-nbctl lsp-set-addresses p1 "00:00:00:00:00:02 192.168.0.2" $ovn-nbctl lsp-add sw p2 $ovn-nbctl lsp-set-addresses p2 "00:00:00:00:00:03 192.168.0.3" $ovn-nbctl pg-add pg1 p1 p2 $ovn-nbctl acl-add pg1 to-lport 2003 "inport==@pg1 && ip4 && ip4.src == 192.168.0.0/16 && udp && udp.dst == 9000" allow $ovn-nbctl acl-add pg1 to-lport 2001 "inport==@pg1" reject $ovn-trace 'inport == "p2" && eth.src == 00:00:00:00:00:03 && eth.dst == 00:00:00:00:00:02 && ip.ttl==42 && ip4.src == 192.168.0.3' Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2074537 Signed-off-by: Lorenzo Bianconi <lorenzo.bianconi@redhat.com> Acked-by: Mark Michelson <mmichels@redhat.com> Signed-off-by: Mark Michelson <mmichels@redhat.com>

@pg1

Avoid the following crash in ovn-trace due to a reject action infinite loop: AddressSanitizer:DEADLYSIGNAL ==569410==ERROR: AddressSanitizer: stack-overflow on address 0x7ffc0a991bc4 (pc 0x000000415081 bp 0x7ffc0a993fa0 sp 0x7ffc0a991b30 T0) #0 0x415081 in trace_actions utilities/ovn-trace.c:2617 #1 0x41d878 in trace_actions utilities/ovn-trace.c:2622 #2 0x41d878 in trace__ utilities/ovn-trace.c:3007 #3 0x41e3a8 in execute_output utilities/ovn-trace.c:1700 #4 0x416fcf in trace_actions utilities/ovn-trace.c:2640 #5 0x41d878 in trace_actions utilities/ovn-trace.c:2622 #6 0x41d878 in trace__ utilities/ovn-trace.c:3007 #7 0x41788d in execute_next utilities/ovn-trace.c:2307 #8 0x41788d in trace_actions utilities/ovn-trace.c:2644 #9 0x41a7c0 in trace_actions utilities/ovn-trace.c:2622 #10 0x41a7c0 in execute_sctp4_abort utilities/ovn-trace.c:1985 #11 0x419671 in execute_reject utilities/ovn-trace.c:2049 #12 0x419671 in trace_actions utilities/ovn-trace.c:2860 #13 0x41d878 in trace_actions utilities/ovn-trace.c:2622 #14 0x41d878 in trace__ utilities/ovn-trace.c:3007 #15 0x41e3a8 in execute_output utilities/ovn-trace.c:1700 #16 0x416fcf in trace_actions utilities/ovn-trace.c:2640 #17 0x41d878 in trace_actions utilities/ovn-trace.c:2622 #18 0x41d878 in trace__ utilities/ovn-trace.c:3007 #19 0x41788d in execute_next utilities/ovn-trace.c:2307 #20 0x41788d in trace_actions utilities/ovn-trace.c:2644 #21 0x41a7c0 in trace_actions utilities/ovn-trace.c:2622 #22 0x41a7c0 in execute_sctp4_abort utilities/ovn-trace.c:1985 #23 0x419671 in execute_reject utilities/ovn-trace.c:2049 #24 0x419671 in trace_actions utilities/ovn-trace.c:2860 #25 0x41d878 in trace_actions utilities/ovn-trace.c:2622 #26 0x41d878 in trace__ utilities/ovn-trace.c:3007 #27 0x41e3a8 in execute_output utilities/ovn-trace.c:1700 #28 0x416fcf in trace_actions utilities/ovn-trace.c:2640 #29 0x41d878 in trace_actions utilities/ovn-trace.c:2622 #30 0x41d878 in trace__ utilities/ovn-trace.c:3007 #31 0x41788d in execute_next utilities/ovn-trace.c:2307 #32 0x41788d in trace_actions utilities/ovn-trace.c:2644 #33 0x41a7c0 in trace_actions utilities/ovn-trace.c:2622 #34 0x41a7c0 in execute_sctp4_abort utilities/ovn-trace.c:1985 #35 0x419671 in execute_reject utilities/ovn-trace.c:2049 #36 0x419671 in trace_actions utilities/ovn-trace.c:2860 #37 0x41d878 in trace_actions utilities/ovn-trace.c:2622 #38 0x41d878 in trace__ utilities/ovn-trace.c:3007 #39 0x41e3a8 in execute_output utilities/ovn-trace.c:1700 #40 0x416fcf in trace_actions utilities/ovn-trace.c:2640 #41 0x41d878 in trace_actions utilities/ovn-trace.c:2622 #42 0x41d878 in trace__ utilities/ovn-trace.c:3007 #43 0x41788d in execute_next utilities/ovn-trace.c:2307 .... The issue can be triggered with the following reproducer: $ovn-nbctl ls-add sw $ovn-nbctl lsp-add sw p1 $ovn-nbctl lsp-set-addresses p1 "00:00:00:00:00:02 192.168.0.2" $ovn-nbctl lsp-add sw p2 $ovn-nbctl lsp-set-addresses p2 "00:00:00:00:00:03 192.168.0.3" $ovn-nbctl pg-add pg1 p1 p2 $ovn-nbctl acl-add pg1 to-lport 2003 "inport==@pg1 && ip4 && ip4.src == 192.168.0.0/16 && udp && udp.dst == 9000" allow $ovn-nbctl acl-add pg1 to-lport 2001 "inport==@pg1" reject $ovn-trace 'inport == "p2" && eth.src == 00:00:00:00:00:03 && eth.dst == 00:00:00:00:00:02 && ip.ttl==42 && ip4.src == 192.168.0.3' Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2074537 Signed-off-by: Lorenzo Bianconi <lorenzo.bianconi@redhat.com> Acked-by: Mark Michelson <mmichels@redhat.com> Signed-off-by: Mark Michelson <mmichels@redhat.com>

ateleshev mentioned this issue May 7, 2020

Fix typo for validation of logical ip in ipv6 mode #43

Closed

numansiddique pushed a commit that referenced this issue May 8, 2020

Fix typo for validation of logical ip in ipv6 mode.

5d80bb5

Reported-at: #42 Submitted-at: #43 Signed-off-by: Artem Teleshev <artem.teleshev@gmail.com> Signed-off-by: Numan Siddique <numans@ovn.org>

numansiddique closed this as completed May 11, 2020

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Is it a typo in check the validity of nat->logical_ip? #42

Is it a typo in check the validity of nat->logical_ip? #42

ateleshev commented May 7, 2020 •

edited

Loading

Is it a typo in check the validity of nat->logical_ip? #42

Is it a typo in check the validity of nat->logical_ip? #42

Comments

ateleshev commented May 7, 2020 • edited Loading

ateleshev commented May 7, 2020 •

edited

Loading