fix json inlining #9
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
fw2568
pushed a commit
that referenced
this pull request
Nov 1, 2022
SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior in
lib/dpif-netlink.c:1077:40: runtime error:
left shift of 1 by 31 places cannot be represented in type 'int'
#0 0x73fc31 in dpif_netlink_port_add_compat lib/dpif-netlink.c:1077:40
#1 0x73fc31 in dpif_netlink_port_add lib/dpif-netlink.c:1132:17
#2 0x2c1745 in dpif_port_add lib/dpif.c:597:13
#3 0x07b279 in port_add ofproto/ofproto-dpif.c:3957:17
#4 0x01b209 in ofproto_port_add ofproto/ofproto.c:2124:13
#5 0xfdbfce in iface_do_create vswitchd/bridge.c:2066:13
#6 0xfdbfce in iface_create vswitchd/bridge.c:2109:13
#7 0xfdbfce in bridge_add_ports__ vswitchd/bridge.c:1173:21
#8 0xfb5319 in bridge_add_ports vswitchd/bridge.c:1189:5
#9 0xfb5319 in bridge_reconfigure vswitchd/bridge.c:901:9
#10 0xfae0f9 in bridge_run vswitchd/bridge.c:3334:9
#11 0xfe67dd in main vswitchd/ovs-vswitchd.c:129:9
#12 0x4b6d8f (/lib/x86_64-linux-gnu/libc.so.6+0x29d8f)
#13 0x4b6e3f in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x29e3f)
#14 0x562594eed024 in _start (vswitchd/ovs-vswitchd+0x787024)
Fixes: 526df7d ("tunnel: Provide framework for tunnel extensions for VXLAN-GBP and others")
Acked-by: Mike Pattrick <mkp@redhat.com>
Signed-off-by: Ilya Maximets <i.maximets@ovn.org>
fw2568
added a commit
that referenced
this pull request
Feb 5, 2024
* Windows build for master branch (#8) * merged windows build * Update build-and-test.yml Removed windows steps from build-and-test workflow * Update build-windows.yml rename windows build workflow (no tests active) * fix json inlining (#9) restored ovs upstream json inline * remove pthreads code after build (#10) * improved windows build workflow (#12) * generate artificats for windows * fix install step in windows workflow * copy pthreads * added TargetVersion for Analyze configurations (#11) * Update README.rst * driver branding (#13) * Revert "driver branding (#13)" (#14) This reverts commit 8f437fd. * dbosoft branding (#15) * branding * branding: fixed check for extension * Update build-windows.yml (#16) * fixed version variable (#17) * Build with SSL enabled (#18) - Update build-windows.yml to download openssl - openssl 1.1 renamed libraries * update appveyor openssl download to openssl 1.1 * OpenSSL DLLs are missing in build output (#19) copy DLLs of openssl 1.1 to c:\openvswitch * fix BSOD in kernel driver (#20) In case OVS_KEY_ATTR_IN_PORT is not in netlink attributes a BSOD will occur. * Driver: windows server 2016 compatibility (#21) * lower driver build version for server 2016 * use ExAllocatePoolPriorityUninitialized * Update build-windows.yml * disable SSL build disable ssl to build server issue - cannot find headers * disable parallel build on windows option -j sometimes cause build errors currently
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
reverted json.c changes in dbosoft branch and inline json commands again, but this time without causing a fault at runtime.