Re: [ovs-dev] [PATCH] treewide: undefined behavior, passing null in nonnull parameters

2017-06-14 Thread Ben Pfaff
On Tue, Jun 13, 2017 at 01:08:52PM -0400, Lance Richardson wrote: > This patch addresses all ubsan errors of the "null vs. nonnull" flavor. > The remaining errors are: > > $ grep runtime tests/testsuite.dir/*/testsuite.log > tests/testsuite.dir/0044/testsuite.log:+tests/test-hash.c:59:40: runtime

Re: [ovs-dev] [PATCH] treewide: undefined behavior, passing null in nonnull parameters

2017-06-13 Thread Lance Richardson
> From: "Lance Richardson" > To: d...@openvswitch.org > Sent: Tuesday, 13 June, 2017 12:57:38 PM > Subject: [ovs-dev] [PATCH] treewide: undefined behavior, passing null in > nonnull parameters > > Eliminate a number of instances of undefined behavior related

Re: [ovs-dev] [PATCH] treewide: undefined behavior, passing null in nonnull parameters

2017-06-13 Thread Ben Pfaff
On Tue, Jun 13, 2017 at 12:57:38PM -0400, Lance Richardson wrote: > Eliminate a number of instances of undefined behavior related to > passing NULL in parameters having "nonnull" annotations. > > Found with gcc's undefined behavior sanitizer. > > Signed-off-by: Lance Richardson Thanks, applied

[ovs-dev] [PATCH] treewide: undefined behavior, passing null in nonnull parameters

2017-06-13 Thread Lance Richardson
Eliminate a number of instances of undefined behavior related to passing NULL in parameters having "nonnull" annotations. Found with gcc's undefined behavior sanitizer. Signed-off-by: Lance Richardson --- lib/netlink.c | 5 - lib/ofpbuf.c | 2 +- lib/svec.c